Opened 3 years ago

Closed 3 years ago

#21300 closed defect (fixed)

Recent changes broke GETCONF *Port

Reported by: atagar Owned by: nickm
Priority: Very High Milestone: Tor: 0.3.0.x-final
Component: Core Tor/Tor Version:
Severity: Major Keywords: regression
Cc: Actual Points:
Parent ID: Points:
Reviewer: Sponsor:


Hi Nick, shot ya an email earlier about this. Tor changes pushed yesterday broke stem's integ tests. In particular GETCONF is broke. Repro is simple. Here's an example...

% ./tor-prompt

>>> GETINFO config-text
ControlPort 9051
CookieAuthentication 1
DataDirectory /home/atagar/.tor
ExitPolicy reject *:*
HiddenServiceStatistics 0
Log notice stdout
SocksPort 0
250 OK

>>> GETCONF ControlPort
250 ControlPort


Child Tickets

Change History (8)

comment:1 Changed 3 years ago by nickm

Keywords: regression added
Milestone: Tor: 0.3.0.x-final

Hm, probably caused by #20956.

comment:2 Changed 3 years ago by dgoulet

Plausible and to that effect. I've seen this on my relay last night when upgrading. It didn't cause any issues except that warning as far as I can tell.

Jan 24 02:54:10.000 [warn] Can't return context-sensitive 'dirport' on its own

comment:3 Changed 3 years ago by nickm

Summary: Recent changes broke GETCONFRecent changes broke GETCONF *Port

Yeah; we need to make it so asking for FooPort works again.

comment:4 Changed 3 years ago by nickm

Owner: set to nickm
Status: newassigned

comment:5 Changed 3 years ago by nickm

Status: assignedneeds_review

Okay, I think I have a fix. It makes it so *all* LINELIST_S options are queryable individually from the controller interface. It's not useful to get most of them out-of-context, but it isn't harmful. What do we think of my branch bug21300 ?

comment:6 Changed 3 years ago by dgoulet

Tested on my relay, it's fixing the warning I reported earlier. Code lgtm. atagar should ack though once stem tests passes.


comment:7 Changed 3 years ago by atagar

Hi there, Nick can run the stem integ tests just as well as I can. It's just a tor make target. No point in me pulling and compiling. :P

comment:8 Changed 3 years ago by nickm

Resolution: fixed
Status: needs_reviewclosed

I can confirm that this makes the getconf failures go away, so merging it.

Thanks all!

Note: See TracTickets for help on using tickets.