Will Discourse ask for a username if it's not provided to /session/sso_login?

johncs · April 2, 2019, 12:23am

My account system does not have a username field: users log in with an email only. What will happen when I redirect the user to /session/sso_login and the username field in the payload is blank? Will Discourse correctly ask the user for that missing information?

I don’t see mention of this case in the SSO post. And although I see that username is an optional field, it seems like the implication is that it’s only optional if the user has a Discourse account already (ie: they aren’t new).

Thanks for your time! I realize I could code it up and just try it out myself, but it’d take a awhile to do so and I’m hoping one of you might be able to give a quick answer.

Falco · April 2, 2019, 12:27am

We will auto-generate the username using provided fields, like email and name:

github.com/discourse/discourse

lib/user_name_suggester.rb

80eace426

module UserNameSuggester
  GENERIC_NAMES = ['i', 'me', 'info', 'support', 'admin', 'webmaster', 'hello', 'mail', 'office', 'contact', 'team']

  def self.suggest(name, allow_username = nil)
    return unless name.present?
    name = parse_name_from_email(name)
    find_available_username_based_on(name, allow_username)
  end

  def self.parse_name_from_email(name)
    if name =~ User::EMAIL
      # When 'walter@white.com' take 'walter'
      name = Regexp.last_match[1]
      # When 'me@eviltrout.com' take 'eviltrout'
      name = Regexp.last_match[2] if GENERIC_NAMES.include?(name)
    end
    name
  end

  def self.find_available_username_based_on(name, allow_username = nil)

This file has been truncated. show original

johncs · April 2, 2019, 12:30am

Wonderful, thank you. That’s a super useful answer, and a super useful feature : the difference between me being able to use SSO or not.

Topic		Replies	Views
Bad automatic choosing of username when using SSO (DiscourseConnect) SSO	1	950	August 18, 2022
No username check prompt when logging in with SSO SSO discourseconnect	2	1378	February 12, 2022
Discourse/Drupal SSO Username Questions Product Support	1	892	September 22, 2021
Populating Full Name with Email Address if Left Blank by SSO Feature pr-welcome	3	1200	October 4, 2016
Recreating the single sign on logic for generating usernames Development	4	1202	January 5, 2018

Will Discourse ask for a username if it's not provided to /session/sso_login?

AI-generated summary

Related topics