Use updated scopes for new oauth provider

metabrainz · Dec 4, 2024 · c503d95 · c503d95
1 parent cb81b61
commit c503d95
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 3 deletions.
diff --git a/critiquebrainz/frontend/views/oauth.py b/critiquebrainz/frontend/views/oauth.py
@@ -35,4 +35,4 @@ def authorize_prompt():
             code = oauth.generate_grant(client_id, current_user.id, redirect_uri, scope)
             return redirect(build_url(redirect_uri, dict(code=code, state=state)))
         except OAuthError as e:
-            raise BadRequest(e.desc)
+            raise BadRequest(e.desc)
diff --git a/critiquebrainz/ws/oauth/provider.py b/critiquebrainz/ws/oauth/provider.py
@@ -212,7 +212,8 @@ def get_authorized_user(self, scopes):
                 raise exceptions.InvalidToken
 
             token_scopes = token["scope"]
-            for scope in scopes:
+            new_scopes = [f"critiquebrainz:{s}" for s in scopes]
+            for scope in new_scopes:
                 if scope not in token_scopes:
                     raise exceptions.InvalidToken
 

diff --git a/docker/docker-compose.dev.yml b/docker/docker-compose.dev.yml
@@ -57,4 +57,4 @@ services:
       dockerfile: Dockerfile.webpack
     command: npm run dev
     volumes:
-      - ..:/code:z
+      - ../critiquebrainz:/code/critiquebrainz:z