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

Docs: securing_apps/topics/oidc/java/installed-adapter.adoc #29063

Closed
1 of 2 tasks
ctrworthin opened this issue Apr 24, 2024 · 1 comment
Closed
1 of 2 tasks

Docs: securing_apps/topics/oidc/java/installed-adapter.adoc #29063

ctrworthin opened this issue Apr 24, 2024 · 1 comment

Comments

@ctrworthin
Copy link

Before reporting an issue

  • I have read and understood the above terms for submitting issues, and I understand that my issue may be closed without action if I do not follow them.

Area

docs

Describe the bug

File: securing_apps/topics/oidc/java/installed-adapter.adoc

The documentation for CLI / Desktop applications mention both desktop and manual  variants in the second paragraph, however the manual variant has no further elaboration beyond the subsequent sentence, which states that difference between the two:

The KeycloakInstalled adapter supports a desktop and a manual variant. The desktop variant uses the system browser to gather the user credentials. The manual variant reads the user credentials from STDIN.

Version

24.0.2

Regression

  • The issue is a regression

Expected behavior

Proportional documentation of the manual variant.

Actual behavior

Only a brief mention of the manual variant, with no further explanation beyond it's primary distinction from the desktop variant. Other than the second paragraph in which it is mentioned, it reads as there is no alternative to authenticating through a browser.

How to Reproduce?

Anything else?

No response

@rmartinc
Copy link
Contributor

In the manual option the user should do the operation in the browser manually. Then copy the code and pass it to the java using a Reader. I suppose it was not documented because it's not very useful. Check the code here.

Nevertheless the adapters are going to be removed in teh next version of keycloak. The KeycloakInstalled was indeed removed just a few days ago from main (see #28790). So no updates in documentation will be done as the KeycloakInstalled code and documentation are already not present in the project.

Closing this as not planned.

@rmartinc rmartinc closed this as not planned Won't fix, can't repro, duplicate, stale Apr 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants