[Author Prev][Author Next][Thread Prev][Thread Next][Author Index][Thread Index]
Re: [tor-bugs] #29434 [Core Tor/Tor]: Consider removing linux-tor-prio.sh from contrib/operator-tools
#29434: Consider removing linux-tor-prio.sh from contrib/operator-tools
-------------------------------------------------+-------------------------
Reporter: rl1987 | Owner: (none)
Type: enhancement | Status:
| merge_ready
Priority: Medium | Milestone: Tor:
| 0.4.1.x-final
Component: Core Tor/Tor | Version:
Severity: Normal | Resolution:
Keywords: technical-debt, fast-fix, nickm- | Actual Points: 0.1
merge, asn-merge |
Parent ID: | Points: 0.1
Reviewer: juga, teor | Sponsor:
-------------------------------------------------+-------------------------
Changes (by teor):
* status: needs_revision => merge_ready
* points: => 0.1
* actualpoints: => 0.1
* keywords: => technical-debt, fast-fix, nickm-merge, asn-merge
* reviewer: juga => juga, teor
* type: task => enhancement
Comment:
I have reviewed this pull request, and I think we should remove this
script.
(But now I can't merge it, because I have reviewed it.)
Here is why:
Replying to [comment:5 juga]:
> Replying to [comment:2 rl1987]:
> > I asked [0] and nobody replied. I think it's safe to remove it.
>
> i found the link [0]: https://lists.torproject.org/pipermail/tor-
relays/2019-February/016995.html
>
> In a fast search i found that someone was still using it in 2015 [1].
Our earliest supported release is 0.2.9, which was released in December
2016:
https://trac.torproject.org/projects/tor/wiki/org/teams/NetworkTeam/CoreTorReleases#Current
So we don't support things that were only used back in 2015.
And we never fixed a bug in the script, where it confuses kilobits and
kilobytes:
https://lists.torproject.org/pipermail/tor-talk/2015-January/036458.html
So the script has been unmaintained for at least 4 years.
> Just some questions:
> Is it possible that some other application/device using Tor is using
it?.
If they are, they can keep a copy of the script, and maintain it
themselves.
> Any idea on how we could find out easy?.
Remove the script, document the removal in the release notes, and see if
someone complains.
> Do you know if there's some other recommended way to achieve the same
this script does?.
BandwidthRate and BandwidthBurst are supported, cross-platform torrc
options that limit Tor's bandwidth. They are integrated with the bandwidth
weighting system.
I would not recommend using this script instead, because large changes in
relay bandwidth are not a good experience for Tor users. (And they lead to
inaccurate bandwidth weights on the network.)
> Should the script itself be removed from too?.
This pull request removes the script from Tor's future releases.
--
Ticket URL: <https://trac.torproject.org/projects/tor/ticket/29434#comment:6>
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