Re: [PATCH] Ability to specify groups in groups (second time)

Hi Martin,

> This is the second time I submit this patch. First time was a diff
> against 1.0.8 since collab.net was down. This diff is against HEAD.
>
> * subversion/mod_authz_svn/mod_authz_svn.c
> (group_contains_user): Made it call itself if it encounters a
> group. An extra argument 'level' added to guard against excessive
> recursion.
> All callers changed.
>
> This is about the access control file specified via the
> AuthzSVNAccessFile directive in httpd.conf. The [groups] part does not
> allow one to specify other groups so it gets rather cumbersome to deal
> with lots of users who are in different groups. For example:

I know the patch has already been filed and I'm rather late with my
reaction, but could you change the patch to detect circular reference
instead of posing the somewhat arbitrary limit of 10 on group nesting?

Thanks!

bye,

Erik.

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Sun Oct 10 14:32:49 2004