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

[Self-hosted] UnauthorizedGuestAccess error after updating from 24.8.1 to 24.9.3/24.10.1/24.11.1 #536

Open
nikosatwork opened this issue Oct 11, 2024 · 10 comments
Assignees
Labels
Area: General UX Issues with general UX bug For issues and unexpected behaviors High High Priority Issues (to be fixed within 2 sprints) Waiting for: Product Owner

Comments

@nikosatwork
Copy link

nikosatwork commented Oct 11, 2024

Describe the bug
Not able to 'Login via GitHub Enterprise' after upgrading to 24.9.3, 24.10.1 or 24.11.1 from 24.8.1

Exception
UnauthorizedGuestAccess: Unauthorized

in graphql_api/types/query/query.py#L51

Environment (please complete the following information):

  • Browser: Any
  • System: Any

To Reproduce
Steps to reproduce the behavior:

  1. Upgrade to 24.9.3, 24.10.1 or 24.11.1
  2. Hit 'Login via GitHub Enterprise' in the home screen
  3. Always get redirected back to the home screen & not able to proceed

Expected behavior
Login successfuly

Additional context

Note we have tried setting both guest_access: true & guest_access: false

UPDATE: This is happening only when guest_access: false

This is urgent as we need to update to the newer version because of #523

@covecod covecod bot moved this to Waiting for: Product Owner in GitHub Issues with 👀 Oct 11, 2024
@nikosatwork
Copy link
Author

cc @drazisil-codecov

@drazisil-codecov
Copy link

@nikosatwork ,

I don'think the settings system that well, but is IS_ENTERPRISE set in the environment anywhere? What if you try adding it?

@drazisil-codecov drazisil-codecov moved this from Waiting for: Product Owner to Waiting for: Community in GitHub Issues with 👀 Oct 15, 2024
@nikosatwork
Copy link
Author

I think setting guest_access: true fixed it, but I don't understand the logic

@covecod covecod bot moved this from Waiting for: Community to Waiting for: Product Owner in GitHub Issues with 👀 Oct 22, 2024
@drazisil-codecov
Copy link

I think setting guest_access: true fixed it, but I don't understand the logic

This is the feature that was created for a company who had "public" projects on their private instance. It's what controls if you can see projects on Codecov when you are not signed in.

It was probably set to false by default, since that was the setting it needed to be before when the company was on hosted cloud.

(Being vague due to not sure if the company has legally given us permission to name them as a customer, and this is open comms)

@drazisil-codecov drazisil-codecov added bug For issues and unexpected behaviors High High Priority Issues (to be fixed within 2 sprints) Area: General UX Issues with general UX labels Oct 22, 2024
@drazisil-codecov
Copy link

drazisil-codecov commented Oct 22, 2024

#523 (comment)

In 24.10.1 I get Anauthorized if just having 'Activation Status: False' and visiting a private repo in Codecov

Image

Which should be

Image

Am I missing something perhaps?
guest_access: true by the way

Thanks!

@eliatcodecov
Copy link
Contributor

Assigning to @RulaKhaled to investigate to see if we can determine if any underlying bug is present and fix it. Can coordinate with @drazisil-codecov as she has context on this issue.

@RulaKhaled
Copy link

RulaKhaled commented Nov 19, 2024

@nikosatwork After upgrading to the latest version, is this still an issue?

Not the activation issue, but the login issue

@spalmurray-codecov
Copy link

This could be related to a change we made on API recently - cc @suejung-sentry

@spalmurray-codecov
Copy link

Oh wait that change (codecov/codecov-api#910) hasn't been merged yet

@nikosatwork
Copy link
Author

Hi @RulaKhaled

@nikosatwork After upgrading to the latest version, is this still an issue?

Still the same issue on version 24.11.1 when guest_access: false

@nikosatwork nikosatwork changed the title [Self-hosted] UnauthorizedGuestAccess error after updating from 24.8.1 to 24.9.3/24.10.1 [Self-hosted] UnauthorizedGuestAccess error after updating from 24.8.1 to 24.9.3/24.10.1/24.11.1 Nov 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area: General UX Issues with general UX bug For issues and unexpected behaviors High High Priority Issues (to be fixed within 2 sprints) Waiting for: Product Owner
Projects
Status: Waiting for: Product Owner
Development

No branches or pull requests

5 participants