Skip to content
This repository has been archived by the owner on Mar 16, 2021. It is now read-only.

Add generic oidc #132

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

jovandeginste
Copy link

No description provided.

Signed-off-by: Jo Vandeginste <[email protected]>
Eg. when using Keycloak

Signed-off-by: Jo Vandeginste <[email protected]>
@memory
Copy link
Collaborator

memory commented Sep 9, 2019

Appears fine on first blush. @mxschmitt any thoughts?

@mxschmitt
Copy link
Owner

@jovandeginste if you fix the tests, we can merge it
cc @memory

@jovandeginste
Copy link
Author

@mxschmitt I can't reproduce the failing test:

$ go test -v ./...
=== RUN   TestInitShortener
[...]
--- PASS: TestCheckforPrivateKey (0.00s)
PASS
ok  	github.com/mxschmitt/golang-url-shortener/internal/util	(cached)
$ echo $?
0

nmasse-itix added a commit to nmasse-itix/golang-url-shortener that referenced this pull request Jan 4, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants