[tor-bugs] #13035 [Tor Browser]: Make sure our cache isolation works with cache2

[Tor Bug Tracker & Wiki]

#13035: Make sure our cache isolation works with cache2
-------------------------+-------------------------------------------------
     Reporter:  gk       |      Owner:  mcs
         Type:  task     |     Status:  needs_information
     Priority:  major    |  Milestone:
    Component:  Tor      |    Version:
  Browser                |   Keywords:  ff38-esr, tbb-linkability, tbb-
   Resolution:           |  fingerprinting, TorBrowserTeam201506
Actual Points:           |  Parent ID:
       Points:           |
-------------------------+-------------------------------------------------
Changes (by mcs):

 * status:  assigned => needs_information
 * keywords:  ff38-esr, tbb-linkability, tbb-fingerprinting => ff38-esr,
     tbb-linkability, tbb-fingerprinting, TorBrowserTeam201506


Comment:

 I opened a new ticket for updating the automated tests (and posted a
 patch).  See #16356.

 The only remaining question that Kathy and I have for this ticket is
 whether we should patch nsHttpChannel::DoInvalidateCacheEntry() to use our
 modified (isolated) cache keys.  That would involve passing a non-empty
 string as the second parameter to cacheStorage->AsyncDoomURI() within that
 method.  This is not new code and not something we patched in the past...
 and Kathy and I do not understand the implications of not patching it.
 But it seems like the wrong key is being used there.

 Mike, what is your opinion?

--
Ticket URL: <https://trac.torproject.org/projects/tor/ticket/13035#comment:5>
Tor Bug Tracker & Wiki <https://trac.torproject.org/>
The Tor Project: anonymity online