-
Notifications
You must be signed in to change notification settings - Fork 923
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
OAuth2 login flow returns error 500 #5400
Comments
We are seeing the same issue. OSM is in Readonly-Mode https://en.osm.town/@osm_tech/113656639378201391 but since I can log into the website, the readonly has to be only for certain parts of the website. I assumed the Rails parts would work but the editing part wouldn't. But then again commenting on notes is also prohibited so maybe only the user sessions are "allowed" since those can be deleted when switching back to Amsterdam as primary (or maybe they are not stored in the DB at all)… |
Creating new OAuth logins requires creating records in the oauth tables in the database. Existing tokens should work fine I think but you won't be able to create new ones. |
@tomhughes Thanks. Is there capacity to improve the handing of this error case by showing the "we are in readonly" message in step 1 of the oAuth process so users don't run into the uncaught 500? |
I think it has to do with the following: Does anyone know how long this maintenance work will last? |
|
Thanks for the link @HolgerJeromin! I thought that the fact that I can see the openstreetmap.org site means the failure was resolved, but I now learned that it's in read-only mode. |
I should clarify here that this issue is "fixed" in that #5406 means the site now won't return a 500 error for the OAuth login flow. The read-only status on www.osm.org is still ongoing, and is a matter for the OSMF OWG. |
URL
Https://israelhiking.osm.org.il
How to reproduce the issue?
This is a standard login flow from a third party app:
Screenshot(s) or anything else?
I think this is relatively new, maybe after the outage yesterday, but it might existed even before.
The text was updated successfully, but these errors were encountered: