[Author Prev][Author Next][Thread Prev][Thread Next][Author Index][Thread Index]
[tor-bugs] #34225 [Applications/Tor Browser]: Add shortName in nsIURI
#34225: Add shortName in nsIURI
------------------------------------------+--------------------------------
Reporter: sysrqb | Owner: tbb-team
Type: defect | Status: new
Priority: Medium | Milestone:
Component: Applications/Tor Browser | Version:
Severity: Normal | Keywords:
| TorBrowserTeam202009
Actual Points: | Parent ID:
Points: | Reviewer:
Sponsor: |
------------------------------------------+--------------------------------
In #28805, we integrated the https-everywhere aliasing system for
(securedrop) onion addresses. I think we can simplify the patch (and
future patches) by including the alias/name within the current browser
instance uri (`browser.currentURI`). We can achieve this by adding a
`shortName` attribute (or something similar) in `netwerk/base/nsIURI.idl`.
For simplicity, `host` can remain as the onion address, and we can add an
additional attribute for the human-meaningful name.
(Naming is hard, but luckily this is an implementation detail so it
doesn't really matter which attribute label we choose)
This will simplify patches where, in the current situation, we must pass
around both the current `uri` and `onionAliasURI`.
--
Ticket URL: <https://trac.torproject.org/projects/tor/ticket/34225>
Tor Bug Tracker & Wiki <https://trac.torproject.org/>
The Tor Project: anonymity online
_______________________________________________
tor-bugs mailing list
tor-bugs@xxxxxxxxxxxxxxxxxxxx
https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-bugs