adlnet / xAPI-Spec

The xAPI Specification describes communication about learner activity and experiences between technologies.
https://adlnet.gov/projects/xapi/
898 stars 404 forks source link

Document PUT requirements eliminate natural idempotence of PUT #1027

Open bscSCORM opened 7 years ago

bscSCORM commented 7 years ago

Note that these requirements do not allow an exception for the same exact body being PUT multiple times:

If a PUT request is received without either header for a resource that already exists, the LRS:

I suggest we change the language above to:

If a PUT request is received without either header for a resource that already exists *but is not identical to the body of the request*, the LRS:

(detailed requirements unchanged)
garemoko commented 7 years ago

+1, we can tweak the language whenever it's added.

Would you agree this is a minor change?

bscSCORM commented 7 years ago

Yes, minor not patch or major, it will require an LRS change but no clients should be relying on the old behavior.

andyjohnson commented 7 years ago

+1, this became too process oriented and less to the point. Agree this is minor.

andyjohnson commented 7 years ago

Per the 2/1/17 call, @bscSCORM 's revision is sound