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

GitHub Support#27

Closed
mrauer wants to merge 2 commits into
gini:masterfrom
mrauer:feature/mrauer/github-support
Closed

GitHub Support#27
mrauer wants to merge 2 commits into
gini:masterfrom
mrauer:feature/mrauer/github-support

Conversation

@mrauer

@mrauer mrauer commented Oct 18, 2019

Copy link
Copy Markdown

No description provided.

@tlvenn

tlvenn commented Nov 2, 2019

Copy link
Copy Markdown

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

tlvenn commented Nov 4, 2019

Copy link
Copy Markdown

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

dkerwin commented Nov 4, 2019

Copy link
Copy Markdown
Contributor

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

dkerwin commented Nov 7, 2019

Copy link
Copy Markdown
Contributor

Hey @mrauer,

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

@dkerwin

dkerwin commented Nov 24, 2019

Copy link
Copy Markdown
Contributor

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

tlvenn commented Nov 24, 2019

Copy link
Copy Markdown

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

dkerwin commented Nov 25, 2019

Copy link
Copy Markdown
Contributor

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