Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix issue 12471 #25122

Merged
merged 7 commits into from
May 19, 2020
Merged

Fix issue 12471 #25122

Changes from all commits
Commits
Show all changes
7 commits
Select commit Hold shift + click to select a range
4aafb9a
Adding Audio and Visual converter for ffmpeg.
ankenyr Jun 28, 2017
795ea73
Merge pull request #1 from ytdl-org/master
ankenyr May 4, 2020
7c58b51
Adding chmod step to write_json_file to keep permissions in line wit…
ankenyr May 4, 2020
177a9ad
Revert "Adding Audio and Visual converter for ffmpeg."
ankenyr May 4, 2020
bf4361a
Adding additional permissions and respecting umask
ankenyr May 4, 2020
ac6256f
Fixing white space, adding try/except block, and specifying permissio…
ankenyr May 19, 2020
86a4153
Removing stat as unused import.
ankenyr May 19, 2020
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 6 additions & 0 deletions youtube_dl/utils.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1835,6 +1835,12 @@ def write_json_file(obj, fn):
os.unlink(fn)
except OSError:
pass
try:
mask = os.umask(0)
os.umask(mask)
os.chmod(tf.name, 0o666 & ~mask)
except OSError:
ankenyr marked this conversation as resolved.
Show resolved Hide resolved
pass
os.rename(tf.name, fn)
except Exception:
try:
Expand Down