Lucas Lemos

Results 96 comments of Lucas Lemos

@ReneWerner87 I could only think about splitting the Path and checking the suffix, but that would remove any possbility of having any /[n+]/(livez | readyz) routes. But I think this...

@ReneWerner87 yep, I'll check them and write a test with StrictRouting mode too. Thanks for the help and heads up!

@ReneWerner87 I'll send a PR with the reports and results by Sunday, is that okay?

@lino-levan the PR is already merged, @kt3k @bartlomieju is this issue solved with the mentioned PR?

@ReneWerner87 @gaby could you guys take a look at this for me? I don't think I have messed up the benchmark that badly but maybe something else can be worked...

@gaby the only other thing I could think of is splitting the route by slashes and checking the last route to see if it's either one of the set up...

@ReneWerner87 yours fail in the strict routing test but @gaby solution passed it no prob, benchmark results: ![image](https://github.com/gofiber/fiber/assets/48929501/cf654794-4ca7-42da-b234-e8365e697f4c)

It was kind of what I wanted to do at first, but maybe I misunderstood @ReneWerner87 saying he wanted a different approach than that

I managed to extract 7 ns/op of @gaby solution by using switch case, let me see if I can do more

@gaby without the +1 looks better, because then we wouldn't have to trim the already set up config to remove the slashes, passed it fine and it looks very good...