Parameter interaction: disable upnp if -proxy set #6153

pull laanwj wants to merge 1 commits into bitcoin:master from laanwj:2015_05_proxy_noupnp changing 1 files +4 −0
  1. laanwj commented at 9:23 AM on May 18, 2015: member

    To protect privacy, do not use UPNP when a proxy is set. The user may still specify -listen=1 to listen locally (for a hidden service), so don't rely on this happening via the -listen case.

    Fixes #2927.

  2. Parameter interaction: disable upnp if -proxy set
    To protect privacy, do not use UPNP when a proxy is set. The user may
    still specify -listen=1 to listen locally (for a hidden service), so
    don't rely on this happening through -listen.
    
    Fixes #2927.
    8c35b6f3be
  3. laanwj added the label Bug on May 18, 2015
  4. Diapolo commented at 4:18 PM on May 25, 2015: none

    You should update the -upnp help message also to e.g. Use UPnP to map the listening port (default: 1 when listening and no -proxy).

    utACK, when updated

  5. laanwj commented at 5:53 AM on May 26, 2015: member

    I'm not going to change the help message for 0.11 (string freeze), but after that feel free to do that.

  6. laanwj merged this on May 27, 2015
  7. laanwj closed this on May 27, 2015

  8. laanwj referenced this in commit 0bcf04f046 on May 27, 2015
  9. laanwj referenced this in commit ebd7d8d78c on Jun 2, 2015
  10. reddink referenced this in commit 084db0cb83 on May 27, 2020
  11. MarcoFalke locked this on Sep 8, 2021

github-metadata-mirror

This is a metadata mirror of the GitHub repository bitcoin/bitcoin. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2026-04-13 15:15 UTC

This site is hosted by @0xB10C
More mirrored repositories can be found on mirror.b10c.me