We should be able to add / and /* to RouterService

[sbernard31]

Currently this is not possible to do :

RouterService.RouteBuilder routerBuilder = new RouterService.RouteBuilder();
routerBuilder //
        .any("/", new RootResource() //
        .any("/myResource", new MyResource() //
        .any("/*", new Default());
router = routerBuilder.build();

If you do that RootResource will never be called.
I think this is because / and /* will create 2 RequestMatcher which will be considered as equal and so Default will override RootResource in handlers hashmap.
Maybe the solution is to add isPrefixed attribute to equals() and hashcode() methods ?

(triggered by #27 (comment))

[sbernard31]

Not directly linked but not sure to get why isPrefixed is transient ? 🤔

static final class RequestMatcher {
    private final Method method;
    private final String uriPath;
    private transient final boolean isPrefixed;

[szysas]

Well, that transient was to prevent adding isPrefixed to equals method.

[sbernard31]

Oh I was thinking transient was about Java Serialization. Not sure to know how this is related to equals ? Maybe something related to IDE code generation ?

[szysas]

That is needed by this testing tool: https://jqno.nl/equalsverifier/ to skip fields from being tested.

[sbernard31]

Oh I didn't know about EqualsVerifier maybe this is a library we should use in Leshan 🤔

About transient, I read documentation : https://jqno.nl/equalsverifier/manual/ignoring-fields/#transient-fields and they also talk about serialization maybe it make more sense to use : https://jqno.nl/equalsverifier/manual/ignoring-fields/#ignoring-fields
(but clearly this is very small details)