UPnP setting doesn't stick #815

issue gavinandresen opened this issue on February 9, 2012
  1. gavinandresen commented at 2:49 PM on February 9, 2012: contributor

    From https://bitcointalk.org/index.php?topic=63165.msg739328#msg739328

    Unchecking UPnP ports doesnt seem to save me selection.

  2. TheBlueMatt commented at 9:03 PM on February 9, 2012: member

    Thats not possible afaict, see OptionsModel::setData in qt/optionsmodel.cpp. In case MapPortUPnP:, there is no modification of successful, meaning it should always return true. Maybe Im missing some part of the way Qt works though.

  3. slothbag commented at 1:08 AM on February 10, 2012: none

    I'm not using any command line args or conf file, I tried it on a different PC and it still wont let me disable UPnP. Everytime I restart the application the UPnP is ticked again.

    I also tried with a brand new empty wallet file and its behaving the same way.

    I also got a few "Application has crashed blah blah C++" errors while shutting down and starting up the app, expected in a RC release I spose.

    I'm using windows 7.

  4. TheBlueMatt commented at 1:17 AM on February 10, 2012: member

    Oh, on reboot, well thats a completely different story.

  5. laanwj commented at 11:50 AM on February 11, 2012: member

    I can reproduce this.

  6. gavinandresen commented at 3:41 PM on February 16, 2012: contributor

    I introduced this when I modified the code to let command-line options override wallet settings.

    I'll fix...

  7. gavinandresen closed this on Feb 28, 2012

  8. lateminer referenced this in commit decee4bc8c on Apr 6, 2019
  9. MarcoFalke locked this on Sep 8, 2021
Labels

Milestone
V0.6


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-15 18:16 UTC

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