[openstreetmap/openstreetmap-website] Drop support for basic authentication (PR #5141)

Anton Khorev notifications at github.com
Mon Sep 2 16:47:40 UTC 2024


@AntonKhorev commented on this pull request.



>      # abilities take into account terms agreement too.
-    def test_api_write_and_terms_agreed_via_basic_auth
+    def test_api_write_and_terms_agreed_via_oauth

Do we need this test? What's the difference between it and the previous `_via_token` test? Only token scopes?

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

Message ID: <openstreetmap/openstreetmap-website/pull/5141/review/2275984077 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20240902/a10a113e/attachment.htm>


More information about the rails-dev mailing list