`RequestManagerImpl` may generate incorrect email links
- Linguagem predominante
- Java
- Estrelas
- 0
- Forks
- 0
- Métricas de merge de PRs
- Nenhum PR com merge em 30d
Descrição
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.
Guia de contribuição
Nenhum guia de contribuição indexado para este repositório
Avaliação
Esta issue ainda não foi avaliada.