Controller's get_conf_map concurrently modified dictionary

Oddly I'm not reproing this issue when I run the tests, but I definitely see the bug. Iteration requires a shallow copy of the dictionary's keys. https://trac.torproject.org/projects/tor/ticket/30882

Controller's get_conf_map concurrently modified dictionary
b5aecb74 · Damian Johnson · 1a11b0cd · b5aecb74
Commit b5aecb74 authored Jun 14, 2019 by Damian Johnson
--- a/stem/control.py
+++ b/stem/control.py
@@ -2361,7 +2361,7 @@ class Controller(BaseController):
      # entries since the user didn't request those by their key, so we can't
      # be sure what they wanted.

-      for key in reply:
+      for key in list(reply):
        if not key.lower() in MAPPED_CONFIG_KEYS.values():
          user_expected_key = _case_insensitive_lookup(params, key, key)