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

docs: explain windows 10 alg limitation #258

Merged
merged 1 commit into from
Nov 19, 2024
Merged

docs: explain windows 10 alg limitation #258

merged 1 commit into from
Nov 19, 2024

Conversation

cpb8010
Copy link
Contributor

@cpb8010 cpb8010 commented Nov 19, 2024

Description

Because windows 10 only supports RS256, it's not supported by our ES256
verifier, so until we add an RS256 verifier this platform won't work.
We've attempted to limit these sign-ups via the browser, but it doesn't
appear to stop users from choosing that option. Documenting it is the
next best thing, but we'll probably add a more explicit error next.

Linked Issues

https://github.com/matter-labs/zksync-account-sdk/issues/182

Additional context

The current NFT demo (as of this writing) will also fail for Windows 11, so it requires a small update so these docs are correct

Because windows 10 only supports RS256, it's not supported by our ES256
verifier, so until we add an RS256 verifier this platform won't work.
We've attempted to limit these sign-ups via the browser, but it doesn't
appear to stop users from choosing that option. Documenting it is the
next best thing, but we'll probably add a more explicit error next.
@cpb8010 cpb8010 requested a review from a team as a code owner November 19, 2024 02:17
@cpb8010 cpb8010 self-assigned this Nov 19, 2024
Copy link

Visit the preview URL for this PR (updated for commit 3a7b637):

https://zksync-docs-staging-5eb09--pr258-sso-windows-piio5gy1.web.app

(expires Tue, 26 Nov 2024 02:21:32 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: bfaafba5fa82d4f63473aaa76a21fabf1fbb3a11

@cpb8010 cpb8010 merged commit 18340f1 into main Nov 19, 2024
9 checks passed
@cpb8010 cpb8010 deleted the sso-windows branch November 19, 2024 15:06
Copy link

🎉 This PR is included in version 1.44.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants