kickscondor / fraidycat

Follow blogs, wikis, YouTube channels, as well as accounts on Twitter, Instagram, etc. from a single page.
Other
1.76k stars 56 forks source link

Fraidycat causes Firefox to request every URL twice (when typed in the address bar, outside of the extension) #212

Closed coaxial closed 3 years ago

coaxial commented 3 years ago

Before you start.

Desktop (please complete the following information):

Describe the bug When FraidyCat is enabled as an addon, any URL I enter in the URL bar (in the browser, the one you get with Ctrl-L) is requested twice. The addon just has to be enabled for this to happen, it doesn't have to be open or anything. There is no reason for the second GET request to happen, the browser should only fetch the requested URL once.

Repro:

http server logs:

$ python3 -m http.server
Serving HTTP on 0.0.0.0 port 8000 (http://0.0.0.0:8000/) ...
# Initial load, addon disabled
127.0.0.1 - - [17/Mar/2021 19:56:28] "GET / HTTP/1.1" 200 -
127.0.0.1 - - [17/Mar/2021 19:56:28] code 404, message File not found
127.0.0.1 - - [17/Mar/2021 19:56:28] "GET /favicon.ico HTTP/1.1" 404 -

# refresh, addon enabled
127.0.0.1 - - [17/Mar/2021 19:56:57] "GET / HTTP/1.1" 200 -
127.0.0.1 - - [17/Mar/2021 19:56:57] "GET / HTTP/1.1" 200 -

# refresh, addon disabled
127.0.0.1 - - [17/Mar/2021 19:57:13] "GET / HTTP/1.1" 200 -

# refresh, addon enabled
127.0.0.1 - - [17/Mar/2021 20:00:13] "GET / HTTP/1.1" 200 -
127.0.0.1 - - [17/Mar/2021 20:00:13] "GET / HTTP/1.1" 200 -

# refresh, addon disabled
127.0.0.1 - - [17/Mar/2021 20:00:29] "GET / HTTP/1.1" 200 -
kickscondor commented 3 years ago

Sweet! Thank you for this test case - this is addressed by aabefcd. And is also covered in #178. Hope to have it out very soon.

Sincerest apologies for this bug.

coaxial commented 3 years ago

No worries, thanks for your hard work!

kickscondor commented 3 years ago

Ok the code that was doing this has been replaced and an update is out now in Firefox. Thanks for holding the line. Let me know how it goes!

coaxial commented 3 years ago

Thanks, looks like the fix works for me.