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

Add note about Gitlab & Gitea internal connections to docs #711

Merged
merged 3 commits into from
Jan 27, 2022

Conversation

jdoubleu
Copy link
Contributor

This PR adds a little note to the docs to enable internal connections when the Woodpecker CI server and GitLab are run on the same host. This caused the webhooks to fail.

[..] similar thing applys to newest gitea too

@6543 also mentioned this might apply to other VCS (e.g. Gitea). Should we check for all of them before merging this PR?

Previous monologue

Right now I cannot sync any repo from GitLab with WP.
I get an error and the request (on WP) logs:

POST https://gitlab.example.com/api/v4/projects/192/hooks: 422 {error: Invalid url given}

At least this route seems to exist: https://docs.gitlab.com/ee/api/projects.html#add-project-hook

The message "Invalid url given" probably refers to the message body and not the request (URL) itself.
HTTP 422 is for "Unprocessable".

Okay, this fixed it: https://gitlab.com/gitlab-org/gitlab/-/issues/25867#note_215577922
Probably because I was running the Woodpecker Server and the GitLab Server on the same host.

@6543
Copy link
Member

6543 commented Jan 20, 2022

to be specific go-gitea/gitea#17482 - gitea >= v1.16.0 do need extra settings if you like to run it on the same host

@6543 6543 added the documentation docu & docs label Jan 20, 2022
@6543
Copy link
Member

6543 commented Jan 20, 2022

@jdoubleu would be nice if you could add it to the docs too. thanks!

@jdoubleu
Copy link
Contributor Author

I'll look into it. However, I might not be able to test & verify this, just gonna search for the settings in Gitea and add a note about it.

@6543
Copy link
Member

6543 commented Jan 20, 2022

Oh you dont have to verify just add it into docs, i can ensure you it is that way ...

https://docs.gitea.io/en-us/config-cheat-sheet/#webhook-webhook

@6543 6543 added this to the 0.15.0 milestone Jan 25, 2022
@6543 6543 changed the title add note about Gitlab internal connections to docs Add note about Gitlab & Gitea internal connections to docs Jan 27, 2022
@6543 6543 merged commit 48089f5 into woodpecker-ci:master Jan 27, 2022
@6543
Copy link
Member

6543 commented Jan 27, 2022

thanks :)

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.

2 participants