Extend flag parameter to support comma-separated list of flags

split from https://trac.torproject.org/projects/tor/ticket/23829?replyto=3#comment:3

The flag parameter currently does not accept a comma-separated list of flags. We could extend it towards doing so.

added component::metrics/onionoo owner::karsten priority::medium resolution::wontfix severity::normal status::closed type::enhancement labels

Sounds good to me. We might consider doing #23913 (moved) to support queries like search=flag:exit flag:guard rather than search=flag:exit,guard. But we might also support both. I don't feel strongly, even though that might change when I start implementing this. Let's see.

Trac:
Status: new to accepted
Owner: metrics-team to karsten

This would be required to allow aggregations in Relay Search of "Fast+Exit".

This still seems like a worthwhile enhancement, but I'm not going to find the time to do it anytime soon. Re-assigning to metrics-team for now.

Trac:
Status: accepted to assigned
Owner: karsten to metrics-team

Adding this back to my list. Regarding comment 1 above, I'd say let's explicitly support flag=exit,guard now and leave multiple parameters as in flag=exit&flag=guard for all parameters for #23913 (moved).

Trac:
Status: assigned to accepted
Owner: metrics-team to karsten

Please review commit 49cdff7 in my task-23914 branch.

Trac:
Status: accepted to needs_review

Hmm, maybe we can't make this change after all. With the currently planned flag=exit,guard parameter, we'd return relays that have both flags (AND). But if we also add a parameter like lookup=fingerprint1,fingerprint2, we'd return relays with either of the two fingerprints (OR). This might be too confusing, especially if we add more parameters that treat comma-separated values as OR, not AND. Maybe we'll have to fall back to #23913 (moved) with two parameters flag=exit and flag=guard to support searches for relays having all required flags (AND). Moving out of review for now.

Trac:
Status: needs_review to accepted

Closing this ticket for the reason stated in the previous comment and after leaving a comment on #23913 (moved).

Trac:
Status: accepted to closed
Resolution: N/A to wontfix

closed

mentioned in issue #25448 (moved)