Lusito / forget-me-not

Make the browser forget website data, except for the data you want to keep.
zlib License
227 stars 25 forks source link

Some cookies not being auto-deleted #40

Closed Joel889 closed 6 years ago

Joel889 commented 6 years ago

Go to github.com, close tab, cookies remain even after the timer-based cleaning mechanism has run. There are no whitelist/greylist etc settings for that domain. Manually running clean-up gets rid of them.

Joel889 commented 6 years ago

Do you know when you'll have time to fix this? It's sort of a big problem that directly affects the addon's main functionality.

Lusito commented 6 years ago

It's actually fixed in the code already.. the Problem I have right now is that mozilla removed the possibility to release beta versions, so I'll have to extensively test the new code all by myself before releasing a new version (and there are many things to keep in mind).

The past few weeks I was busy getting a new apartment in the city where my new job is and driving back and forth between cities. I finally got one, so I should be able to put some time into it. Hopefully I'll be able to release a new version within the week.

Sorry about the delay.

Joel889 commented 6 years ago

It appears this is still an issue, but to a much lesser extend. I'm not sure how to reproduce it, but even without having any whitelist, there are a few cookies that did not get deleted automatically even with all tabs closed for a while.

Lusito commented 6 years ago

Is it maybe rather related to #38 (i.e. subdomain cookies)? Or could it be, that the cookies can't be deleted at all (see Bug 1440263)? I.e. does manual cleanup also not remove them?

If not, this is probably still a different issue. I'd need more detailed information about the cookies not being removed.

Joel889 commented 6 years ago

Download anything from sourceforge [.] net. This particular 3rd party cookie isn't being removed, even though 3rd party auto cleaning is enabled. The cookie disappears when clearing manually through the extension.

Lusito commented 6 years ago

Thanks for the example, I will try to reproduce it.