Ticket #10397 (closed defect: fixed)

Opened 8 months ago

Last modified 6 months ago

Don't allow username with all capital letters

Reported by: adeel.asghar@liu.se Assigned to: hasienda
Priority: normal Component: AccountManagerPlugin
Severity: normal Keywords: user registration check
Cc: rjollos Trac Release: 0.12

Description

I have installed TracAccountManager? 0.4dev-r12000. This allows users to register with a username that can have all capital letters. Is there a way to stop this? Later I want to set permissions on the users but i can't set them if the username is all capital letters.

Attachments

Change History

10/01/12 20:48:08 changed by hasienda

  • status changed from new to closed.
  • severity changed from blocker to normal.
  • cc set to rjollos.
  • priority changed from highest to normal.
  • keywords set to configuration support.
  • resolution set to worksforme.

You're welcome to test trunk as it's currently release candidate for acct_mgr-0-4. Thank you for taking care.

Still, there is a another, preferred way to ask for support than suggesting a bug in the corresponding plugin, and you've already been told to not use a ticket for these kind of questions at our new ticket page. This is about playing nice in a FOSS environment, where this ticket system here is dedicated to plugin development issues and real bugs.

Better ask this kind of question to our t:wiki:MailingList, please.

I went there to replicate your question and give an appropriate answer as well:

https://groups.google.com/d/topic/trac-users/0MQW7fO77RY/discussion

Last note: prio=highest and severity=blocker? Don't treat me like that for contributing my knowledge and time here - for free. Understand, that attracting attention this way is considered bad habit or worse. You do use development code and seem to not have spend much time with neither commit log history nor wiki documentation for this plugin, if any. Don't dare to do that again, please!

10/01/12 22:02:17 changed by hasienda

  • status changed from closed to reopened.
  • resolution deleted.

Well, nobody is perfect. A fellow developer finally convinced me, that you've hit a weak spot indeed. So let's deal with it like programmers.

10/01/12 22:02:33 changed by hasienda

  • keywords changed from configuration support to user registration check.

(follow-up: ↓ 6 ) 10/01/12 22:09:56 changed by rjollos

Of course, your original point remains very valid and should not be lost on the reporter - that the issues are best discussed over the mailing list prior to opening a defect. Now, with the mailing list there is greater exposure to the change and the greater possibility of follow-up comments and opinions. Thank you for your efforts to fix the issue!

10/01/12 22:24:08 changed by hasienda

(In [12083]) AccountManagerPlugin: Prevent all upper-cased usernames, refs #10397.

Thanks to Adeel Asgha for the initial suggestion despite of my first reaction.

Thinking a bit, and after a friendly nudge by Ryan this seems like a pretty basic requirement indeed, that should even have come-up much earlier. Easy enough to enforce it in BasicCheck for new user registrations now.

(in reply to: ↑ 4 ; follow-up: ↓ 7 ) 10/01/12 22:36:26 changed by hasienda

Replying to rjollos:

Of course, your original point remains very valid and should not be lost on the reporter - that the issues are best discussed over the mailing list prior to opening a defect. Now, with the mailing list there is greater exposure to the change and the greater possibility of follow-up comments and opinions.

Thought to react more often like so in the future, for saving us a bit of tedious Q&A sessions in our ticket system. Even more as ticket comments perform sub-optimal for extended arguments, or worse.

Thank you for your efforts to fix the issue!

You're welcome. Not always is it possible to just go for it. Luckily I could today, and glad to have it in for the next release. All the recent work on the registration makes me feel confident, that acct_mgr-0.4 will be indeed a major milestone and big improvement compared to current stable. Once again thanks for your feedback.

(in reply to: ↑ 6 ; follow-up: ↓ 8 ) 10/02/12 00:36:36 changed by anonymous

Replying to hasienda:

Replying to rjollos:

Of course, your original point remains very valid and should not be lost on the reporter - that the issues are best discussed over the mailing list prior to opening a defect. Now, with the mailing list there is greater exposure to the change and the greater possibility of follow-up comments and opinions.

Thought to react more often like so in the future, for saving us a bit of tedious Q&A sessions in our ticket system. Even more as ticket comments perform sub-optimal for extended arguments, or worse.

Thank you for your efforts to fix the issue!

You're welcome. Not always is it possible to just go for it. Luckily I could today, and glad to have it in for the next release. All the recent work on the registration makes me feel confident, that acct_mgr-0.4 will be indeed a major milestone and big improvement compared to current stable. Once again thanks for your feedback.

Thanks hasienda for fixing it. I tried to find a mailing list but couldn't find it here. Perhaps I haven't searched properly. Thanks again!!!

(in reply to: ↑ 7 ) 10/02/12 17:47:43 changed by hasienda

Replying to anonymous:

Replying to hasienda:

...

Thanks hasienda for fixing it. I tried to find a mailing list

Really? Well, it would be a big overhead to put it into every single plugin page.

but couldn't find it here. Perhaps I haven't searched properly. Thanks again!!!

You'll find a hint on the bottom of our front page in section Feedback and Suggestions, of course. It's directing to t:wiki:MailingList. Consider a subscription to trac-users, that will enable you to ask your support questions as advised above, and track plugin issues, that others point out as well. Thanks for consideration.

Technical note: This ticket is left open intentionally and will be closed, when tractags-0.7 get released.

12/01/12 16:55:52 changed by hasienda

  • status changed from reopened to closed.
  • resolution set to fixed.

(In [12398]) AccountManagerPlugin: Releasing version 0.4, pushing development to acct_mgr-0.5dev.

Availability of that code as stable release closes #874, #3459, #4677, #5295, #5691, #6616, #7577, #8076, #8685, #8770, #8791, #8990, #9052, #9079, #9090, #9139, #9246, #9252, #9547, #9618, #9676, #9843, #9852, #9940, #10023, #10028, #10123, #10142, #10204, #10276, #10397, #10412, #10594, #10625 and #10644.

Some more issues have been worked-on, yet without confirmed resolution, refs #5464 (for JiraToTracIntegration), #8927 and #10134.

And finally there are some issues and enhancement requests showing progress, but known to require more work to resolve them satisfactorily, refs #843, #1600, #5964, #8217, #8933.

Thanks to all contributors and followers, that enabled and encouraged a good portion of this development work.


Add/Change #10397 (Don't allow username with all capital letters)




Change Properties
Action