CopyBot/Red-DiscordBot
1
0
Fork 0
mirror of https://github.com/Cog-Creators/Red-DiscordBot.git synced 2025-11-07 11:48:55 -05:00
Code Issues Packages Projects Releases Wiki Activity

[Downloader] Disable all git auth prompts on clone/pull (#3159)

Browse Source 
* fix(downloader): disable all git auth prompts on clone/pull

* chore(changelog): add towncrier entry
This commit is contained in:
jack1142 2019-12-08 01:54:49 +01:00 committed by Michael H
parent 672050727f
commit 0f62614055
2 changed files with 13 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
changelog.d/downloader/3159.bugfix.rst Normal file
View File

@ -0,0 +1 @@
Disable all git auth prompts when adding/updating repo with Downloader.

15
redbot/cogs/downloader/repo_manager.py
View File

@ -87,13 +87,21 @@ class ProcessFormatter(Formatter):
class Repo(RepoJSONMixin):
GIT_CLONE = "git clone --recurse-submodules -b {branch} {url} {folder}"
GIT_CLONE_NO_BRANCH = "git clone --recurse-submodules {url} {folder}"
GIT_CLONE = (
"git clone -c credential.helper= -c core.askpass="
" --recurse-submodules -b {branch} {url} {folder}"
)
GIT_CLONE_NO_BRANCH = (
"git -c credential.helper= -c core.askpass= clone --recurse-submodules {url} {folder}"
)
GIT_CURRENT_BRANCH = "git -C {path} symbolic-ref --short HEAD"
GIT_CURRENT_COMMIT = "git -C {path} rev-parse HEAD"
GIT_LATEST_COMMIT = "git -C {path} rev-parse {branch}"
GIT_HARD_RESET = "git -C {path} reset --hard origin/{branch} -q"
GIT_PULL = "git -C {path} pull --recurse-submodules -q --ff-only"
GIT_PULL = (
"git -c credential.helper= -c core.askpass= -C {path}"
" pull --recurse-submodules -q --ff-only"
)
GIT_DIFF_FILE_STATUS = (
"git -C {path} diff-tree --no-commit-id --name-status"
" -r -z --line-prefix='\t' {old_rev} {new_rev}"
@ -513,6 +521,7 @@ class Repo(RepoJSONMixin):
"""
env = os.environ.copy()
env["GIT_TERMINAL_PROMPT"] = "0"
env.pop("GIT_ASKPASS", None)
# attempt to force all output to plain ascii english
# some methods that parse output may expect it
# according to gettext manual both variables have to be set: