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

Update code to reflect the new EGI checkin error message(Master) #522

Conversation

Sae126V
Copy link
Contributor

@Sae126V Sae126V commented Sep 6, 2024

Resolves #498
GT-469

If a user authenticates with EGI CheckIn, but does not have the correct permission to view gocdb, they are given a basic error message.

This error message references an older entitlement / the EGI wiki. This should be updated to the newer entitlement and update the references.

@Sae126V Sae126V requested a review from a team as a code owner September 6, 2024 14:11
@Sae126V Sae126V changed the title Update code to reflect the new EGI checkin error message Update code to reflect the new EGI checkin error message(MASTER) Sep 6, 2024
@Sae126V Sae126V changed the title Update code to reflect the new EGI checkin error message(MASTER) Update code to reflect the new EGI checkin error message(Dev) Sep 6, 2024
@Sae126V Sae126V changed the title Update code to reflect the new EGI checkin error message(Dev) Update code to reflect the new EGI checkin error message(Master) Sep 6, 2024
@Sae126V Sae126V force-pushed the GT-469-Update-to-CheckIn-related-entitlement-error-Dev-fork branch 2 times, most recently from 3bd922e to 8dbcafd Compare September 6, 2024 14:23
@Sae126V Sae126V marked this pull request as draft September 6, 2024 14:24
@Sae126V Sae126V force-pushed the GT-469-Update-to-CheckIn-related-entitlement-error-Dev-fork branch from 8dbcafd to 1a810cd Compare September 6, 2024 14:25
@Sae126V Sae126V force-pushed the GT-469-Update-to-CheckIn-related-entitlement-error-Dev-fork branch 2 times, most recently from 2cb57c9 to b0fb206 Compare September 10, 2024 13:18
@Sae126V Sae126V force-pushed the GT-469-Update-to-CheckIn-related-entitlement-error-Dev-fork branch from b0fb206 to 16bcadb Compare September 10, 2024 14:35
@Sae126V Sae126V marked this pull request as ready for review September 11, 2024 07:11
@gregcorbett
Copy link
Member

Note to Future Greg: It's hard to test this, as you don't have a suitable EGI CheckIn account to test every code path. Manual testing on a dev machine (by forcing the relevant if statements to evaluate to true or false as appropriate) would indicate it works as expected.

@gregcorbett
Copy link
Member

LGTM, Subject to final role out of 5.12.1-rc.1 into production) I'll approve and merge the PRs (this and #523) tomorrow - this is one of those things we can only really test by putting it in production.

Copy link
Member

@gregcorbett gregcorbett left a comment

Choose a reason for hiding this comment

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

LGTM

@gregcorbett
Copy link
Member

The MariaDB 10.3 checks do seem to have passed: https://github.com/GOCDB/gocdb/actions/runs/10900091938/job/30246944295

@gregcorbett gregcorbett merged commit 2d546fe into GOCDB:master Sep 17, 2024
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants