Add trailing slash to dashboard

[jvalkeal]

• Boot 3.x changed handling of trailing slash meaning it's now adviced to define both /dashboard and /dashboard/ in UiController

Essentially to fix latter:

jvalkealahti@cypher:/tmp/xxx$ http --headers http://localhost:9393/dashboard
HTTP/1.1 302 
Connection: keep-alive
Content-Language: en-GB
Content-Length: 0
Date: Mon, 13 May 2024 12:36:17 GMT
Keep-Alive: timeout=60
Location: http://localhost:9393/dashboard/index.html

jvalkealahti@cypher:/tmp/xxx$ http --headers http://localhost:9393/dashboard/
HTTP/1.1 500 
Connection: close
Content-Type: application/hal+json
Date: Mon, 13 May 2024 12:36:21 GMT
Transfer-Encoding: chunked
Vary: Origin
Vary: Access-Control-Request-Method
Vary: Access-Control-Request-Headers

[corneil]

LGTM

[onobc]

Because this approach only fixes this controller, until we adjust globally (recommended by the release notes) I would rather set the default as it was before using something like:

@Configuration
public class WebConfiguration implements WebMvcConfigurer {

    @Override
    public void configurePathMatch(PathMatchConfigurer configurer) {
      configurer.setUseTrailingSlashMatch(true);
    }
  
}

Wdyt?

[jvalkeal]

@onobc sorry I missed your question. I'd rather not to set it globally because setUseTrailingSlashMatch is already deprecated. We anyway need to handle this in a recommended way going forward.

[onobc]

@onobc sorry I missed your question

No worries @jvalkeal.

I'd rather not to set it globally because setUseTrailingSlashMatch is already deprecated. We anyway need to handle this in a recommended way going forward.

If we don't fix it globally then maybe we should not fix the dashboard either. Since this is a major update we can just list this as a breaking change.

[onobc]

Thanks @jvalkeal - please see my comment.