Return-Path: X-Original-To: apmail-couchdb-dev-archive@www.apache.org Delivered-To: apmail-couchdb-dev-archive@www.apache.org Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by minotaur.apache.org (Postfix) with SMTP id 375759C8B for ; Tue, 14 Feb 2012 19:36:53 +0000 (UTC) Received: (qmail 45762 invoked by uid 500); 14 Feb 2012 19:36:52 -0000 Delivered-To: apmail-couchdb-dev-archive@couchdb.apache.org Received: (qmail 45724 invoked by uid 500); 14 Feb 2012 19:36:52 -0000 Mailing-List: contact dev-help@couchdb.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@couchdb.apache.org Delivered-To: mailing list dev@couchdb.apache.org Received: (qmail 45715 invoked by uid 99); 14 Feb 2012 19:36:52 -0000 Received: from athena.apache.org (HELO athena.apache.org) (140.211.11.136) by apache.org (qpsmtpd/0.29) with ESMTP; Tue, 14 Feb 2012 19:36:52 +0000 X-ASF-Spam-Status: No, hits=-0.7 required=5.0 tests=RCVD_IN_DNSWL_LOW,SPF_PASS X-Spam-Check-By: apache.org Received-SPF: pass (athena.apache.org: domain of randall.leeds@gmail.com designates 209.85.160.52 as permitted sender) Received: from [209.85.160.52] (HELO mail-pw0-f52.google.com) (209.85.160.52) by apache.org (qpsmtpd/0.29) with ESMTP; Tue, 14 Feb 2012 19:36:46 +0000 Received: by pbbjt11 with SMTP id jt11so964180pbb.11 for ; Tue, 14 Feb 2012 11:36:26 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=gamma; h=mime-version:in-reply-to:references:date:message-id:subject:from:to :content-type:content-transfer-encoding; bh=lg6J2Fyyty7BuNIaNdqgKJkYi+sXpkv7eM/IK+3kuF4=; b=RiiSMD9MrcuYMXCyNhTlOAY8NVmVcBQr9VhpA8EPLBJHKpoaFuZGfxZT4sW2yxzDUV yxDMi/OcUbTQk79aLGr4+ADoFfHTDHNSADFSDG1NewRssoUD6rj6yXxN7fC33BpSdapP NctVboMImh6F6WqONahdQLTLVBEunvVX56j24= MIME-Version: 1.0 Received: by 10.68.135.137 with SMTP id ps9mr61543623pbb.40.1329248186326; Tue, 14 Feb 2012 11:36:26 -0800 (PST) Received: by 10.68.221.98 with HTTP; Tue, 14 Feb 2012 11:36:26 -0800 (PST) In-Reply-To: References: <55B3B2E5-6FD5-4240-B171-A7768FEFD3EF@apache.org> Date: Tue, 14 Feb 2012 11:36:26 -0800 Message-ID: Subject: Re: Issues blocking the 1.2.0 release From: Randall Leeds To: dev@couchdb.apache.org Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable On Tue, Feb 14, 2012 at 11:06, Benoit Chesneau wrote: > On Tue, Feb 14, 2012 at 7:53 PM, Randall Leeds = wrote: >> On Tue, Feb 14, 2012 at 10:41, Jan Lehnardt wrote: >>> >>> On Feb 14, 2012, at 19:35 , Randall Leeds wrote: >>> >>>> On Tue, Feb 14, 2012 at 10:19, Jan Lehnardt wrote: >>>>> >>>>> On Feb 14, 2012, at 19:13 , Randall Leeds wrote: >>>>> >>>>>> On Tue, Feb 14, 2012 at 04:14, Noah Slater wr= ote: >>>>>>> Devs, >>>>>>> >>>>>>> Please outline: >>>>>>> >>>>>>> =C2=A0 - What has been changed since round one of the 1.2.0 release >>>>>>> =C2=A0 - What remains to be fixed for regression purposes >>>>>>> =C2=A0 - Who is doing these fixes, and when will they be done by >>>>>>> >>>>>>> Thanks, >>>>>>> >>>>>>> N >>>>>> >>>>>> I'd like to know if it was always the case that design doc actions o= n >>>>>> system dbs were inaccessible to non-admins or if that's just since t= he >>>>>> recent security changes. If it's recent, why was that part deemed >>>>>> necessary and can we remove it? >>>>> >>>>> It is part of the recent changes and the reason is that a view potent= ially >>>>> leaks information about docs and we don't want that. I'm happy to rel= ax this >>>>> later if we can convince people to write views that don't compromise = their >>>>> security, but until then I opted for the more secure default. >>>>> >>>> >>>> I motion to remove this restriction now, unless there are actions on >>>> the system dbs, installed by default, that leak anything at all. >>>> I see the motivation but I feel it might be overly paranoid. Only an >>>> admin can modify the ddocs. If a user decides to add views to >>>> _replicator or _user they had best think about what they expose and to >>>> whom. >>>> >>>> If there's no objection I can try to tackle this in the evening. >>> >>> I object :) >> >> Hmm. What's your reasoning? > Why do you need views in _users ? > > - beno=C3=AEt The idea was to make it easy to add public profiles, since ?include_docs is subject to the new security hooks, but emit() could publish the public information. There are valid use cases for admin-only views, which this would prevent, though. In that case, we probably shouldn't change anything. -R