Protospace / spaceport

Calgary Protospace's member portal
https://my.protospace.ca
GNU Affero General Public License v3.0
13 stars 9 forks source link

username already taken check? #99

Open frollard opened 1 year ago

frollard commented 1 year ago

https://github.com/Protospace/spaceport/blob/24a0f37faa05884ec87495b50ba394f5f4c3a105/authserver/auth_functions.py#L309

I presume discourse returns a 400ish error but we're not checking for it before trying to change names; don't want to test on live.

tannercollin commented 1 year ago

Yeah Discourse will return an error in this case. But this won't be checked if there's a name collision on sign up (ie. another tanner.collin signs up to Spaceport), which can't happen because Spaceport will prevent signup unless someone set a custom first.last Discourse username that matches. We'll cross that bridge when it comes. first.last was kind of a mistake.