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

Unable to open extension urls, moz-extenstion protocol #310

Closed
amithm7 opened this issue Feb 18, 2023 · 4 comments
Closed

Unable to open extension urls, moz-extenstion protocol #310

amithm7 opened this issue Feb 18, 2023 · 4 comments
Assignees

Comments

@amithm7
Copy link

amithm7 commented Feb 18, 2023

I'm trying to open extension url like this:

moz-extension://3b12022a-75a7-4189-8431-a45b8c5ff24e/popup/index.html#inTab

But this url get's converted to an http protocol url automatically.

https://moz-extension//3b12022a-75a7-4189-8431-a45b8c5ff24e/popup/index.html#inTab
@AndydeCleyre
Copy link

I just installed this extension for the purpose of setting new tab to a moz-extension:// URI, FWIW. Thanks @amithm7 for not only reporting this but submitting the PR.

@cadeyrn
Copy link
Owner

cadeyrn commented Feb 28, 2023

Thank you for the report and the pull request. In the past it was not possible to use moz-extension URIs as new tab page. I will check if this works now but it will take a few more days because I am very busy with other high priority projects right now. I just didn't want to keep you waiting any longer without feedback.

@cadeyrn cadeyrn added this to the nto.next milestone May 7, 2023
@clach04
Copy link

clach04 commented Jul 19, 2023

@amithm7 does this allow data payloads that are base64 encoded?

data:text/html;base64,....

Is so this would solve the use case I'd like which is to spin up a page with html without hitting a web server (local files are not allowed in the extension apparently).

@amithm7
Copy link
Author

amithm7 commented Jul 20, 2023

@clach04 No, the PR I opened doesn't consider data: URI's. Only protocols of the form protocol://, does it consider to be excluded from automatically adding http://.

@cadeyrn cadeyrn closed this as completed in a9f35cc Feb 3, 2024
@cadeyrn cadeyrn self-assigned this Feb 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants