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

Firefox extension omits : after http(s) #68

Closed
jnv opened this issue Jan 26, 2024 · 6 comments
Closed

Firefox extension omits : after http(s) #68

jnv opened this issue Jan 26, 2024 · 6 comments
Assignees
Milestone

Comments

@jnv
Copy link

jnv commented Jan 26, 2024

Steps to reproduce

  1. Install BT extension for Firefox
  2. Click the extension button on some website
  3. Open link from BT in any browser

Expected behavior

The new browser opens the original link.

Actual behavior

The new browser opens the original link without : after https, resulting in connection error page, for example: https//github.com

Notes

Since there were no recent releases of BT or the extension, I suspect this could be caused by very recent update of Firefox to v122. Extension in Edge seems unaffected.

Versions:

  • Windows 11 Enterprise, Version 22H2, OS build 22621.3007
  • Firefox 122.0 (64-bit)
  • Extension: 1.2.1
  • BT 3.6.2
@jnv
Copy link
Author

jnv commented Jan 26, 2024

Hmm, I'm thinking whether there was a change in URL handling mechanism:

browser.tabs.update(tab.id, {
url: "x-bt://" + url
});

The URL added after x-bt:// isn't encoded, so it's possible Firefox trims the : character in order to perhaps prevent some security issue?

@aloneguid
Copy link
Owner

Thanks, I can reproduce this on my Firefox, and it used to work (nothing changed since then) so it's definitely something in the Firefox itself. Will keep digging and come up with a fix.

@aloneguid aloneguid added this to the 3.7.2 milestone Feb 2, 2024
@aloneguid aloneguid self-assigned this Feb 2, 2024
@aloneguid aloneguid added the core label Feb 2, 2024
aloneguid added a commit that referenced this issue Feb 2, 2024
@aloneguid
Copy link
Owner

I can confirm Firefox it removing : - the URL arrives to BT without it. 3.7.2 will fix this issue.

@aloneguid
Copy link
Owner

Please try installing 3.7.2. Firefox extension does not need an update.

@jnv
Copy link
Author

jnv commented Feb 3, 2024

Thank you Ivan! I'm off for a week, but will try it first thing once I'm back.

@jnv
Copy link
Author

jnv commented Feb 12, 2024

@aloneguid The fix works for me in BT v3.7.3. Thank you Ivan!

@jnv jnv closed this as completed Feb 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants