Skip to content
This repository was archived by the owner on Jul 26, 2018. It is now read-only.

fixes None has no attribute "strip" bug for CI Token #6

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

fixes None has no attribute "strip" bug for CI Token #6

Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
50dee45
fixes None has no attribute "strip" bug
Nov 10, 2015
9e01fcd
fixes None has no attribute "strip" bug
Nov 10, 2015
82f10c5
Resolves merge conflict with master
urbanblight Jan 12, 2016
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion authentication_proxy.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -176,7 +176,7 @@ def index(location=None):
# this controll flow is starting to get hairy and I don't see test coverage
# let's look into refactoring for clarity.
if authentic_cci_token(
flask.request.headers.get('X-CI-Token', '').strip()):
flask.request.headers.get('X-CI-Token', None)):
app.logger.debug("CI token authenticted")
flask.session['credentials'] = '{"authenticated_token": "CI"}'
elif 'credentials' not in flask.session:
Expand Down