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

domainRegex used for SIWE domain validation should match subdomains #2362

Closed
1 task done
davisshaver opened this issue Jun 3, 2024 · 3 comments
Closed
1 task done

Comments

@davisshaver
Copy link

Check existing issues

Viem Version

2.13.3

Current Behavior

Subdomains do not currently match the domain regex used for validating SIWE messages.

Example interactive regex: regexr.com/816ai

Expected Behavior

As far as I can tell, subdomains are compliant with RFC 3986 and should match the regex used for validating domains. Subdomains worked with previous implementations of SIWE such as the siwe package.

Steps To Reproduce

No response

Link to Minimal Reproducible Example

https://wp-rainbow.davisshaver.com/2022/03/24/hello-world/

Anything else?

Screenshot 2024-06-03 at 11 48 42 AM Screenshot 2024-06-03 at 11 48 51 AM
@tmm
Copy link
Member

tmm commented Jun 3, 2024

This should take care of that

#2351

@davisshaver
Copy link
Author

davisshaver commented Jun 3, 2024

@tmm Looks like it should, here's a RegExr link which shows the domain validating with the updated regex

@tmm tmm closed this as completed Jun 4, 2024
Copy link
Contributor

This issue has been locked since it has been closed for more than 14 days.

If you found a concrete bug or regression related to it, please open a new bug report with a reproduction against the latest Viem version. If you have any questions or comments you can create a new discussion thread.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jun 20, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants