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.

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

Avoid clashes on .git folders during git-extraction. #10632

Merged
merged 4 commits into from
Feb 8, 2019

Conversation

EnTeQuAk
Copy link
Contributor

@EnTeQuAk EnTeQuAk commented Feb 8, 2019

Files are now extracted into an extracted/ folder to have a 100% clean
surface and avoid any clashes.

In addition to that, we are now appending a random postfix to .git
folders because for security reasons git doesn't allow .git folders in
any subdirectory (e.g for git commit-hooks etc)

Aside from that, this renames the package folder to addon for a better understanding of its usage.

Fixes mozilla/addons#6363

Files are now extracted into a `extracted/` folder to have a 100% clean
surface and avoid any clashes.

In addition to that we are now appending a random postfix to .git
folders because for security reasons git doesn't allow .git folders in
any sub directory (e.g for git commit-hooks etc)

Fixes #10562
@EnTeQuAk EnTeQuAk requested a review from eviljeff February 8, 2019 08:04
@EnTeQuAk EnTeQuAk merged commit 29c91e4 into master Feb 8, 2019
@EnTeQuAk EnTeQuAk deleted the 10562-git-extract-subdir branch February 8, 2019 09:07
MelissaAutumn pushed a commit to thunderbird/addons-server that referenced this pull request Aug 25, 2024
Files are now extracted into an `extracted/` folder to have a 100% clean
surface and avoid any clashes.

In addition to that, we are now appending a random postfix to .git
folders because for security reasons git doesn't allow .git folders in
any subdirectory (e.g for git commit-hooks etc)

Aside from that, this renames the `package` folder to `addon` for a better understanding of its usage. 

Fixes #10562
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.

git-storage extraction runs into "File exists" error
2 participants