#19757: Make a menu to add onion and auth-cookie to TB -------------------------------------------------+------------------------- Reporter: mrphs | Owner: brade Type: defect | Status: | needs_revision Priority: Medium | Milestone: Component: Applications/Tor Browser | Version: Severity: Normal | Resolution: Keywords: ux-team, tbb-usability, tor-hs, | Actual Points: 6.3 TorBrowserTeam202001 | Parent ID: #30000 | Points: 8 Reviewer: pospeselr, acat | Sponsor: | Sponsor27-must -------------------------------------------------+-------------------------
Comment (by mcs): Replying to [comment:31 acat]: > In `savedKeysDialog.js`, line 64, `controllerFailureMsg` is not defined in that scope. I guess it should be declared outside the `try`, as you do in `authPrompt.js` [eslint caught this one :) ] Good catch! We will fix it. > Speaking of eslint, I'm not sure whether we made a decision about enforcing it for new code or not but at least with the latest patches of pospeselr or mine we tried to do so. I leave it up to you, as I think it's not so important and can always be done later. `./mach lint -l eslint --fix browser/components/onionservices` should fix all the auto-fixable issues. We will try using eslint soon. Thanks! > I tested a bit and saw a couple of cases with a behaviour I'm not sure is the intended one: > > * Added private key, checked "Don't remember" -> after "New Identity" accessing to the .onion still works (I would expect we clear that on New Identity). This is an interesting test case. Kathy and I agree that "in memory" keys should be cleared upon New Identity. But we should ask the Network Team (asn and dgoulet) how they think it should work. > * Added private key, checked "Remember", removed from `about:preferences` -> accessing to the .onion still works (but not after "New Identity", which is surprising given the behaviour above). I think this second test case points to a bug in tor. Kathy and I are able to reproduce it. It looks like a new circuit is used to load the .onion the second time, so it is not just a matter of circuit reuse. -- Ticket URL: <https://trac.torproject.org/projects/tor/ticket/19757#comment:32> Tor Bug Tracker & Wiki <https://trac.torproject.org/> The Tor Project: anonymity online
_______________________________________________ tor-bugs mailing list tor-bugs@lists.torproject.org https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-bugs