Skip to content

Support SSL verification via internal CA bundle#294

Merged
gpocentek merged 1 commit intopython-gitlab:masterfrom
wayfair-archive:feature_internal_cert_configuration
Aug 5, 2017
Merged

Support SSL verification via internal CA bundle#294
gpocentek merged 1 commit intopython-gitlab:masterfrom
wayfair-archive:feature_internal_cert_configuration

Conversation

@mdhausman
Copy link
Copy Markdown

@mdhausman
Copy link
Copy Markdown
Author

mdhausman commented Aug 1, 2017

Hm. I don't think my code changes touch anything that's broken in these checks, but these checks are definitely broken somehow.

@mdhausman mdhausman force-pushed the feature_internal_cert_configuration branch 2 times, most recently from a48dcbe to e7fe2a8 Compare August 1, 2017 00:38
@gpocentek
Copy link
Copy Markdown
Contributor

The python3.6 failure is definitely not related to your patch. The functional tests failure is probably a temporary error. Let's see how it goes with a new run.

@gpocentek
Copy link
Copy Markdown
Contributor

The patch looks good BTW, thanks!

@mdhausman
Copy link
Copy Markdown
Author

Glad it looks good. Wish the tests were working, though 😅

@mdhausman mdhausman force-pushed the feature_internal_cert_configuration branch 2 times, most recently from 0789098 to 8f6ef0e Compare August 3, 2017 18:19
@gpocentek
Copy link
Copy Markdown
Contributor

Could you rebase your change on the current master? The tests should be OK now.

@mdhausman mdhausman force-pushed the feature_internal_cert_configuration branch from 8f6ef0e to 4af4748 Compare August 4, 2017 17:52
@mdhausman
Copy link
Copy Markdown
Author

Success! 💃 🎉 🎊 🍾

@gpocentek gpocentek merged commit 0e70dd9 into python-gitlab:master Aug 5, 2017
@gpocentek
Copy link
Copy Markdown
Contributor

Thank you!

@gpocentek gpocentek mentioned this pull request Aug 5, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants