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

Will it include the base attribute? #19

Open
fireflysemantics opened this issue Jan 7, 2020 · 1 comment
Open

Will it include the base attribute? #19

fireflysemantics opened this issue Jan 7, 2020 · 1 comment

Comments

@fireflysemantics
Copy link

Ran into this issue when using the note path module:
https://stackoverflow.com/questions/59605333/not-using-the-base-attribute-when-making-domsanitizer-api-calls

The path created ends up prefixing the path from the application base attribute, and this is undesired.

Just curious whether this module will duplicate that behavior?

@goto-bus-stop
Copy link
Member

goto-bus-stop commented Jan 8, 2020

I'm not sure what that question is about exactly. If you pass URLs to path you will get bad behaviour because the path module only works with file paths, so it doesn't take into account the scheme:// and other URL-specific things.
Is that the information you're looking for or am I misunderstanding something?

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

No branches or pull requests

2 participants