Skip to content
This repository has been archived by the owner on Jan 13, 2021. It is now read-only.

Fix for ssl issue with where bucket name has dots in it #9

Merged
merged 1 commit into from
Dec 13, 2016

Conversation

abhishekmahawar
Copy link

No description provided.

Copy link
Author

@abhishekmahawar abhishekmahawar left a comment

Choose a reason for hiding this comment

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

@idralyuk can you please review and merge this ?

Copy link

@idralyuk idralyuk left a comment

Choose a reason for hiding this comment

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

Looks good, but could you add a test case for this?

@abhishekmahawar
Copy link
Author

This is just a hack for a ssl library issue with dots, and not actually a dataduct feature change. Had a discussion with @vivek about this, we don't feel there's a test case needed for this in dataduct. Ideally, this fix should be in pyhon ssl and related test case should go in there. Please feel free to comment otherwise.

@idralyuk idralyuk merged commit ccd20bf into WiserSolutions:develop Dec 13, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants