Skip to content
This repository has been archived by the owner on Apr 4, 2022. It is now read-only.

Updated info about PATCH requests #16

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

adhulipa
Copy link

@adhulipa adhulipa commented May 30, 2018

Obvious Fix(?)

Based on the description provided by the RFC 5789 specification,
I believe that PATCH requests are not necessarily idempotent (in the same vein as POST requests).

I'm quoting the specific lines from that document for reference:

PATCH is neither safe nor idempotent as defined by [RFC2616], Section
9.1.

A PATCH request can be issued in such a way as to be idempotent,

Based on the description provided by the RFC 5789 specification here - https://tools.ietf.org/html/rfc5789
I believe that PATCH requests are not necessarily idempotent (in the same vein as POST requests).

I'm quoting the specific lines from that document for reference:

   PATCH is neither safe nor idempotent as defined by [RFC2616], Section
   9.1.

   A PATCH request can be issued in such a way as to be idempotent,
@pivotal-issuemaster
Copy link

@adhulipa Please sign the Contributor License Agreement!

Click here to manually synchronize the status of this Pull Request.

See the FAQ for frequently asked questions.

@pivotal-issuemaster
Copy link

@adhulipa Thank you for signing the Contributor License Agreement!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants