Skip to content

Commit ba090b5

Browse files
committed
Implement 503 reply if the GitHub token says 401
And remove not needed variable
1 parent 1854930 commit ba090b5

1 file changed

Lines changed: 5 additions & 1 deletion

File tree

pmaweb/views.py

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -73,6 +73,8 @@
7373

7474
GITHUB_API = 'https://api.github.com/'
7575

76+
class InvalidGitHubToken(HttpResponse):
77+
status_code = 503
7678

7779
def redirect_home_page(request, page):
7880
"""Redirect handled for old website links"""
@@ -99,14 +101,16 @@ def proxy_request(url):
99101
if settings.GITHUB_TOKEN:
100102
request.add_header('Authorization', 'Bearer {0}'.format(settings.GITHUB_TOKEN))
101103
handle = urllib.request.urlopen(request)
102-
code = handle.getcode()
103104
content = handle.read()
104105
except IOError as err:
105106
content = str(err)
106107
if hasattr(err, 'fp'):
107108
content = err.fp.read()
108109
if hasattr(err, 'code') and err.code == 404:
109110
raise Http404(content)
111+
if hasattr(err, 'code') and err.code == 401:
112+
return InvalidGitHubToken(content)
113+
110114
return HttpResponseServerError(content)
111115
return HttpResponse(
112116
content,

0 commit comments

Comments
 (0)