rucio / containers

Containers for Rucio
Apache License 2.0
8 stars 54 forks source link

server: explicitly request client certs for x509 endpoint #216 #218

Closed maany closed 1 year ago

maany commented 1 year ago

Fix #216 The changes in this PR work in conjunction with the changes documented in https://github.com/rucio/rucio/issues/6048 This is required to trigger an x509 client certificate popup in the browser when the WebUI makes a request to the x509 endpoint.

bari12 commented 1 year ago

@mlassnig @rcarpa please 👀 too To me this looks ok, since x509 makes no sense if you do not present a certificate anyway, but please double check if I am missing something.

rcarpa commented 1 year ago

I came across this one: https://github.com/CESNET/gridsite/issues/34 ; are you sure your configuration change will actually work correctly ?

bari12 commented 1 year ago

Push; Is this still needed?

maany commented 1 year ago

@bari12 it is required for x509 to work with the new WebUI ( to explicitly force the browser to request for the user certificate). Bump for 32.0