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

Github Integration Broken #3117

Open
geofferyj opened this issue Jun 7, 2024 · 7 comments
Open

Github Integration Broken #3117

geofferyj opened this issue Jun 7, 2024 · 7 comments

Comments

@geofferyj
Copy link

Self-Hosted Version

24.5.1

CPU Architecture

x86_64

Docker Version

26.1.3

Docker Compose Version

2.27.0

Steps to Reproduce

follow the instructions to add github integration from here

Expected Result

after following the instructions, I expect that on the integrations page, when I click on add integration it should begin the integration flow

Actual Result

after setting all the configurations,
re-running ./install.sh

in the integrations page, when Add Installation button is clicked, it just redirects me to the home page of my self hosted sentry. and I can't login again as the login page is missing.

Screen.Recording.2024-06-07.at.04.28.38.mov
Screen.Recording.2024-06-07.at.04.32.18.mov

Please take a look at the videos above.

Event ID

No response

@aldy505
Copy link
Collaborator

aldy505 commented Jun 10, 2024

Hello, the GitHub integration is not broken. The docs is just misleading for now. There is a PR waiting to be merged here: https://github.com/getsentry/develop/pull/1235/files -- you should follow that one, add that additional config. I've pinged the related codeowners yet I don't get any response either, so we'll have to make do for now.

@geofferyj
Copy link
Author

I don't know if I am doing something wrong but the integration still didn't work, SSO works now though.

when registration is enabled, signin with email and password is unavailable
I thought the integration flow will be the same as hosted sentry, clicking on add integration just takes you to the github app then back. please refer to the video below (PS. No detrimental info in the video)

Screen.Recording.2024-06-10.at.11.07.47.mov

@hubertdeng123
Copy link
Member

I've taken a look at the docs PR up above. The integration flow is not the same as SaaS Sentry here, have you confirmed that you have set the appropriate callback urls?

@geofferyj
Copy link
Author

yes I have confirmed that the URLs are correct.

@hubertdeng123
Copy link
Member

Based on the behavior, it seems like perhaps there may be an error there somewhere, since it is redirecting you back without completing installation. What is the url you are being redirected to after clicking install and authorize?

@geofferyj
Copy link
Author

it takes me back to home

here are the relevant URLs configured

public link: https://github.com/apps/penilabs-monitoring
home-url:https://monitoring.penilabs.com

callback URLs: - https://monitoring.penilabs.com/auth/sso/
- https://monitoring.penilabs.com/extensions/github/setup/

webhook-url: https://monitoring.penilabs.com/extensions/github/webhook/

Update: if I remove https://monitoring.penilabs.com/auth/sso/, integration works but sso stops working
then I add it back and everything seems to be working how

does the order of the callbacks matter?

@azaslavsky
Copy link
Contributor

Update: if I remove https://monitoring.penilabs.com/auth/sso/, integration works but sso stops working
then I add it back and everything seems to be working how

So it works when you reverse the order of the callback URLs? I do not believe order should matter, but I am also not super well-versed in the specifics of this extension or the GitHub API.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Waiting for: Community
Status: No status
Development

No branches or pull requests

4 participants