Skip to content
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

Generate keys commands #108

Merged
merged 1 commit into from
Apr 9, 2024
Merged

Generate keys commands #108

merged 1 commit into from
Apr 9, 2024

Conversation

fbourigault
Copy link
Contributor

@fbourigault fbourigault commented Oct 20, 2022

Here is my proposal to implement #106.

The first commit is an import of the command and the test from LexikJWTAuthenticationBundle to make review of change easier.

To do

  • Make $algorithm a command option as LeagueOAuth2ServerBundle doesn't have such configuration option.
  • Register the command as a service.

@fbourigault
Copy link
Contributor Author

Can someone approve the workflows so I can get feedback from the whole matrix?

@chalasr
Copy link
Member

chalasr commented Aug 7, 2023

My apologies for the delay Fabien, my OSS time was very limited lately I totally forgot to get back on this PR.
Could you please rebase?

@stefantalen
Copy link

ping @fbourigault @chalasr

I think this still is a valuable addition, any updates on this?

composer.json Outdated Show resolved Hide resolved
@fbourigault
Copy link
Contributor Author

I've rebased this PR and applied changes from LexikJWTAuthenticationBundle.

@chalasr
Copy link
Member

chalasr commented Apr 6, 2024

@fbourigault Apologies for the delay here. I'm wiling to move forward :) Can you please have a look at failing tests?

@chalasr
Copy link
Member

chalasr commented Apr 9, 2024

It took time, but here we go, this is in now. Thank you very much @fbourigault.

@chalasr chalasr merged commit 58d4b11 into thephpleague:master Apr 9, 2024
21 checks passed
@fbourigault fbourigault deleted the generate-keys-commands branch April 10, 2024 07:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants