Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Implements DHIS2-18585
Description:
This pull request addresses an issue with the login page where the dynamic base URL was not considering the URL context path (e.g.,
/dev
,/prod
). Without this fix, the application attempted to send the authentication request to the wrong endpoint (/api/auth/login
instead of/dev/api/auth/login
), leading to a failure when hosted under specific context paths like/dev
.Changes:
getBaseUrl()
function to account for the context path (e.g.,/dev
,/dev-2-41
, etc.) in the URL.POST /api/auth/login
are sent to/dev/api/auth/login
when the app is hosted under/dev
.login.html
, that part is correctly stripped off while preserving the context path.Problem Addressed:
/dev/login.html
), the login form submitted requests tohttp://localhost:8080/api/auth/login
instead of the expected/dev/api/auth/login
.