[openstreetmap/openstreetmap-website] Drop support for OAuth 1 (PR #5058)

[mmd]

@mmd-osm commented on this pull request.



>                       elsif current_user
-                       ClientApplication.all_permissions
+                       Oauth.scopes.map { |s| :"allow_#{s.name}" }

Since basic auth is disabled, I would assume that we don’t need this case anymore. Is this still to planned for anther PR, which also removes basic auth usage in unit tests?

-- 
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/5058#pullrequestreview-2271953895
You are receiving this because you are subscribed to this thread.

Message ID: <openstreetmap/openstreetmap-website/pull/5058/review/2271953895 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20240830/4ed91a66/attachment.htm>