ch-lukas commented on PR #3569:
URL:
https://github.com/apache/incubator-kie-tools/pull/3569#issuecomment-4442875804
Thanks @thiagoelg, the checkbox makes sense.
To keep scope tight I plan to:
- Add the checkbox to the Kubernetes auth session wizard only. OpenShift
stays always-proxied. OpenShift clusters are basically never local, so
the toggle would never be unchecked there and would just add surface
area.
- Treat legacy stored sessions (no `useCorsProxy` field) as unchecked, so
existing setups keep working unchanged. New sessions get the wizard.
Open question: for new sessions, should the wizard default to checked or
unchecked? Your screenshot shows checked, happy to go with that if it's
deliberate.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
