Note: There are a few cases of duplicates in user autocompletion which are being worked on.

Clear Adobe Flash Cookies (LSOs) when Cookies is selected in Clear Recent History

VERIFIED FIXED in Firefox 4.0b12

Status

()

Firefox
Private Browsing
--
enhancement
VERIFIED FIXED
7 years ago
4 years ago

People

(Reporter: beltzner, Assigned: dwitte@gmail.com)

Tracking

(Depends on: 1 bug)

Trunk
Firefox 4.0b12
Points:
---
Dependency tree / graph
Bug Flags:
in-testsuite +

Firefox Tracking Flags

(blocking2.0 final+)

Details

(Whiteboard: [softblocker][requires Flash 10.3])

Attachments

(1 attachment, 1 obsolete attachment)

10.73 KB, patch
dwitte@gmail.com
: review+
Details | Diff | Splinter Review
Bug 508167 will implement support for the NPAPI methods that Adobe Flash will support in Adobe Flash 10.2+, allowing us to tell that application to delete their version of cookies (LSOs) as well.

Previously we have seen a lot of evidence that users are surprised that Flash objects store data on their system and further that this data isn't cleared when browser cookies are cleared. It seems evident that to the user, cookies = cookies, and so the best way to match expectations would be to interpret cookie-clearing actions as meaning "web and flash cookies".

When "Clear Recent History..." is selected from the Tools menu, if the "Cookies" checkbox is selected, we should also use the new API from bug 508167 to clear Adobe Flash LSOs for the same time period. Yes, this will clear all LSOs even those set by other browsers; that's acceptable, IMO.
Severity: normal → enhancement
Component: Menus → Private Browsing
OS: Mac OS X → All
QA Contact: menus → private.browsing
Hardware: x86 → All
Version: unspecified → Trunk
Blocks: 565561
(Assignee)

Updated

7 years ago
Blocks: 625495
(Assignee)

Comment 1

7 years ago
Created attachment 509015 [details] [diff] [review]
patch

This adds relevant bits to sanitize.js, and some tests. Ehsan, I'm not sure if you want to review this or delegate to Josh, who's reviewing the backend bits in bug 508167 -- it might make sense for both of you to look at it, since Josh will be familiar with how the guts works.
Assignee: nobody → dwitte
Status: NEW → ASSIGNED
Attachment #509015 - Flags: review?(ehsan)
Comment on attachment 509015 [details] [diff] [review]
patch

>+        // Clear plugin data.
>+        let ph = Components.classes["@mozilla.org/plugin/host;1"]
>+                           .getService(Ci.nsIPluginHost);
>+        const phInterface = Ci.nsIPluginHost_MOZILLA_2_0_BRANCH;
>+        const FLAG_CLEAR_ALL = phInterface.FLAG_CLEAR_ALL;
>+        ph.QueryInterface(phInterface);

Please use Cc and Ci in the new code you're adding too.

>+        // Determine age range in seconds. (-1 means clear all.) We don't know
>+        // that this.range[1] is actually now, so we compute age range based
>+        // on the lower bound.
>+        let age = this.range ? (Date.now() / 1e3 - this.range[0] / 1e6) : -1;

s/1e3/1000/ and s/1e6/1000000/.  This code is unreadable enough as it stands.  ;-)

>+        let tags = ph.getPluginTags();
>+        for (let i = 0; i < tags.length; i++) {
>+          try {
>+            ph.clearSiteData(tags[i], null, FLAG_CLEAR_ALL, age);
>+          } catch (e) {
>+            // If the plugin doesn't support clearing by age, clear everything.
>+            if (e.result == Cr.NS_ERROR_PLUGIN_TIME_RANGE_NOT_SUPPORTED) {

Also check here that age != -1.

>+              try {
>+                ph.clearSiteData(tags[i], null, FLAG_CLEAR_ALL, -1);
>+              } catch (e) {
>+                // Ignore errors from the plugin
>+              }
>+            }
>+          }
>+        }
>+

r=me with the above nits addressed.
Attachment #509015 - Flags: review?(ehsan) → review+
(Assignee)

Comment 3

7 years ago
Created attachment 509263 [details] [diff] [review]
patch v2

Fixed nits.
Attachment #509015 - Attachment is obsolete: true
Attachment #509263 - Flags: review+
(Assignee)

Comment 4

7 years ago
Comment on attachment 509263 [details] [diff] [review]
patch v2

Requesting approval. This is a softblocker and wanted for 2.0.
Attachment #509263 - Flags: approval2.0?
(Assignee)

Updated

7 years ago
Whiteboard: [softblocker]
If this is a softblocker, it should be marked as such.
blocking2.0: --- → ?
blocking2.0: ? → final+
Attachment #509263 - Flags: approval2.0?
(Assignee)

Comment 6

7 years ago
http://hg.mozilla.org/mozilla-central/rev/2ff5b8624463
Status: ASSIGNED → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED
(Assignee)

Comment 7

