Modify ↓
Opened 12 years ago
Closed 12 years ago
#11559 closed enhancement (fixed)
CommitTicketUpdater ignores [trac] ignore_auth_case
| Reported by: | Ryan J Ollos | Owned by: | Ryan J Ollos |
|---|---|---|---|
| Priority: | normal | Milestone: | 1.0.2 |
| Component: | version control | Version: | 1.0-stable |
| Severity: | normal | Keywords: | CommitTicketUpdater |
| Cc: | Branch: | ||
| Release Notes: |
|
||
| API Changes: | |||
| Internal Changes: | |||
Description (last modified by )
It was discussed on the mailing list whether permission checks performed in CommitTicketUpdater should take into account the [trac] ignore_auth_case parameter. The authorization currently fails checking for TICKET_APPEND when the username in Subversion differs from that in Trac by casing.
Attachments (0)
Change History (4)
comment:1 by , 12 years ago
| Description: | modified (diff) |
|---|
comment:2 by , 12 years ago
Proposed changes can be found in log:rjollos.git:t11559. I didn't want to mutate the changeset object by changing the author property, but maybe there is a cleaner way than applying _authname each time we may need a lower-cased author.
comment:3 by , 12 years ago
| Milestone: | next-stable-1.0.x → 1.0.2 |
|---|---|
| Owner: | set to |
| Status: | new → assigned |
Note:
See TracTickets
for help on using tickets.
I was too hasty in putting together the previous patch. I'll post a tested patch shortly.