Cookie based Sticky Sessions for HAProxy with TLS reencrypt#751
Cookie based Sticky Sessions for HAProxy with TLS reencrypt#751pruivo wants to merge 8 commits intokeycloak:mainfrom
Conversation
Signed-off-by: Tomas Kyjovsky <[email protected]>
…backend section and adding "host by by_port for" params Signed-off-by: Tomas Kyjovsky <[email protected]>
Signed-off-by: Tomas Kyjovsky <[email protected]>
Signed-off-by: Tomas Kyjovsky <[email protected]>
Signed-off-by: Tomas Kyjovsky <[email protected]>
Closes keycloak#750 Signed-off-by: Pedro Ruivo <[email protected]>
ahus1
left a comment
There was a problem hiding this comment.
Thank you for this PR. It doesn't look that ugly, it looks quite manageable. So we would include it by default, at least for the HAProxy setup?
See below for some nitbits.
Co-authored-by: Alexander Schwartz <[email protected]> Signed-off-by: Pedro Ruivo <[email protected]>
It can be included sure, but it looks messy and isn't maintainable. You can add a server dynamically, but not add new mappings, which may be a problem when scaling up and down the cluster. |
OK, I didn't know about that one. Maybe add it to the README so I won't forget. Thanks! |
Signed-off-by: Pedro Ruivo <[email protected]>
Closes #750
Depends on #749