Skip to content

Make possible for origin to be a list of domains #49

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

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

tanguilp
Copy link
Owner

WIP.

Hard to determine how we should test it.

@tanguilp tanguilp marked this pull request as draft March 30, 2025 19:34
@tanguilp
Copy link
Owner Author

cc @idyll

@idyll
Copy link

idyll commented Mar 30, 2025

I appreciate the thinking here but this isn't going to handle the case that was hoping to address in my PR: allow subdomains to authenticate with the same Passkey as the bare domain (unless I know all the subdomains ahead of time.)

Totally understand that you're super worried about problems either through malice or incompetence. (Although I'm going to pretend you think I'm evil rather than dumb if that's okay.)

My fork is working fine for me for now, but at some point I will rework the PR to explain exactly what it's doing and why. And why I don't think there will be an issue with it.

@tanguilp
Copy link
Owner Author

Definitely assuming malice ;)

How about an optional origin check function? With an example of how to check for subdomains dynamically, using for instance https://hexdocs.pm/domainname/DomainName.html#ends_with?/2

Thinking about this, tell me what's your opinion.

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

Successfully merging this pull request may close these issues.

2 participants