Merge pull request #42 from siemens/refactor-adapt-to-23-9-1

refactor: adapt to Sentry 23.9.1

Author: dlouzan

.github/workflows/publish.yml

@@ -12,11 +12,11 @@ jobs:
   publish:
     runs-on: ubuntu-latest
     steps:
-      - uses: actions/checkout@v2
-      - uses: actions/setup-python@v2
+      - uses: actions/checkout@v4
+      - uses: actions/setup-python@v4
         with:
           python-version: "3.8"
-      - uses: abatilo/actions-poetry@v2.0.0
+      - uses: abatilo/actions-poetry@v2
       - name: Publish package
         run: |
           poetry install -n --no-dev

.github/workflows/test.yml

@@ -15,8 +15,8 @@ jobs:
   black:
     runs-on: ubuntu-latest
     steps:
-      - uses: actions/checkout@v2
-      - uses: actions/setup-python@v2
+      - uses: actions/checkout@v4
+      - uses: actions/setup-python@v4
         with:
           python-version: "3.8"
       - uses: psf/black@stable
@@ -25,15 +25,15 @@ jobs:
   commitlint:
     runs-on: ubuntu-latest
     steps:
-      - uses: actions/checkout@v2
+      - uses: actions/checkout@v4
         with:
           fetch-depth: 0
       - uses: wagoid/commitlint-github-action@v3
   test:
     runs-on: ubuntu-latest
     services:
       postgres:
-        image: postgres:9.6
+        image: postgres:14.5
         env:
           POSTGRES_HOST_AUTH_METHOD: trust
         # Set health checks to wait until postgres has started
@@ -45,26 +45,28 @@ jobs:
         ports:
           - 5432:5432
       memcached:
-        image: memcached:1.5-alpine
+        image: memcached:1.6.21-alpine
         ports:
           - 11211:11211
       redis:
-        image: redis:5.0-alpine
+        image: redis:6.2.12-alpine
         ports:
           - 6379:6379
     steps:
-      - uses: actions/checkout@v2
-      - uses: actions/setup-python@v2
+      - uses: actions/checkout@v4
+      - uses: actions/setup-python@v4
         with:
           python-version: "3.8"
-      - uses: abatilo/actions-poetry@v2.0.0
+      - uses: abatilo/actions-poetry@v2
       - uses: actions/setup-node@v3
         with:
           node-version: '16'
       - name: Setup required upstream yarn version
         run: yarn set version 1.22.5
       - run: |
           sudo apt-get update && sudo apt-get install -y libxmlsec1-dev libmaxminddb-dev
-          pip3 install -U wheel
+          python3 -m venv venv
+          . venv/bin/activate
+          venv/bin/pip3 install -U wheel
           make develop
           make test

.gitignore

@@ -6,3 +6,4 @@
 venv
 .venv
 .coverage
+.vscode/

oidc/provider.py

@@ -38,8 +38,6 @@ def get_authorize_params(self, state, redirect_uri):
 
 class OIDCProvider(OAuth2Provider):
     name = ISSUER
-    client_id = CLIENT_ID
-    client_secret = CLIENT_SECRET
 
     def __init__(self, domain=None, domains=None, version=None, **config):
         if domain:
@@ -59,16 +57,22 @@ def __init__(self, domain=None, domains=None, version=None, **config):
         self.version = version
         super().__init__(**config)
 
+    def get_client_id(self):
+        return CLIENT_ID
+
+    def get_client_secret(self):
+        return CLIENT_SECRET
+
     def get_configure_view(self):
         return OIDCConfigureView.as_view()
 
     def get_auth_pipeline(self):
         return [
-            OIDCLogin(self.client_id, domains=self.domains),
+            OIDCLogin(self.get_client_id(), domains=self.domains),
             OAuth2Callback(
                 access_token_url=TOKEN_ENDPOINT,
-                client_id=self.client_id,
-                client_secret=self.client_secret,
+                client_id=self.get_client_id(),
+                client_secret=self.get_client_secret(),
             ),
             FetchUser(
                 domains=self.domains,