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
[TIMOB-11278] iOS: Add Ti.UI.WebView.ignoreSslError (Parity) #8786
Conversation
id reloadDataProperties; | ||
SEL reloadMethod; | ||
SEL reloadMethod; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Notes 1/2: I moved all properties to the updated indentation, so we have it easier in the future to add more properties without indentation issues.
[self loadURLRequest:newRequest]; | ||
[newRequest release]; | ||
|
||
return NO; | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Note 2/2: I moved the requestHeaders
logic into the newHeaders
scope, so it doesn't affect requests without custom request headers. This fixes a possible glitch in TIMOB-24329
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
insecureConnection = [[[NSURLConnection alloc] initWithRequest:request delegate:self] retain];
In above line retain will cause leak. Any reason for retaining after alloc ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Functionality is working fine.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
FT and CR passed.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
FR passed.
verified that iOS devices are now able to ignore invalid SSL certificate.
Ran the test-case from the ticket and a modified version.
ENV
SDK Version : Local 6.1.0 & 6.0.2
Mac OS Version : 10.12.2
Xcode Version : Xcode 8.2 Build version 8C38
Appc CLI : 6.1.0
Appc NPM : 4.2.9-1
Node : v4.6.0
Device: iPhone 6 (10.2.1) iPhone 5 (9.3.5)
JIRA: https://jira.appcelerator.org/browse/TIMOB-11278