Skip to content
This repository was archived by the owner on May 6, 2024. It is now read-only.

fix: Use repo owner user for git operations - #6717

Merged
nadeemshahzad merged 1 commit into
masterfrom
nadeem/fix_git_clone
Apr 13, 2022
Merged

fix: Use repo owner user for git operations #6717
nadeemshahzad merged 1 commit into
masterfrom
nadeem/fix_git_clone

Conversation

@nadeemshahzad

Copy link
Copy Markdown
Contributor

Git has released a new version ( https://github.blog/2022-04-12-git-security-vulnerability-announced/ ) to address a security Vulnerbiltuy and This version changes Git’s behavior when looking for a top-level .git directory to stop when its directory traversal changes ownership from the current user. This PR will fix the issue of git operations in the git clone role.

Configuration Pull Request

Make sure that the following steps are done before merging:

  • A SRE team member has approved the PR if it is code shared across multiple services and you don't own all of the services.
  • Are you adding any new default values that need to be overridden when this change goes live? If so:
    • Update the appropriate internal repo (be sure to update for all our environments)
    • If you are updating a secure value rather than an internal one, file a SRE ticket with details.
    • Add an entry to the CHANGELOG.
  • If you are making a complicated change, have you performed the proper testing specified on the Ops Ansible Testing Checklist? Adding a new variable does not require the full list (although testing on a sandbox is a great idea to ensure it links with your downstream code changes).
  • Think about how this change will affect Open edX operators. Have you updated the wiki page for the next Open edX release?

@nadeemshahzad
nadeemshahzad merged commit 8f5cea0 into master Apr 13, 2022
@nadeemshahzad
nadeemshahzad deleted the nadeem/fix_git_clone branch April 13, 2022 09:57
0x29a pushed a commit to open-craft/configuration that referenced this pull request May 4, 2022
…it_clone

fix: Use repo owner user for git operations
(cherry picked from commit 8f5cea0)
0x29a pushed a commit to open-craft/configuration that referenced this pull request May 28, 2022
…it_clone

fix: Use repo owner user for git operations
(cherry picked from commit 8f5cea0)
0x29a pushed a commit to open-craft/configuration that referenced this pull request May 30, 2022
…it_clone

fix: Use repo owner user for git operations
(cherry picked from commit 8f5cea0)
kaustavb12 pushed a commit to open-craft/configuration that referenced this pull request Aug 29, 2022
…it_clone

fix: Use repo owner user for git operations
(cherry picked from commit 8f5cea0)
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants