★ wanayoo — archive 1999 https://github.com/AFNetworking/AFNetworking/pull/3679Nouvelle recherche | Portail wanayoo
Skip to content

/ AFNetworking

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

Jump to bottom

Removed ambiguous array creation in `AFSecurityPolicy` #3679

Merged
merged 1 commit into from Oct 11, 2016
Merged

Removed ambiguous array creation in `AFSecurityPolicy` #3679

merged 1 commit into from Oct 11, 2016

Conversation

@sgl0v
Copy link
Contributor

@sgl0v sgl0v commented Sep 9, 2016

Removed ambiguous array creation in AFSecurityPolicy. According to SecTrustCreateWithCertificates documentation:

If you want to specify only one certificate, you can pass a SecCertificateRef object; otherwise, pass an array of SecCertificateRef objects.
@kcharwood
Copy link
Contributor

@kcharwood kcharwood commented Oct 5, 2016

Can you rebase this with master to get Xcode 8 coverage?
@sgl0v
Removed ambiguous array creation. According to documentation SecTrust…
Loading status checks…
4a70d4a 
…CreateWithCertificates takes SecCertificateRef object or an array of certificates.
@sgl0v sgl0v force-pushed the sgl0v:tls_pinning branch from 2742dc6 to 4a70d4a Oct 10, 2016
@sgl0v
Copy link
Contributor Author

@sgl0v sgl0v commented Oct 10, 2016

@kcharwood Done.
@kcharwood kcharwood added this to the 3.2.0 milestone Oct 11, 2016
@kcharwood kcharwood merged commit 9fea224 into AFNetworking:master Oct 11, 2016
1 check passed
1 check passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@kcharwood
Copy link
Contributor

@kcharwood kcharwood commented Oct 11, 2016

Thanks!
@kcharwood kcharwood changed the title Removed ambiguous array creation in AFSecurityPolicy Removed ambiguous array creation in `AFSecurityPolicy` Oct 11, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
fixed security
Projects
None yet
Milestone
3.2.0
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
@sgl0v @kcharwood
You can’t perform that action at this time.