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

Fix for RT 107924 #27

Merged
merged 3 commits into from Jun 21, 2016

Conversation

Projects
None yet
2 participants
@openstrike

openstrike commented Jun 20, 2016

Here is an implementation (with tests) of a fix for RT 107924. It supposes that Steffen's second suggestion ("Handle ssl_opts in a special way in SOAP::Lite") is the way to go, which may not necessarily be how you want to play it but seemed like a good first cut to me.

@redhotpenguin

This comment has been minimized.

Show comment
Hide comment
@redhotpenguin

redhotpenguin Jun 21, 2016

Owner

Looks good - do you want me to do a release?

Owner

redhotpenguin commented Jun 21, 2016

Looks good - do you want me to do a release?

@redhotpenguin redhotpenguin merged commit 58c13a3 into redhotpenguin:master Jun 21, 2016

@openstrike

This comment has been minimized.

Show comment
Hide comment
@openstrike

openstrike Jun 21, 2016

Thanks. Entirely up to you when to roll a release. It's probably worth updating the RT ticket, though.

openstrike commented Jun 21, 2016

Thanks. Entirely up to you when to roll a release. It's probably worth updating the RT ticket, though.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment