diff options
| -rw-r--r-- | paramiko/pkey.py | 3 | ||||
| -rw-r--r-- | sites/www/changelog.rst | 5 |
2 files changed, 6 insertions, 2 deletions
diff --git a/paramiko/pkey.py b/paramiko/pkey.py index e95d60ba..7f32c8a5 100644 --- a/paramiko/pkey.py +++ b/paramiko/pkey.py @@ -318,8 +318,7 @@ class PKey (object): :raises IOError: if there was an error writing the file. """ - with open(filename, 'w', o600) as f: - # grrr... the mode doesn't always take hold + with open(filename, 'w') as f: os.chmod(filename, o600) self._write_private_key(tag, f, data, password) diff --git a/sites/www/changelog.rst b/sites/www/changelog.rst index 0ceca078..7884b044 100644 --- a/sites/www/changelog.rst +++ b/sites/www/changelog.rst @@ -2,6 +2,11 @@ Changelog ========= +* :bug:`824` Fix the implementation of ``PKey.write_private_key_file`` (this + method is only publicly defined on subclasses; the fix was in the private + real implementation) so it passes the correct params to ``open()``. This bug + apparently went unnoticed and unfixed for 12 entire years. Congrats to John + Villalovos for noticing & submitting the patch! * :support:`801 (1.17+)` Skip a Unix-only test when on Windows; thanks to Gabi Davar. * :support:`792 (1.17+)` Minor updates to the README and demos; thanks to Alan |