RADAR-base / ManagementPortal

Management Portal to manage research studies
Apache License 2.0
21 stars 16 forks source link

Fix/login redirects #882

Closed Bdegraaf1234 closed 3 months ago

Bdegraaf1234 commented 3 months ago

Login redirects were not set correctly in #879, additionally the antmatchers were not set in all required locations.

This should resolve the issue and add some other small improvements.

Bdegraaf1234 commented 3 months ago

@mpgxvii please correct me if I am wrong but for me therequests to /api/redirect/login/ returned 401 at some point. I am double checking this as we speak though

mpgxvii commented 3 months ago

@mpgxvii please correct me if I am wrong but for me therequests to /api/redirect/login/ returned 401 at some point. I am double checking this as we speak though

The previous security config were working for me. It was just the front end url that didn't work. Let me double check as well.

Bdegraaf1234 commented 3 months ago

I think you were right pauline, apologies. I will remove the lines, can you review the this PR?