7 years ago
(Wrong link, should be http://hg.mozilla.org/mozilla-central/rev/37094ed97c9e)
Blocks: 632746
Flags: in-testsuite+
Target Milestone: --- → Firefox 4.0b12

Updated

7 years ago
Depends on: 633433

Updated

7 years ago
Depends on: 633463

Updated

7 years ago
No longer depends on: 633463

Updated

7 years ago
Depends on: 633463

Updated

7 years ago
No longer depends on: 633463

Comment 8

7 years ago
I had a look at the current revision of the file sanitize.js and noticed a weird mixture of Cc and Ci and Components.classes and Components.interfaces
That file could use a check and cleanup

Some examples:
140         var cookieMgr = Components.classes["@mozilla.org/cookiemanager;1"]
141                                   .getService(Ci.nsICookieManager);

208         const Cc = Components.classes;
209         const Ci = Components.interfaces;
210         var cacheService = Cc["@mozilla.org/network/cache-service;1"].
211                            getService(Ci.nsICacheService);

http://mxr.mozilla.org/mozilla-central/source/browser/base/content/sanitize.js
(Assignee)

Comment 9

7 years ago
Yup, feel free to file a bug and patch!

Updated

7 years ago
Depends on: 633463

Comment 10

7 years ago
Just one question, shouldn't be this functionality also added under Options->Privacy->"Show Cookies.."->"Remove All Cookies"? When user click this button expects all cookies to be deleted including LSOs, since ff4 will now support deleting of LSOs.

Updated

6 years ago
Depends on: 646074

Comment 11

6 years ago
Still reproduceble on Mozilla/5.0 (Windows NT 6.1; rv:2.2a1pre) Gecko/20110404 Firefox/4.2a1pre.

The .sol files are not deleted when the Tools > Clear Recent History item is clicked.
I have to delete manually the .sol files.

Comment 12

6 years ago
I also can still reproduce. I think this bug needs to be reopened.
Vlad or Will, can you please tell me which URL you are using to test this fix?

Comment 14

6 years ago
I have accumulate some history (random pages), no specific URL.
(In reply to comment #14)
> I have accumulate some history (random pages), no specific URL.

Can you please tell me a Flash application which definitely stores a Flash cookie on disk? Should be one of those entries from within your history. Thanks.

Comment 16

6 years ago
Any URL will produce these results for me, but Gmail and YouTube are a couple in specific.

Comment 17

6 years ago
Youtube for example. I have tried on the latest build, Mozilla/5.0 (Windows NT 6.1; rv:6.0a1) Gecko/20110501 Firefox/6.0a1 and the issue it's still reproducible. After clearing the History, the lso's are still present in the Macromedia folder under Win7. (Users/user/Appdata/Roaming/Macromedia/FlashPlayer/#SharedObjects/"random created directory/)
Ok, was able to get it to reproduce on OS X with Fx4 final. No .sol files are getting deleted via the clear recent history dialog or on shutdown. Same with Aurora or Nightly builds.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---

Comment 19

6 years ago
Make sure you have Flash 10.2 or newer, as Adobe only seems to have implemented the interface there.
That has already been told in comment 0. So I'm aware of it. Just for reference here my installed version of Flash:

    Version: 10.2.159.1
    Shockwave Flash 10.2 r159

Comment 21

6 years ago
I see this on Linux as well with:

Mozilla/5.0 (X11; Linux i686; rv:6.0a1) Gecko/20110502 Firefox/6.0a1

and:

Shockwave Flash

    File: libflashplayer.so
    Version: 
    Shockwave Flash 10.2 r159

Steps to reproduce:

1. Delete ~/.macromedia and ~/.adobe

2. Close all browsers.

3. Start Nightly and create a fresh profile (./firefox --profilemanager)

4. Start with that profile and go to youtube.com

5. Start playing any video.

6. Open a new blank tab and close all other tabs (no need to watch any of the video).

7. Go to Tools -> Clear Recent History...

8. Select "Everything" for the time range and check all options, then "Clear Now".

9. Close the browser.

10. cd ~/.macromedia

11. ls --recursive

Results:

================================================

~/.macromedia$ ls --recursive
.:
Flash_Player

./Flash_Player:
macromedia.com  #SharedObjects

./Flash_Player/macromedia.com:
support

./Flash_Player/macromedia.com/support:
flashplayer

./Flash_Player/macromedia.com/support/flashplayer:
sys

./Flash_Player/macromedia.com/support/flashplayer/sys:
settings.sol  #s.ytimg.com

./Flash_Player/macromedia.com/support/flashplayer/sys/#s.ytimg.com:
settings.sol

./Flash_Player/#SharedObjects:
H73THJQY

./Flash_Player/#SharedObjects/H73THJQY:
s.ytimg.com

./Flash_Player/#SharedObjects/H73THJQY/s.ytimg.com:
videostats.sol

================================================

The ~/.macromedia/Flash_Player/macromedia.com/support/flashplayer/sys/settings.sol file is for the global settings of the Flash player and doesn't reveal any private information as far as I can tell.

The ~/.macromedia/Flash_Player/macromedia.com/support/flashplayer/sys/#s.ytimg.com/settings.sol file is for that site's individual settings (such as "always ask to use the webcam"). The file itself I don't think reveals any private information, but clearly the directory name does.

The ~/.macromedia/Flash_Player/#SharedObjects/H73THJQY/s.ytimg.com/videostats.sol is the site's cookie which could reveal a number of pieces of private information (and obviously the directory name does as well).

The last two files can be seen in the Flash Settings Manager here: http://www.macromedia.com/support/documentation/en/flashplayer/help/settings_manager06.html#117645 and here: http://www.macromedia.com/support/documentation/en/flashplayer/help/settings_manager07.html#117717

The ~/.adobe directory also gets created again but there don't seem to be any files in there.

cd ~/.adobe

ls --recursive

================================================

~/.adobe$ ls --recursive 
.:
Flash_Player

./Flash_Player:
AssetCache

./Flash_Player/AssetCache:
269N8LUX

./Flash_Player/AssetCache/269N8LUX:

================================================
Josh, can you take a look, please?

Comment 23

6 years ago
I believe the functionality in question here is supported only in Flash 10.3. Can someone confirm that it works with the Flash 10.3 beta or release candidate?

Comment 24

6 years ago
(In reply to comment #23)
> I believe the functionality in question here is supported only in Flash 10.3.
> Can someone confirm that it works with the Flash 10.3 beta or release
> candidate?

I just tested the functionality in Nightly 2011-05-04 and Flash Player 10.3.181.5 RC 1 from FileHippo, and the LSOs do clear properly on browser exit along with the regular cookies.
(In reply to comment #23)
> I believe the functionality in question here is supported only in Flash 10.3.
> Can someone confirm that it works with the Flash 10.3 beta or release
> candidate?

Also checked the web today and you are right. It's not Flash 10.2 we have to test with but Flash 10.3:

https://threatpost.com/en_us/blogs/chrome-ie-give-users-ability-delete-flash-cookies-050411

Vlad and Daniel, can you both please test on your platform? I will check on OS X.

For now I will mark this bug as fixed again.
Status: REOPENED → RESOLVED
Last Resolved: 7 years ago6 years ago
Resolution: --- → FIXED
Whiteboard: [softblocker] → [softblocker][requires Flash 10.3]

Comment 26

6 years ago
After repeating the steps from comment17, there are no LSO's in the folder, not even if I watched some youtube clips, instead in that folder are some SOL files, and these cannot be deleted only manual. I don't know if this is intended or not.
Works fine for me now with Flash 10.3RC1 and Mozilla/5.0 (Macintosh; Intel Mac OS X 10.6; rv:2.0.1) Gecko/20100101 Firefox/4.0.1

Any files including the .sol files are getting deleted under:
~/Library/Preferences/Macromedia/Flash Player/#SharedObjects

Only the one .sol file under the following folder remains because it seems to contain some general settings:

~/Library/Preferences/Macromedia/Flash Player/macromedia.com/support/flashplayer/sys

Vlad and Daniel, can you please re-test on Windows and Linux? Thanks.

Comment 28

6 years ago
Works fine for me using the latest flash 10.3 rc on: 
Mozilla/5.0 (Windows NT 6.1; rv:6.0a1) Gecko/20110511 Firefox/6.0a1

and also on:
Mozilla/5.0 (Macintosh; Intel Mac OS X 10.6; rv:6.0a1) Gecko/20110511 Firefox/6.0a1

Due to some technical issues, I couldn't see if it's working or not on Ubuntu. Perhaps Daniel Cater will help us in this matter so that the status can be changed to Verified Fixed

Comment 29

6 years ago
I didn't test it earlier because I hadn't worked out where in the home directory to install the release candidate (so that the stable version stayed in /usr).

Turns out $HOME/.mozilla/plugins/ is the place, but it doesn't exist by default.

Looks like 10.3 final was just released, so I tested with that (Shockwave Flash 10.3 r181).

Following the same steps as I did in comment 21, I can see that this is fixed. The directories and files that contain private information disappear from the filesystem, and also from the Flash settings manager.

People on Firefox 4 and above should see the benefits of this once Flash 10.3 is pushed out as an update by Adobe.
Thanks Daniel! Setting bug to verified fixed.
Status: RESOLVED → VERIFIED

Comment 31

6 years ago
This change had a destructive effect on my flash local shared objects, as I upgraded to the newest firefox 4 + flash plugin 10.3.
It was only games, so I don't care that much.

I would have thought that the flash shared objects could be a subset of the cookies/history thing, so as to be controlled separately.

The cookies could have the same kind of Settings button that history has.

The upgrade of a "keep cookies until I close firefox" could have been mapped to a "keep cookies until I close firefox" with settings containing a deselected flash shared obj checkbox, with the same cookie settings effecting the "clear history when firefox closes".

It is difficult (impossible?) to reproduce the behaviour I would like, which is to remove all cookies and history at each browser close, with the exception of local shared objects, which I'd like to keep.
Claudio please file a new bug for it. This bug is closed. Thanks.
Depends on: 886832
You need to log in before you can comment on or make changes to this bug.