Add client_secret to kcLogin command

[Shuyinsama]

Also tested the changes to a Keycloak 9.0.1 and 10.0.2 instance

[Shuyinsama]

I've updated the PR to make the client_secret property optional. It seems that the initial code was working for some people even though I've never got this working without client_secret.

This way we can still accommodate both ways

[Shuyinsama]

@Fredx87 Do you have an idea when you will be able to review this PR? I am eagerly waiting so we can start using these changes in our project :)

[RubenGarcia]

I also request merging of this feature.

[Thanoonazad]

Hi guys I just want to ask if there is any reason why this pull requests did not be merged yet?

@qws777
@Shuyinsama
@RubenGarcia

[Shuyinsama]

@Thanoonazad I have no idea. I have ultimately implemented this code into my own project. No need for a fork or whatever. This repo is not that big and you can copy paste most of the code to your project and it will work.

[metalmini]

@Fredx87 Is there a reason this PR lingers for so long? I mean, everything is ✅ and the reviewer already approved it.

[JordiGiros]

@Fredx87 Can you merge this, please? Great addition and a lot of people waiting for it.

[jdtully]

I have been watching for movement on this for over a year, what can be done if the otirignal poster never responds? ?

…

On Fri, May 7, 2021 at 5:41 AM Michael Schouman ***@***.***> wrote: @Fredx87 <https://github.com/Fredx87> Is there a reason this PR lingers for so long? I mean, everything is ✅ and the reviewer already approved it. — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub <#15 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/ALVKJ2V2KB6QV7DA3LE4D3LTMOYT3ANCNFSM4NYN6PGA> .

-- Jeffrey Tully