fix: always uppercase method option

[murisceman]

🔗 Linked issue

❓ Type of change

• 📖 Documentation (updates to the documentation, readme, or JSdoc annotations)
• 🐞 Bug fix (a non-breaking change that fixes an issue)
• 👌 Enhancement (improving an existing functionality like performance)
• ✨ New feature (a non-breaking change that adds functionality)
• 🧹 Chore (updates to the build process or auxiliary tools and libraries)
• ⚠️ Breaking change (fix or feature that would cause existing functionality to change)

📚 Description

The JavaScript Fetch API supports lowercased get, post, delete, and put methods, but not the lowercased patch method. The method option in ofetch accepts a string | undefined, and the mentioned remark may be easy to forget without literal types. Therefore, it makes sense to standardize the behavior of all methods by converting them to uppercase. By convention, standardized methods are defined in all-uppercase, so this change should not break anything.

Additionally, the isPayloadMethod function already checks the method name by converting it to uppercase.

// Works
fetch('https://dummyjson.com/products/1', {
    method: 'PATCH',
    headers: { 'Content-Type': 'application/json' },
    body: JSON.stringify({ title: 'Test' }),
  })
    .then((res) => res.json())
    .then(console.log)

// Doesn't work
fetch('https://dummyjson.com/products/1', {
    method: 'patch', // <-- lowercased
    headers: { 'Content-Type': 'application/json' },
    body: JSON.stringify({ title: 'Test' }),
  })
    .then((res) => res.json())
    .then(console.log)

// Works
fetch('https://dummyjson.com/products/1', {
    method: 'put', // <-- also lowercased, but it works
    headers: { 'Content-Type': 'application/json' },
    body: JSON.stringify({ title: 'Test' }),
  })
    .then((res) => res.json())
    .then(console.log)

📝 Checklist

• I have linked an issue or discussion.
• I have updated the documentation accordingly.

[pi0]

THanks for pr. Do you have any reference about PATCH exception?

[murisceman]

https://fetch.spec.whatwg.org/#methods

[danielroe]

I can confirm patch exception - it's an unfortunate inconsistency and has been at the bottom of several 'bugs' I investigated.

[pi0]

Thanks!