-
Notifications
You must be signed in to change notification settings - Fork 12
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
RDISCROWD-6956 add ssl cert path to proxy service request #923
Conversation
Pull Request Test Coverage Report for Build 9287214280Details
💛 - Coveralls |
pybossa/api/__init__.py
Outdated
ssl_cert = current_app.config.get('SSL_CERT_PATH') | ||
ret = requests.post(url, headers=headers, json=payload['data'], verify=ssl_cert if ssl_cert else True) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This could be
ssl_cert = current_app.config.get('SSL_CERT_PATH', False)
ret = requests.post(url, headers=headers, json=payload['data'], verify=ssl_cert)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@dchhabda verify
is True
by default so this would change existing behavior for missing certs. I will change the first line to:
ssl_cert = current_app.config.get('SSL_CERT_PATH', True)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
for missing certs path in configs, IMO ssl_cert
would be set to False
which would superimpose default verify=True
behavior. Let's connect offline if needed. Thanks!
* RDISCROWD-6956 add ssl cert path to proxy service request
Issue number of the reported bug or feature request: RDISCROWD-6956
Describe your changes
add ssl cert path to proxy service request