[tor-bugs] #23981 [Metrics/CollecTor]: Fix NPE and include "bridge-distribution-request" lines in sanitized bridge descriptors (was: Include "bridge-distribution-request" lines in sanitized bridge descriptors)

[Tor Bug Tracker & Wiki]

#23981: Fix NPE and include "bridge-distribution-request" lines in sanitized bridge
descriptors
-------------------------------+--------------------------
 Reporter:  karsten            |          Owner:  iwakeh
     Type:  defect             |         Status:  accepted
 Priority:  High               |      Milestone:
Component:  Metrics/CollecTor  |        Version:
 Severity:  Normal             |     Resolution:
 Keywords:                     |  Actual Points:
Parent ID:                     |         Points:
 Reviewer:                     |        Sponsor:
-------------------------------+--------------------------

Old description:

> From CollecTor's logs:
>
> {{{
> 2017-10-25 02:09:20,131 WARN o.t.c.b.SanitizedBridgesWriter:952
> Unrecognized line 'bridge-distribution-request any'. Skipping.
> }}}
>
> We should investigate what the exact specification of this line is,
> decide whether it's harmless to keep, add it to CollecTor's bridge
> descriptor module, try it out locally, get it reviewed, put out a
> release, and deploy that.

New description:

 For NPE log excerpt see comment:1.

 From CollecTor's logs:

 {{{
 2017-10-25 02:09:20,131 WARN o.t.c.b.SanitizedBridgesWriter:952
 Unrecognized line 'bridge-distribution-request any'. Skipping.
 }}}

 We should investigate what the exact specification of this line is, decide
 whether it's harmless to keep, add it to CollecTor's bridge descriptor
 module, try it out locally, get it reviewed, put out a release, and deploy
 that.

--

Comment (by iwakeh):

 Adapted summary and description.

 Maybe open a different ticket for the unrecognized line?

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