trac-admin fails silently when using all CAPS for a group name

[trac.20.khaz@…]

Hello!

On Gentoo with Trac 0.8.4, I'm seeing that if I have a permission table where my user has a role that is all caps (SUPERUSER in my case), it wasn't being used.

That said, if it was 'superuser', it works fine.

I'm fine with that, I just wouldn't mind:

• People verifying that I'm not on crack and this *is* actually happening for others

( allowing the above is a defect: )

• Documentation on the permissions page to show what characters are valid/etc
• Perhaps some error checking or warnings to the user.

Thanks!

[Matthew Good]

Yes, you can't use all capitals as a group name, since all caps is reserved for permissions. The WebAdmin plugin does validate the permission values you use, but trac-admin still needs to do this.

A related issue is that WebAdmin also checks the permission values when you delete permissions, which means that if you disable a component that provided certain permissions you can't remove the permission settings since they're not "valid". I can attach a patch here for that since I don't think that we have the authz set up for WebAdmin yet.

[anonymous]

We had something similar with version 0.9.4dev; when the group name was "KPIT_developers" the permissions were ignored. Our first guess was that the problem was either the group length or use of underscore in the name. To make it work, we changed the group name to "kpit". May be worth including in the test cases for the fix?

[Matthew Good]

The test when adding a permission was previously fixed in r2358. The problem with removing permissions from disabled components is fixed in r4359 and r4360.

[Christian Boos]

#3688 and #4291 were related.

[sid]

There is also a note on the TracPermissions page about the requirement that group names must be lowercase. I've added bold so hopefully it is easier to see for people running into the same problem (which I also did at one point).