Opened 6 years ago

Closed 6 years ago

#12613 closed defect (fixed)

Tweak PT_STATE_LOCATION for child PTs

Reported by: infinity0 Owned by: RushingWookie
Priority: Medium Milestone:
Component: Circumvention/Pluggable transport Version:
Severity: Keywords: fog
Cc: Actual Points:
Parent ID: Points:
Reviewer: Sponsor:


For every new child PT that is launched, we should append something like "/fog" to PT_STATE_LOCATION, to avoid clobbering equivalent PTs that were launched directly by tor.

(Mildly related to #12612)

Child Tickets

Change History (4)

comment:1 Changed 6 years ago by RushingWookie

I think we should include a uuid in the "/fog" append as that way if the same pt is launched with different parameters, they won't share the same directory.

On second thought this is a problem for pts storing state past each launch as they'll have a new directory each time

Last edited 6 years ago by RushingWookie (previous) (diff)

comment:2 Changed 6 years ago by infinity0

You can do this in torrc too though - define different methods that use the same underlying program.

I think it's OK to follow tor's lead and just use a static directory. If child PTs (of either tor or fog) need to have different state directories when run with different options, they can implement this behaviour themselves and use different subdirectories underneath TOR_PT_STATE_LOCATION.

comment:3 Changed 6 years ago by RushingWookie

Ok, the patch for this has been accepted on the master branch of fog. Closing this issue.

comment:4 Changed 6 years ago by RushingWookie

Resolution: fixed
Status: newclosed
Note: See TracTickets for help on using tickets.