starred/certbot-certbot
1
0
Fork 0
mirror of https://github.com/certbot/certbot.git synced 2026-08-04 12:21:51 +02:00
Code Issues Packages Projects Releases Wiki Activity

Use the file handle provided by mkstemp for opening the file.

Browse Source
This commit is contained in:
kevinlondon
2015-09-21 08:05:55 -07:00
parent 31af7b3a02
commit ff9f12aea6
1 changed files with 3 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files
letsencrypt/revoker.py
+3 -3
View File
@@ -288,12 +288,12 @@ class Revoker(object):
:class:`letsencrypt.revoker.Cert` :class:`letsencrypt.revoker.Cert`
""" """
_, list_path2 = tempfile.mkstemp(".tmp", "LIST") newfile_handle, list_path2 = tempfile.mkstemp(".tmp", "LIST")
idx = 0 idx = 0
with open(self.list_path, "rb") as orgfile: with open(self.list_path, "rb") as orgfile:
csvreader = csv.reader(orgfile) csvreader = csv.reader(orgfile)
with open(list_path2, "wb") as newfile: with os.fdopen(newfile_handle, "wb") as newfile:
csvwriter = csv.writer(newfile) csvwriter = csv.writer(newfile)
for row in csvreader: for row in csvreader:
@@ -308,7 +308,7 @@ class Revoker(object):
"Did not find all cert_list items to remove from LIST") "Did not find all cert_list items to remove from LIST")
shutil.copy2(list_path2, self.list_path) shutil.copy2(list_path2, self.list_path)
os.remove(list_path2) newfile.close()
def _row_to_backup(self, row): def _row_to_backup(self, row):
"""Convenience function """Convenience function