`RequestManagerImpl` may generate incorrect email links
- Lingua principale
- Java
- Stelle
- 0
- Fork
- 0
- Metriche di merge delle PR
- Nessuna PR unita negli ultimi 30g
Descrizione
When the `RequestManager` generates notification emails for incoming broker requests, it creates a link to the query by concatenating the `serverUrl` with the request path. However, there's a bug when the `serverUrl` in DWH preferences doesn't end with a forward slash "/".
Current implementation:
https://github.com/aktin/dwh-query/blob/cd48a3401b17f35b4ba3fae85725b6b5791eb527/request-manager/src/main/java/org/aktin/request/manager/RequestManagerImpl.java#L502-L503
Example of the issue:
- When serverUrl is "http://aktinvm"
- Generated URL becomes: "http://aktinvmaktin/admin/..." (invalid)
- Expected URL: "http://aktinvm/aktin/admin/..."
The system needs to check if serverUrl ends with "/" and append it if missing to ensure proper URL generation.
Guida per i contributori
Nessuna guida per i contributori indicizzata per questo repository
Valutazione
Questa issue non è ancora stata valutata.