Opened 5 weeks ago

Closed 12 days ago

#23237 closed enhancement (implemented)

Add 'GETINFO ip-to-country/available'

Reported by: atagar Owned by: nickm
Priority: Low Milestone: Tor: 0.3.2.x-final
Component: Core Tor/Tor Version:
Severity: Minor Keywords: easy, tor-control, review-group-23
Cc: Actual Points: .1
Parent ID: Points:
Reviewer: dgoulet Sponsor:

Description

Hi Nick. Very minor ask but if we had a 'GETINFO ip-to-country/available' option to determine if tor has a geoip database available that would simplify Stem a bit. Stem tracks 'is the geoip database available' so it can avoid 'GETINFO ip-to-country/*' requests that are doomed to fail anyway.

Thanks!

Child Tickets

Change History (9)

comment:1 Changed 5 weeks ago by catalyst

Keywords: easy tor-control added
Milestone: Tor: 0.3.2.x-final

comment:2 Changed 4 weeks ago by nickm

Yes, we should totally do this. Anybody have a moment, or should I take it on?

comment:3 Changed 4 weeks ago by nickm

Actual Points: .1

Branch feature23237 in my public repository; corresponding torspec patch in feature23237_spec.

comment:4 Changed 4 weeks ago by nickm

Owner: set to nickm
Status: newaccepted

comment:5 Changed 4 weeks ago by nickm

Status: acceptedneeds_review

comment:6 Changed 4 weeks ago by atagar

Thanks Nick! Spec change looks good to me.

comment:7 Changed 13 days ago by nickm

Keywords: review-group-23 added

Put 0.3.2 needs_review and merge_ready tickets into review-group-23.

comment:8 Changed 13 days ago by dgoulet

Reviewer: dgoulet
Status: needs_reviewmerge_ready

lgtm;

comment:9 Changed 12 days ago by nickm

Resolution: implemented
Status: merge_readyclosed

Merged!

Note: See TracTickets for help on using tickets.