-
Notifications
You must be signed in to change notification settings - Fork 63
feat: introspection_endpoint #256
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
Conversation
Ahhhhh you're the best...! Ok, I'm a little out of my element, but I'll try to sort out how to build and test this. I expect the approach I'd take is to create a Does that sound like an OK approach? I'm seeing that you use something called "jib" to build the existing containers (which doesn't use a docker daemon?) and so it seems I'm going a little off-road, but wondering: would this still be useful as a sample |
To be clear, this is how I'm thinking I'd aim to get things to work: version: '3.7'
services:
your_app:
build: .
ports:
- 8080:8080
mock-oauth2-server:
# Not using this.
#image: ghcr.io/navikt/mock-oauth2-server:$MOCK_OAUTH2_SERVER_VERSION
# And instead using this (with example file that might also become available upstream?)
build: ./support/mock-auth.Dockerfile
environment:
MOCK_AUTH_REPO_SLUG: navikt/mock-oauth2-server
MOCK_AUTH_REPO_BRANCH: introspection_endpoint
ports:
- 8080:8080
hostname: host.docker.internal |
a bit hakky but; by comment this out in
You can run |
I don't yet have a simple way to have the whole team using it (since we rely on docker and our dev environment doesn't support building java projects for everyone easily), but I've built it locally and it works for me! |
@tommytroen 🚤? |
* removed unnecessary null check on form parameters * changed data type for exp, iat, nbf to long
@ybelMekk Looks good! Added a few minor changes. |
Thanks so so much @tommytroen and @ybelMekk 🙏 |
No description provided.