vbatts/quay
Archived
1
0
Fork 0
Code Issues Pull requests Releases Wiki Activity

Merge pull request #1868 from coreos-inc/registry-health-check

Browse source 
Add a configuration flag for external TLS termination
This commit is contained in:
josephschorr 2016-09-29 11:16:06 +02:00 • committed by GitHub
commit 9def2cf055
2 changed files with 10 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

5
config.py
View file

@ -77,6 +77,11 @@ class DefaultConfig(object):
DB_TRANSACTION_FACTORY = create_transaction
# If set to true, TLS is used, but is terminated by an external service (such as a load balancer).
# Note that PREFERRED_URL_SCHEME must be `https` when this flag is set or it can lead to undefined
# behavior.
EXTERNAL_TLS_TERMINATION = False
# If true, CDN URLs will be used for our external dependencies, rather than the local
# copies.
USE_CDN = True

6
health/services.py
View file

@ -16,7 +16,11 @@ def _check_registry_gunicorn(app):
if len(hostname_parts) == 2:
port = ':' + hostname_parts[1]
registry_url = '%s://localhost%s/v1/_internal_ping' % (app.config['PREFERRED_URL_SCHEME'], port)
scheme = app.config['PREFERRED_URL_SCHEME']
if app.config.get('EXTERNAL_TLS_TERMINATION', False):
scheme = 'http'
registry_url = '%s://localhost%s/v1/_internal_ping' % (scheme, port)
try:
return client.get(registry_url, verify=False, timeout=2).status_code == 200
except Exception: