Skip to content

gitea: support git credential manager oauth flow#800

Merged
thespad merged 2 commits into
linuxserver:masterfrom
cmruffin:master
Jul 3, 2026
Merged

gitea: support git credential manager oauth flow#800
thespad merged 2 commits into
linuxserver:masterfrom
cmruffin:master

Conversation

@cmruffin

Copy link
Copy Markdown
Contributor

linuxserver.io


  • [X ] I have read the contributing guideline and understand that I have made the correct modifications

Description

git credential manager sets up an oauth flow which requires a redirect to a localhost port to complete the oauth flow. The proxt_redirect http:// scheme$:// directive in proxy.conf causes this to be rewritten to https://. Make an exception for redirects to localhost so that the oauth login will complete.

Benefits of this PR and context

git operations using the git credential manager with gitea can successfully set up oauth tokens.

How Has This Been Tested?

Test git clone / git fetch operations that initiate an oauth login using git for windows default installation which uses git credential manager.

@LinuxServer-CI

Copy link
Copy Markdown

This pull request has been automatically marked as stale because it has not had recent activity. This might be due to missing feedback from OP. It will be closed if no further activity occurs. Thank you for your contributions.

@cmruffin

Copy link
Copy Markdown
Contributor Author

@aptalca @thespad can I get a review on this PR?

@thespad thespad left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Happy to merge this once the changelog date is updated

Comment thread gitea.subdomain.conf.sample Outdated

@cmruffin cmruffin left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

TY

@cmruffin cmruffin requested a review from thespad July 3, 2026 18:16
@LinuxServer-CI LinuxServer-CI moved this from PRs to PRs Ready For Team Review in Issue & PR Tracker Jul 3, 2026
@github-project-automation github-project-automation Bot moved this from PRs Ready For Team Review to PRs Approved in Issue & PR Tracker Jul 3, 2026
@thespad thespad merged commit 36d351e into linuxserver:master Jul 3, 2026
4 checks passed
@LinuxServer-CI LinuxServer-CI moved this from PRs Approved to Done in Issue & PR Tracker Jul 3, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Development

Successfully merging this pull request may close these issues.

3 participants