darkhelm/notifications-admin
1
0
Fork 0
mirror of https://github.com/GSA/notifications-admin.git synced 2026-05-05 16:38:59 -04:00
Code Issues Packages Projects Releases Wiki Activity

This pull request removes the need for the req and pay in the claims JWT.

Browse Source 
Use the new version of the notifications-python-client. This version no longer adds the req and pay to the claims of the jwt.

The change is backward compatible so an older client that sends a JWT with the extra claims will pass authentication.
Once all the clients have been updated to not include the extra claims some updates to exclude them from the method signatures will happen as well.

The documentation has been updated to reflect this change.

https://www.pivotaltracker.com/story/show/116971293
This commit is contained in:
Rebecca Law
2016-04-15 11:04:35 +01:00
parent 99d5a3afc7
commit 1fe5ae0371
9 changed files with 7 additions and 43 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
app/__init__.py
View File

@@ -2,7 +2,6 @@ import os
import re
import dateutil
import datetime
import urllib
from flask import (
Flask,
@@ -16,10 +15,10 @@ from flask import (
from flask._compat import string_types
from flask_login import LoginManager
from flask_wtf import CsrfProtect
from notifications_python_client import HTTPError
from notifications_python_client.errors import HTTPError
from pygments import highlight
from pygments.lexers import JavascriptLexer
from pygments.formatters import HtmlFormatter
from pygments.formatters.html import HtmlFormatter
from pygments.lexers.javascript import JavascriptLexer
from werkzeug.exceptions import abort
from app.notify_client.api_client import ServiceAPIClient