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

avoid overriding of the property kCFStreamPropertySSLSettings #343

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

tkanzakic
Copy link

the call to the function

CFReadStreamSetProperty((CFReadStreamRef)[self readStream], kCFStreamPropertySSLSettings, sslProperties);

to set the client certificate was overriding the previous values of the property kCFStreamPropertySSLSettings set to ignore certificates validation

the call to the function

CFReadStreamSetProperty((CFReadStreamRef)[self readStream], kCFStreamPropertySSLSettings, sslProperties);

to set the client certificate was overriding the previous values of the property kCFStreamPropertySSLSettings set to ignore certificates validation
@anivaros
Copy link

Please, merge this critical pull request!

@jogu
Copy link
Collaborator

jogu commented Mar 23, 2015

@anivaros There's generally not anybody left maintaining ASIHTTPRequest that actually uses it (and you really shouldn't be using it anymore unless you have a legacy huge project that just can't update to something more modern for some reason.)

I find the patch pretty difficult to read because of the white space changes (and because I'm tired).

Comments along the lines of "I have tested this code and it works. I have reviewed the code and it does not seem to have any potential to cause regressions." would be helpful.

Does the original problem only occur when certificate validation is disabled?

@anivaros
Copy link

anivaros commented Mar 24, 2015

unless you have a legacy huge project that just can't update to something more modern for some reason

It's my situation :)

I've tested this patch on own fork, and it works fine. Code seems to be ok too.
Problem occurs only on iOS simulator (I don't know why) when certificate validation is disabled, but I need to use simulator for autotesting my project.

A weak ago i've switched to CocoaPods and moved there ASIHttp too. You'll help me a lot by making new version of .podspec (1.8.3) with that fix.

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.

3 participants