Fiber: 🐛 Invalid http method crashes fiber

[dioslibre]

Fiber version
1.13.3

Issue description
Lowercase patch crashes fiber, 'PATCH' works fine. This problem does not happen with other methods.

Code snippet

package main

import "github.com/gofiber/fiber"

func main() {
  app := fiber.New()

  // Steps to reproduce

 app.Post("/api/one/:model", u.VerifyToken, d.Create)
// await fetch(url, { method: 'post', opts }) ==>  OK
// await fetch(url, { method: 'POST', opts }) ==> OK

 app.Patch("/api/one/:model", u.VerifyToken, d.Update)
// await fetch(url, { method: 'patch', opts }) ==>  crash
// await fetch(url, { method: 'PATCH', opts }) ==> OK

  app.Listen(3000)
}

[welcome[bot]]

Thanks for opening your first issue here! 🎉 Be sure to follow the issue template! If you need help or want to chat with us, join us on Discord https://gofiber.io/discord

[ReneWerner87]

thanks for the bug report

after some tests with, curl, httpie and node-fetch and several different constellations we don't know exactly how you produced the bug, please provide a step by step guide to the bug

[dioslibre]

First, I should mention that the work you did with fiber is awesome, Thanks a million.

Why would you test with curl, httpie, node-fetch and all the several different constellations you speak of and not test with browser fetch?! perhaps I should have been more specific.

Anyway, here's a simple working example demonstrating what I'm talking about :
https://github.com/dioslibre/fiber-patch-bug

[kiyonlin]

Hi @dioslibre I ran your demo, and I found fetch send an OPTIONS request first before PATCH in my chrome so that fiber returns 405 Method Not Found.

[dioslibre]

hey @kiyonlin, I'm using chrome also, I don't get 405 Method Not Found, I get net::ERR_CONNECTION_REFUSED and fiber crashes.

[dioslibre]

@Fenny it also crashes with curl, and not only with lowercase patch, but lowercase post also. I ran this curl -X post http://localhost:3000/api and it crashed. What did you do to test this on curl?

[kiyonlin]

I already fixed this bug(#672), but it hasn't released yet. Under normal circumstances, http methods are all uppercase.

Anyway, thanks for your reporting!

[dioslibre]

@kiyonlin 'Under normal circumstances, http methods are all uppercase' Yes, you're right.
Thank you for taking the time, and for coding this beautiful library.
I'll close this for now.