Skip to content

Add GitHub OAuth App setup hint to docker-compose page#6643

Merged
qwerty287 merged 2 commits into
woodpecker-ci:mainfrom
mateuszkamola:docs/docker-compose-github-oauth-hint
May 26, 2026
Merged

Add GitHub OAuth App setup hint to docker-compose page#6643
qwerty287 merged 2 commits into
woodpecker-ci:mainfrom
mateuszkamola:docs/docker-compose-github-oauth-hint

Conversation

@mateuszkamola

Copy link
Copy Markdown
Contributor

The docker-compose page uses GitHub as its example forge but doesn't mention that a GitHub OAuth App needs to be registered beforehand, or what the callback URL should be. This adds a note pointing to the GitHub forge docs, and a hint that WOODPECKER_HOST is used as the base for the /authorize callback URL.

@woodpecker-bot

woodpecker-bot commented May 25, 2026

Copy link
Copy Markdown
Contributor

Surge PR preview deployment was removed

@qwerty287

Copy link
Copy Markdown
Contributor

Thanks, but you don't necessarily need github. Yes, that's the example, but you can register fir a different forge as well. It would be nice if this could be more generic and covers all forges 🙃

@mateuszkamola

Copy link
Copy Markdown
Contributor Author

Thanks, but you don't necessarily need github. Yes, that's the example, but you can register fir a different forge as well. It would be nice if this could be more generic and covers all forges 🙃

I agree. I dropped mentioning Github specifically, left only info about registering OAuth App with the chosen forge first and to take a look at the forge docs. I think with that I would save few hours on setting up woodpecker the first time :)

@qwerty287 qwerty287 merged commit 4e03df0 into woodpecker-ci:main May 26, 2026
6 of 7 checks passed
@woodpecker-bot woodpecker-bot mentioned this pull request May 25, 2026
1 task
@6543 6543 added the documentation docu & docs label May 26, 2026
@6543 6543 changed the title docs: add GitHub OAuth App setup hint to docker-compose page Add GitHub OAuth App setup hint to docker-compose page May 26, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation docu & docs

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants