Skip to content
This repository has been archived by the owner on Dec 24, 2020. It is now read-only.

Support ECDSA signing keys #325

Closed
kevinchalet opened this issue Aug 16, 2016 · 1 comment
Closed

Support ECDSA signing keys #325

kevinchalet opened this issue Aug 16, 2016 · 1 comment
Assignees
Milestone

Comments

@kevinchalet
Copy link
Member

IdentityModel supports ECDSA keys, but ASOS' discovery endpoint doesn't expose them yet. This is something I'd really like to fix in the next version.

Unfortunately, supporting EDCSA will have 2 side effects:

  • Referencing the ECDSA types will likely crash on Mono, where these types don't exist.
  • ECDsa.ExportParameters is only available on .NET Standard 1.6/.NET Desktop 4.6.3 (ASOS currently supports .NET Desktop 4.5.1 and .NET Standard 1.4).

Question: do we still need/want to support <.NET Desktop 4.6.3 and <.NET Standard 1.6?

@kevinchalet kevinchalet added this to the 1.0.0-beta7 milestone Aug 16, 2016
@kevinchalet kevinchalet self-assigned this Aug 16, 2016
@kevinchalet
Copy link
Member Author

kevinchalet commented Aug 23, 2016

ASOS will continue to support net451 and netstandard1.4 but ECDSA keys will only be supported when targeting netstandard1.6 (and the future net463 TFM).

Note: the OWIN/Katana version won't have ECDSA support as there's no ECDsaSecurityKey in IdentityModel 4.5.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Development

No branches or pull requests

1 participant