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.

Sign up for GitHub

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

[AndroidCrypto] Handle setting non-default application protocols #51187

Merged
merged 6 commits into from
Apr 16, 2021
Merged

[AndroidCrypto] Handle setting non-default application protocols #51187

merged 6 commits into from
Apr 16, 2021

Conversation

elinor-fung
Copy link
Member

@elinor-fung elinor-fung commented Apr 13, 2021 •
edited
Loading

  • Set ApplicationProtocols when supported by the platform
    • Throws PNSE if platform doesn't support configuration (< API level 29)
  • Request client certificate when running in server mode and RemoteCertRequired is true
  • Fix writing large content to stream
  • Update test utilities to check for alpn support on Android

cc @jkoritzinsky @steveisok @AaronRobinsonMSFT @bartonjs @wfurt

@ghost
Copy link

ghost commented Apr 13, 2021

Tagging subscribers to this area: @dotnet/ncl, @vcsjones
See info in area-owners.md if you want to be subscribed.

Issue Details
  • Set ApplicationProtocols when supported by the platform
    • Throws PNSE if platform doesn't support configuration (< API level 29)
  • Request client certificate when running in server mode and RemoteCertRequired is true
  • Fix writing large content to stream
Author: elinor-fung
Assignees: -
Labels:

area-System.Net.Security
Milestone: -

@elinor-fung elinor-fung mentioned this pull request Apr 13, 2021
Open
36 tasks
@elinor-fung elinor-fung merged commit 237d863 into dotnet:main Apr 16, 2021
@elinor-fung elinor-fung deleted the applicationProtocols-android branch April 16, 2021 18:00
@ghost ghost locked as resolved and limited conversation to collaborators May 16, 2021
@karelz karelz added this to the 6.0.0 milestone May 20, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@bartonjs bartonjs bartonjs left review comments

@jkoritzinsky jkoritzinsky jkoritzinsky approved these changes

@wfurt wfurt Awaiting requested review from wfurt

Assignees
No one assigned
Labels
area-System.Net.Security os-android
Projects
None yet
Milestone
6.0.0
Development

Successfully merging this pull request may close these issues.
4 participants
@elinor-fung @jkoritzinsky @bartonjs @karelz