yetanalytics / lrs

Protocols, specifications, and logic for building an xAPI Learning Record Store (LRS) in Clojure(Script).
https://www.yetanalytics.com/lrs
Apache License 2.0
4 stars 1 forks source link

LRS-40 swap terms for 401/403 #37

Closed milt closed 3 years ago

milt commented 3 years ago

LRS-40

This is just a naming change, but it is an important distinction so I think it is worth correcting downstream.

The big change is the :com.yetanalytics.lrs.auth/forbidden keyword needs to change to :com.yetanalytics.lrs.auth/unauthorized.

I've changed it in the spec, but still check the old one for now too. This means that spec-based tests and instrumentation will complain, as they should, but the production runtime will still work.