Opened 7 years ago

Last modified 21 months ago

#7362 new defect

Don't kill the managed proxy if a transport failed to launch

Reported by: asn Owned by:
Priority: Medium Milestone: Tor: unspecified
Component: Core Tor/Tor Version:
Severity: Normal Keywords: tor-bridge pt
Cc: Actual Points:
Parent ID: Points:
Reviewer: Sponsor:

Description

We currently do:

  } else if (!strcmpstart(line, PROTO_SMETHOD_ERROR)) {
    if (mp->conf_state != PT_PROTO_ACCEPTING_METHODS)
      goto err;

    parse_server_method_error(line);
    goto err;

which means that we destroy managed proxies that were supposed to spawn multiple transports but failed to launch one of them.

I think we should log a warning, but keep the managed proxy around. Some transports are better than no transports; and the warning message should hopefully prompt the user into fixing the issue.

Child Tickets

Change History (2)

comment:1 Changed 2 years ago by nickm

Milestone: Tor: very long termTor: unspecified

Batch modify: these tickets seem to be things that wouldn't actually be a big redesign or a big amount of work, so they belong "Unspecified", not "Very Long Term"

comment:2 Changed 21 months ago by teor

Severity: Normal

Set all open tickets without a severity to "Normal"

Note: See TracTickets for help on using tickets.