-
Notifications
You must be signed in to change notification settings - Fork 24.7k
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 OIDC test fixture #89526
Update OIDC test fixture #89526
Conversation
Pinging @elastic/es-security (Team:Security) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
This commit updates the OIDC test fixture to use the latest demo docker image from https://connect2id.com. As part of this change some of the docker image demo login has changed from prior versions and the test code is updated accordingly. fixes elastic#89477 reverts elastic#89478 / e949dff.
This commit updates the OIDC test fixture to use the latest demo docker image from https://connect2id.com. As part of this change some of the docker image demo login has changed from prior versions and the test code is updated accordingly. fixes elastic#89477 reverts elastic#89478 / e949dff.
💔 Backport failed
You can use sqren/backport to manually backport by running |
This commit updates the OIDC test fixture to use the latest demo docker image from https://connect2id.com. As part of this change some of the docker image demo login has changed from prior versions and the test code is updated accordingly. fixes #89477 reverts #89478 / e949dff.
This commit updates the OIDC test fixture to use the latest demo docker image from https://connect2id.com. As part of this change some of the docker image demo login has changed from prior versions and the test code is updated accordingly. fixes #89477 reverts #89478 / e949dff.
Thanks, Jake! It's worth noting that this issue still exists since we still aren't using a docker image that provides an |
This commit updates the OIDC test fixture to use the latest demo docker image from https://connect2id.com. As part of this change some of the docker image demo login has changed from prior versions and the test code is updated accordingly. fixes elastic#89477 reverts elastic#89478 / e949dff.
This commit updates the OIDC test fixture to use the latest demo docker image from https://connect2id.com. As part of this change some of the docker image demo login has changed from prior versions and the test code is updated accordingly. fixes #89477 reverts #89478 / e949dff.
This commit updates the c2id docker image to the latest released version. This commit also introduces a multi-stage build with the openjdk image that supports ARM architectures. So our oidc tests now supports multiple archtectures. related: elastic#89526 part of: elastic#71378
This commit updates the OIDC test fixture to use the latest demo docker image
from https://connect2id.com. As part of this change some of the docker image demo
login has changed from prior versions and the test code is updated accordingly.
fixes #89477
reverts #89478 / e949dff.
To figure out the required changes I setup a local proxy to watch traffic and manually executed some of the workflows and adjusted the test setup accordingly.
When run as a suite of tests depending on order some tests would fail... I found that setting op.authz.alwaysPromptForConsent=true and op.authz.alwaysPromptForAuth=true allows for the tests to pass consistently.
Buildscan with only these tests: https://gradle-enterprise.elastic.co/s/xnysxq5b2m6za