8000 Proxy: TLS reencryt and passthrough docs by pruivo · Pull Request #48037 · keycloak/keycloak · GitHub
[go: up one dir, main page]

Skip to content

Proxy: TLS reencryt and passthrough docs - #48037

Merged
ahus1 merged 3 commits into
keycloak:mainfrom
pruivo:t_48002
Apr 17, 2026
Merged

ahus1 merged 3 commits into
keycloak:mainfrom
pruivo:t_48002

Conversation

@pruivo
@pruivo pruivo commented Apr 14, 2026
Copy link
Copy Markdown
Member

Closes #48002

Closes keycloak#48002

Signed-off-by: Pedro Ruivo <1492066+pruivo@users.noreply.github.com>
@shawkins
Copy link
Copy Markdown
Contributor

Also relates to some expansion / cleanup of docs being done for the operator #47565

@pruivo
pruivo commented Apr 14, 2026
Copy link
Copy Markdown
Member Author

@shawkins, thank you for sharing it. We haven't discussed any operator docs/code changes in our meetings. I'll bring this subject up in the next meeting.

Signed-off-by: Pedro Ruivo <1492066+pruivo@users.noreply.github.com>
@pruivo
pruivo marked this pull request as ready for review April 15, 2026 14:40
@pruivo
pruivo requested review from a team as code owners April 15, 2026 14:40
@pruivo
pruivo commented Apr 15, 2026
Copy link
Copy Markdown
Member Author

I think the document will look better with 3 sections

  • introduction to the theme (this PR)
  • Keycloak with passthrough (strongly recommend PROXY protocol)
  • Keycloak with reencrypt (remaining documentation - headers, exposed paths, client certificate providers)

I've confirmed that enabling the PROXY protocol does not affect the management endpoint, and HaProxy can check the Keycloak health. Both PROXYv1 and PROXYv2 work.

cc @ahus1 @shawkins

@ahus1
ahus1 self-requested a review April 15, 2026 16:34
Signed-off-by: Alexander Schwartz <alexander.schwartz@ibm.com>
@ahus1
ahus1 commented Apr 16, 2026
Copy link
Copy Markdown
Member

@pruivo - thank you for the docs. I've added some notes about the PROXY protocol that is neither encrypted nor signed, and the IP spoofing for the "trusted" proxies.

I also added a TODO about the mTLS between proxy and Keycloak. Maybe you can extend it based on the current state of the HAProxy blueprint?

@ahus1 ahus1 left a comment
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approving after a chat with @pruivo

@ahus1
ahus1 merged commit 69ed451 into keycloak:main Apr 17, 2026
62 checks passed
@pruivo
pruivo deleted the t_48002 branch April 17, 2026 12:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

TLS re-encrypt vs passthrough incl. security pros/cons and best practices

3 participants

0