Fixed a couple of errors.

Author: rohe

flask_rp/application.py

@@ -37,7 +37,9 @@ def oidc_provider_init_app(config_file, name=None, **kwargs):
     app = Flask(name, static_url_path='', **kwargs)
 
     app.rp_config = Configuration.create_from_config_file(config_file)
-    # app.config['SECRET_KEY'] = os.urandom(12).hex()
+
+    # Session key for the application session
+    app.config['SECRET_KEY'] = os.urandom(12).hex()
 
     app.users = {'test_user': {'name': 'Testing Name'}}
 

flask_rp/conf.yaml

@@ -1,3 +1,21 @@
+logging:
+  version: 1
+  root:
+    handlers:
+      - console
+    level: DEBUG
+  loggers:
+    idp:
+      level: DEBUG
+  handlers:
+    console:
+      class: logging.StreamHandler
+      stream: 'ext://sys.stdout'
+      formatter: default
+  formatters:
+    default:
+      format: '%(asctime)s %(name)s %(levelname)s %(message)s'
+
 port: &port 8090
 domain: &domain 127.0.0.1
 base_url: "https://{domain}:{port}"
@@ -71,7 +89,7 @@ clients:
     client_preferences: *id001
     issuer: https://127.0.0.1:5000/
     jwks_uri: 'static/jwks.json'
-    redirect_uris: ['authz_cb/flop']
+    redirect_uris: ['https://{domain}:{port}/authz_cb/flop']
     services: *id002
     add_ons:
       pkce:
@@ -91,3 +109,4 @@ webserver:
   # verify_user: optional
   # The you also need
   # ca_bundle: ''
+  debug: true

flask_rp/views.py

@@ -36,7 +36,7 @@ def send_js(path):
 
 @oidc_rp_views.route('/')
 def index():
-    _providers = current_app.config.get('CLIENTS').keys()
+    _providers = current_app.rp_config.clients.keys()
     return render_template('opbyuid.html', providers=_providers)
 
 
@@ -68,7 +68,7 @@ def rp():
         else:
             return redirect(result['url'], 303)
     else:
-        _providers = current_app.config.get('CLIENTS').keys()
+        _providers = current_app.rp_config.clients.keys()
         return render_template('opbyuid.html', providers=_providers)
 
 

src/oidcrp/oauth2/__init__.py

@@ -1,4 +1,5 @@
 import logging
+from json import JSONDecodeError
 
 from cryptojwt.key_jar import KeyJar
 from oidcmsg.exception import FormatError
@@ -245,6 +246,8 @@ def parse_request_response(self, service, reqresp, response_body_type='',
                 else:
                     raise OidcServiceError("HTTP ERROR: %s [%s] on %s" % (
                         reqresp.text, reqresp.status_code, reqresp.url))
+            except JSONDecodeError: # So it's not JSON assume text then
+                err_resp = {'error': reqresp.text}
 
             err_resp['status_code'] = reqresp.status_code
             return err_resp