Skip to content

Fix file permissions#24561

Merged
willdurand merged 1 commit intomasterfrom
file-perms
Mar 5, 2026
Merged

Fix file permissions#24561
willdurand merged 1 commit intomasterfrom
file-perms

Conversation

@willdurand
Copy link
Copy Markdown
Member

Fixes mozilla/addons#16102


With this PR, we do not have any regular file with the wrong permissions:

$ find . -type f -not -perm 644 | grep -v -e ".sh$" -e "/scripts/" -e "manage.py" -e "/locale/" | wc -l
  0

FTR these files have different permissions on purpose:

$ find . -type f -not -perm 644
./docker/entrypoint.sh
./locale/generate_category_po_files.py
./locale/stats-po.sh
./docs/build-github.zsh
./scripts/check_debian_packages.sh
./scripts/update_assets.py
./scripts/install_deps.py
./scripts/run_l10n_extraction.sh
./scripts/generate_build.py
./scripts/setup.py
./scripts/health_check.py
./scripts/compile_locales.py
./scripts/check_pip_packages.sh
./scripts/health_check_blocks.py
./scripts/sync_host_files.py
./manage.py

@willdurand willdurand requested review from a team and diox and removed request for a team March 4, 2026 12:42
@willdurand willdurand added this to the 2026.03.19 milestone Mar 4, 2026
@willdurand willdurand removed this from the 2026.03.19 milestone Mar 4, 2026
@willdurand willdurand merged commit 82ff62e into master Mar 5, 2026
44 checks passed
@willdurand willdurand deleted the file-perms branch March 5, 2026 10:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: Wrong file permissions in the addons-server repo

2 participants