[metrics-bugs] #24384 [Metrics/Onionoo]: Decode percent-encoded characters in qualified search terms

Tor Bug Tracker & Wiki blackhole at torproject.org
Tue Mar 24 11:50:00 UTC 2020

#24384: Decode percent-encoded characters in qualified search terms
 Reporter:  karsten          |          Owner:  metrics-team
     Type:  defect           |         Status:  needs_review
 Priority:  Medium           |      Milestone:  Onionoo 2.0.0
Component:  Metrics/Onionoo  |        Version:
 Severity:  Normal           |     Resolution:
 Keywords:                   |  Actual Points:
Parent ID:                   |         Points:
 Reviewer:                   |        Sponsor:
Changes (by karsten):

 * status:  needs_revision => needs_review


 I once more spent way more time on this ticket as I should. I feel like we
 made this more complex than necessary. Trying to summarize my findings:

  - Onionoo should decode percent-encoded characters in all its parameters
 including the search parameter.
  - Onionoo shouldn't accept + where it would otherwise expect a space. If
 clients care to escape spaces, they can percent-encode them, which Onionoo
 should understand.
  - That Relay Search change above looks reasonable to me, and we should
 rebase and merge it. And if it's still insufficient we can make another
  - I'd rather want to avoid writing a search and query specification at
 this point. Honestly, I can only spend a limited number of hours on
 character encoding topics before going mad.

 I made a new patch implementing these two Onionoo changes. Please review
 commit c23de2f in my task-24384-2 branch].

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

More information about the metrics-bugs mailing list