You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@whimsical.apache.org by Sam Ruby <ru...@intertwingly.net> on 2019/05/22 11:40:42 UTC
Re: [whimsy] branch master updated: Clarify karma check
FYI: while the underlying LDAP group is named pmc-chairs, it actually
is a bit of a misnomer. It may include officers who are not pmc
chairs.
- Sam Ruby
On Wed, May 22, 2019 at 4:37 AM <se...@apache.org> wrote:
>
> This is an automated email from the ASF dual-hosted git repository.
>
> sebb pushed a commit to branch master
> in repository https://gitbox.apache.org/repos/asf/whimsy.git
>
>
> The following commit(s) were added to refs/heads/master by this push:
> new ba06c26 Clarify karma check
> ba06c26 is described below
>
> commit ba06c26a342753852e6c48311c5738fc6e6fd33b
> Author: Sebb <se...@apache.org>
> AuthorDate: Wed May 22 09:37:10 2019 +0100
>
> Clarify karma check
> ---
> www/board/list-traffic.cgi | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/www/board/list-traffic.cgi b/www/board/list-traffic.cgi
> index afd6727..a7a1279 100755
> --- a/www/board/list-traffic.cgi
> +++ b/www/board/list-traffic.cgi
> @@ -13,7 +13,7 @@ require 'mail'
> # user = ASF::Person.new($USER)
> # unless user.asf_member? or ASF.pmc_chairs.include? user
> # print "Status: 401 Unauthorized\r\n"
> -# print "WWW-Authenticate: Basic realm=\"ASF Members and Officers\"\r\n\r\n"
> +# print "WWW-Authenticate: Basic realm=\"ASF Members and PMC chairs\"\r\n\r\n"
> # exit
> # end
>
>
Re: [whimsy] branch master updated: Clarify karma check
Posted by Sam Ruby <ru...@intertwingly.net>.
On Wed, May 22, 2019 at 12:41 PM sebb <se...@gmail.com> wrote:
>
> On Wed, 22 May 2019 at 12:42, Sam Ruby <ru...@intertwingly.net> wrote:
> >
> > FYI: while the underlying LDAP group is named pmc-chairs, it actually
> > is a bit of a misnomer. It may include officers who are not pmc
> > chairs.
>
> Why is that?
Examples: Officers (i.e., people who author reports) and ASF members
can access the board agenda tool.
From time to time you may see VPs of non-PMC groups in pmc-chairs, and
infrastructure staff in members. So while both ldap-groups identify
their primary purpose, they both have been abused.
I'm aware of some talk within the infra team to rethink all of this.
Not sure what the status of that effort is, but as long as the whimsy
tools are updated along the way, there shouldn't be a problem.
> Is this documented anywhere?
Doubtful.
- Sam Ruby
Re: [whimsy] branch master updated: Clarify karma check
Posted by sebb <se...@gmail.com>.
On Wed, 22 May 2019 at 12:42, Sam Ruby <ru...@intertwingly.net> wrote:
>
> FYI: while the underlying LDAP group is named pmc-chairs, it actually
> is a bit of a misnomer. It may include officers who are not pmc
> chairs.
Why is that?
Is this documented anywhere?
> - Sam Ruby
>
> On Wed, May 22, 2019 at 4:37 AM <se...@apache.org> wrote:
> >
> > This is an automated email from the ASF dual-hosted git repository.
> >
> > sebb pushed a commit to branch master
> > in repository https://gitbox.apache.org/repos/asf/whimsy.git
> >
> >
> > The following commit(s) were added to refs/heads/master by this push:
> > new ba06c26 Clarify karma check
> > ba06c26 is described below
> >
> > commit ba06c26a342753852e6c48311c5738fc6e6fd33b
> > Author: Sebb <se...@apache.org>
> > AuthorDate: Wed May 22 09:37:10 2019 +0100
> >
> > Clarify karma check
> > ---
> > www/board/list-traffic.cgi | 2 +-
> > 1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/www/board/list-traffic.cgi b/www/board/list-traffic.cgi
> > index afd6727..a7a1279 100755
> > --- a/www/board/list-traffic.cgi
> > +++ b/www/board/list-traffic.cgi
> > @@ -13,7 +13,7 @@ require 'mail'
> > # user = ASF::Person.new($USER)
> > # unless user.asf_member? or ASF.pmc_chairs.include? user
> > # print "Status: 401 Unauthorized\r\n"
> > -# print "WWW-Authenticate: Basic realm=\"ASF Members and Officers\"\r\n\r\n"
> > +# print "WWW-Authenticate: Basic realm=\"ASF Members and PMC chairs\"\r\n\r\n"
> > # exit
> > # end
> >
> >