-
Notifications
You must be signed in to change notification settings - Fork 15
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
Switch to OAuth 2 #177
Comments
Would probably fix #138 |
Now i can't upload any changes, basic auth is disabled |
Confirmed. The app no longer allows me to upload changes or log in as of today so is no longer usable as an editor. Please thumbs up this comment if you are also affected by this so we avoid too much comment spam. |
Oops, I'm sorry. I thought the basic authentication would still work. I am currently implementing OAuth2 to fix the issue. I am testing it and hope to deploy it by tomorrow. |
This seems to work now - thanks! - but I notice OSM Go requests the following permission: Create diary entries, comments and make friends. Does it need this? I'm general it would be good if the app explained what permissions it needs and why before asking the user to grant them. I appreciate this is likely a topic for a separate issue. |
I meant to add that in updating the app I unfortunately appear to have lost my pending uploads. |
Is the PWA version still 1.6? |
Don't be sorry, thank you so much for this app wich i do appreciate (https://forum.openstreetmap.fr/t/remplir-osm-avec-organic-maps/25018/19?u=britz)
i downloaded the app just now through Play Store, it's the 1.7.0, i tried it and i confirm : it's working with OAuth2, i can send changeset to OSM
yes, i checked, the version behind https://osmgo.com/#/about and it's still the 1.6.0, i think the owner of the website https://osmgo.com will update it whenever it's possible for him Once again, thank you DoFabien for this release https://github.com/OsmTravel/OsmGo/releases/tag/1.7.0 |
Switch to OAuth 2 from HTTP Basic Auth and OAuth 1.0a. They are being deprecated.
See openstreetmap/operations#867
The text was updated successfully, but these errors were encountered: