On 03.12.2018 16:07, Michael Pilato wrote:
> On 12/3/18 3:15 AM, Julian Foad wrote:
>> It makes me uncomfortable to depart from standard parsing. What if
>> users are relying on Python ConfigParser or other compatible parsing
>> as part of their Subversion authz infrastructure?
> We needn't keep this hypothetical. ViewVC is using (a slightly
> modified[*]) Python ConfigParser in this way.
>
> -- Mike
>
> [*] By default, ConfigParser (well, really the RawConfigParser it
> subclasses) lowercases option names, which can cause username/group
> matching to fail. So ViewVC's code replaces the optionxform method of
> ConfigParser with a noop lambda function. (See
> https://docs.python.org/2/library/configparser.html#ConfigParser.RawConfigParser.optionxform
> for official docs.)
Interesting. Of course, our ConfigParser-like implementation already has
the option to treat section names as case-sensitive, and this option is
used ... that's righjt, by the authz file parser.
-- Brane
Received on 2018-12-03 16:31:12 CET