-
Notifications
You must be signed in to change notification settings - Fork 6
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
Save tab set feature not working #10
Comments
I have just tested with Firefox 72.0.1 on Ubuntu and with Nightly on Windows, and I couldn't reproduce it. It's possible that it was a temporary glitch at Pinboard, the server can be slow sometimes. If you investigate this further, please take a look at the request Params tab in the debugger shown in your screenshot, to see what the tab data sent to Pinboard looks like. |
Still no luck on my end :(
Looks legit. I'm running Firefox 72.0.1 macOS. |
I have tested with Firefox 72.0.1 on macOS High Sierra (10.13.6). Couldn't reproduce it. I even tried with your payload in the debugger and it worked fine. You could try doing the operation in a fresh Firefox profile, to see if it's not something related to the browser's cache, settings or other extensions. Sorry, I don't know what else to suggest. |
Thanks for looking into it, I am also lost regarding this issue. |
I experience this too, did you find a solution @pyrho? |
@tjharman, can you reproduce it reliably? Have you tried with a new Firefox profile, no other addons or custom settings? I don’t have ready access to a Mac, but I did test this when first reported and couldn’t reproduce it at all. From pyrho's description, it doesn’t look like the extension is causing this problem. |
Check out this Twitter thread: https://twitter.com/srtcd424/status/1253015359143251968 It is likely related to the problem you’ve been seeing. |
Thanks I'll wait and see what happens. Agree it does look like a pinboard problem! |
@tjharman sorry no, I contacted pinboard support months back (when I first opened this issue), bumped the mail a few times but never got an answer.... :/ |
Hi just to close this off - I have tested again this morning and it's working for me. Thanks all. |
Still no luck for me :( |
Hey, @pyrho, I only now have noticed in the screenshot you have provided that your browser doesn’t send the Pinboard cookies in that POST request. It should send at least two cookies, I have no idea why your browser is not sending the Pinboard cookies. I will investigate this further. If you can think of anything that might cause this, please let me know. |
@gapop thanks so much for (still) looking into this issue :) I have 6 cookies for pinboard, but only the I tried messing with cookie/privacy related settings in FF to no avail. Here is the output of |
It’s really weird. The domain name with a leading dot does not explain it. It’s actually more permissive. A few questions, not that I suspect anything in particular:
|
I don't, it says "You have not granted this site any special permissions"
I have uBlock, but I don't think it acts on cookies.
I'm always logged in to it.
Yes, but pinboard is not assigned to one. A while back I did mess with the cookie settings via about:config that might be the reason :/ I'll try with a fresh FF install see if I still have the issue, that would settle it. edit: formatting |
It's totally working with a fresh FF install, so I definitely messed something up with my current setup, will keep you posted if I ever find the cause. |
Found the culprit after diffing the
|
@pyrho Do you have anything else odd set? I have that ( |
I experimented with the |
Despite this issue being closed, just wanted to 👍 the suggestion for |
Confirming, |
@rubaboo, thanks for your feedback. By my current understanding of how FPI works, I don't think my extension could be made to work with that feature enabled. Rather than using Pinboard’s API, the extension is designed to load pages – the login form, the Save Bookmark form etc. – directly from Pinboard, just like the old official Firefox extension. This preserves the Pinboard experience and completely avoids exposing credentials and other account data to the extension, but it does require the browser to be able to send the Pinboard login cookie for most extension actions. I think FPI prevents exactly this. I will keep in mind the information you’ve provided and I’ll try to find a way, but I don’t want to give you false hope. If you want to use FPI, you will likely be better off with some other extension that uses the Pinboard API. |
I do not understand this issue at all. I was able to reproduce it in a new profile by simply setting After a few login/logout maneuvers and toggling the As a possible workaround for affected users, try logging in and out of Pinboard both with I will leave this open and will dig some more when I get a chance. |
The issue is 100% observable by toggling
|
Hi,
First thanks for this extension, I've been using it daily for the past few years.
The "Save tab set" feature is not working for me, I think it used to work (that is not a feature I use a lot).
I've checked in the debugger and the proper POST request is made to pinboard.in/tabs/save, the request content looks good too, and the server replies with a happy 200 [1].
The code in the
save_tab_set
function executes without errors until the end too and opens up the pinboard.in/tabs/show URL correctly, but once there nothing is shown I just have an empty page.This may be an issue with pinboard itself, I will try to contact their support too.
1:
The text was updated successfully, but these errors were encountered: