api.methodOverwrite config option and check

[distantnative]

Description

Summary of changes

• Switching default to use pure PATCH requests etc. without x-http-method-override header
• New api.methodOverwrite config option to activate x-http-method-override header
• New check in system settings that shows error if PATCH requests don't work

Reasoning

https://forum.getkirby.com/t/kirby-panel-not-editable-with-bunny-cdn/32215

Changelog

Enhancements

• api.methodOverwrite config option to activate pure PATCH requests

Docs

• Add api.methodOverwrite config option docs

Ready?

• In-code documentation (wherever needed)
• Unit tests for fixed bug/feature
• Tests and CI checks all pass

For review team

• Add lab and/or sandbox examples (wherever helpful)
• Add changes & docs to release notes draft in Notion

[bastianallgeier]

I really like this new option and it would be great if we could get this PR done.

[beromir]

Thank you @distantnative, works great!
But shouldn't the default for the option be false? In my case the method-test in the system view returned a 404 error, because it was using the method override header. After disabling it, it worked.

[distantnative]

@beromir that would be a breaking change for all setups that don't support pure PATCH requests etc. which we could not introduce in 4.5. This is why you actively have to set the config option, if you want the new behavior.

[beromir]

Yeah, makes sense. Thank you.