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

TIMOB-11227 fix to support new ignoreSslError property on web view #3116

Merged
merged 1 commit into from Oct 5, 2012
Merged

TIMOB-11227 fix to support new ignoreSslError property on web view #3116

merged 1 commit into from Oct 5, 2012

Conversation

rusticphilosopher
Copy link
Contributor

@billdawson
Copy link
Contributor

Bill will review.

@ghost ghost assigned billdawson Oct 5, 2012
@billdawson
Copy link
Contributor

CR/FR accepted.

With the property not used at all, the page did not appear. (expected, since default value of new property is false).

With property set to false, page did not appear (obviously).

With property set to true, page appeared despite the bad ssl certificate.

billdawson added a commit that referenced this pull request Oct 5, 2012
TIMOB-11227 fix to support new ignoreSslError property on web view
@billdawson billdawson merged commit 96e5544 into tidev:master Oct 5, 2012
@arthurevans
Copy link
Contributor

Is there doc for this new property?

@rusticphilosopher
Copy link
Contributor Author

@arthurevans no, the behavior on the iOS and Mobile Web side is still in flux so the doc change should be included in a follow up PR once that is settled but this is a fix needed for a customer issue so going ahead and getting it into platform with the expectation that doc will come along once it is decided if this should exist on other platforms or not.

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.

None yet

3 participants