Skip to content
This repository was archived by the owner on May 15, 2025. It is now read-only.

Conversation

mrauer
Copy link

@mrauer mrauer commented Oct 18, 2019

No description provided.

@tlvenn
Copy link

tlvenn commented Nov 2, 2019

It might be a good idea to update the doc and other parts of the code to add Github support in addition to Google and Azure @mrauer.

@tlvenn
Copy link

tlvenn commented Nov 4, 2019

Hi @mrauer,

Did you actually test that PR ?
On my side i do end up with an error:

Failed to write k8s config: Failed to parse token: square/go-jose: compact JWS format must have three parts

And now that I have looked into it, it seems to me that Github does not actually implement OIDC spec so it seems impossible to use it as a provider...

@dkerwin
Copy link
Contributor

dkerwin commented Nov 4, 2019

Thank you for this PR. I'll have a look but I agree with @tlvenn that it may not work

@dkerwin dkerwin mentioned this pull request Nov 7, 2019
@dkerwin
Copy link
Contributor

dkerwin commented Nov 7, 2019

Hey @mrauer,

fyi: once #30 is merged it becomes very easy to add new providers to dexter

@dkerwin
Copy link
Contributor

dkerwin commented Nov 24, 2019

Not that #30 is merged it is very easy to extend dexter with new providers. Would love to see you implement the first github provider ;-)

@tlvenn
Copy link

tlvenn commented Nov 24, 2019

It's great that the PR was merged but keep in mind that until Github decides to support OIDC, it's not really possible to support it.

@dkerwin
Copy link
Contributor

dkerwin commented Nov 25, 2019

You are right. Given that it's not possible with Github without something like dex in the middle I'll close this PR

@dkerwin dkerwin closed this Nov 25, 2019
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