Skip to content
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

feat(CP4D Authentication): add ssl verification for self-signed certificates #147

Merged
merged 2 commits into from
Aug 1, 2022

Conversation

LBExposito
Copy link
Contributor

These modifications allow the user to have a self-signed certificate in the SSL verification process when authenticating to an STT instance on CP4D.

The verify attribute is used by the requests python module to indicate the path to a custom CA Bundle (resquests documentation).

With these changes, it wont be necessary to disable ssl verification anymore when using a self-signed certificate with STT on CP4D.

@CLAassistant
Copy link

CLAassistant commented Jul 27, 2022

CLA assistant check
All committers have signed the CLA.

@bobfang
Copy link
Member

bobfang commented Jul 29, 2022

to elaborate a bit more on this, these changes are for the Authentication step (getting a token). During regular SDK calls we can still pass verify as a kwarg and get it passed through to Requests.

Copy link
Member

@padamstx padamstx left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Change look good, but we just need to add "verify" to the docstrings of each function that it was added to...

@LBExposito
Copy link
Contributor Author

Docstrings are updated !

Copy link
Member

@padamstx padamstx left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@padamstx padamstx merged commit a16685d into IBM:main Aug 1, 2022
ibm-devx-sdk pushed a commit that referenced this pull request Aug 1, 2022
# [3.16.0](v3.15.3...v3.16.0) (2022-08-01)

### Features

* **CP4D Authentication:** add ssl verification for self-signed certificates ([#147](#147)) ([a16685d](a16685d))
@ibm-devx-sdk
Copy link

🎉 This PR is included in version 3.16.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants