It would greatly enhance security if there was an option to force username even if the registration form is not disabled ie. it is shown to the end user after TUPAS authentication.

So, I propose this:

  • In the TUPAS registration settings add an option to force username auto-generation
  • When this is enabled, do not display username field to end user during registration
CommentFileSizeAuthor
#5 2863747.patch10.18 KBtuutti

Comments

softone created an issue. See original summary.

tuutti’s picture

I'm not opposed to this idea, but I'm not sure how this would enhance security. Could you clarify a bit?

softone’s picture

OK.

So, for a normal site this current behaviour is just right since the user can use TUPAS authentication to nicely login and choose whatever username they choose.

But for a site that wants to depend on users giving strongly verified identity on registration, the behaviour explained above would indeed enhance security.

(EDIT: On a side note, it would also be nice to be able to support assigning directly to different user roles for users using TUPAS authentication thus diffirentiating them from users not using TUPAS for registration)

softone’s picture

I've edited out the brain fart from the previous comment. Sorry for that. (And it seems the edits don't trigger an email notification.)

tuutti’s picture

StatusFileSize
new10.18 KB

I guess this makes some sense, especially if the service provider does not want to provide any way to identify the user.

  • tuutti authored c89885a on 8.x-1.x
    Issue #2863747 by tuutti: Registration: Option to force username even if...
tuutti’s picture

Status: Active » Fixed

This is fixed on the latest dev release. I'll push new alpha release once I have fixed the other issue you created about rules support.

softone’s picture

Thank you for the fix.

However, this is now generating a random username. It would make more sense to generate an username from the information provided by the TUPAS authentication, just like when the registration form is disabled. Now just letting the user edit other user info, but not touch the username field.

tuutti’s picture

Status: Fixed » Needs work

That makes a lot more sense. I misunderstood what you originally meant.

softone’s picture

Sorry, my bad. I should have mentioned the username provided by TUPAS authentication in the first place :-)

  • tuutti committed c1cb2f2 on 2863747
    Issue #2863747 by tuutti: Registration: Option to force username even if...

  • tuutti committed c1cb2f2 on 8.x-1.x
    Issue #2863747 by tuutti: Registration: Option to force username even if...
  • tuutti authored ef05ead on 8.x-1.x
    Merge pull request #7 from tuutti/2863747
    
    Issue #2863747 by tuutti:...
tuutti’s picture

Status: Needs work » Needs review

This should be fixed on the latest dev release.

softone’s picture

I can confirm that username forcing works correctly with the current dev.

tuutti’s picture

Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.