WordPress.org
Get WordPress

Make WordPress Core

Opened 10 years ago

Closed 10 years ago

Last modified 10 years ago

#38546 closed defect (bug) (fixed)

REST API: Add PATCH to CORS allowed methods

Reported by: jnylen0 Owned by: pento
Priority: normal Milestone: 4.7
Component: REST API Version: 4.7
Severity: normal Keywords: has-patch
Cc: Focuses:

Description

Currently it is not possible to send a CORS request with the PATCH method, even though this is supported by any "editable" resource in the REST API.

Attachments (1)

38546.diff​ (565 bytes ) - added by jnylen0 10 years ago.

Download all attachments as: .zip

Change History (6)

@jnylen0
10 years ago

#1 @jnylen0
10 years ago

  • Keywords has-patch added

#2 @joehoyle
10 years ago

Nice! I'm remembering there was a good amount of discussion around the use of PATCH, as I *think* there are some different semantics for it - I'm trying to find the github ticket where that was discussed but I'm failing to find it right now. @rmccue @rachelbaker for any help there.

As long as that doesn't impact this, this looks good to me.

#3 @jnylen0
10 years ago

Did a search through the GH issues, and apparently people use the word "patch" a lot when developing OSS ;)

Found this, though... ​https://github.com/WP-API/WP-API/pull/768#issuecomment-73878867

Actually, our use of PATCH is kind of an abuse of the verb [...] We support it with the same semantics as PUT

#4 @pento
10 years ago

  • Owner set to pento
  • Resolution → fixed
  • Status new → closed

In 39042:

REST API: Add PATCH to CORS allowed methods.

Editable resources in the REST API accept the PATCH method, but the CORS headers don't mention it.

Props jnylen0.
Fixes #38546.

#5 @rachelbaker
10 years ago

  • Milestone Awaiting Review → 4.7
Note: See TracTickets for help on using tickets.