You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
Check whether 429 and 500 errors are needed for this API #40
Thanks @Kevsy. Checked against the CAMARA API Design Guide (section 3.2.1, same wording in r4.3 and r4.4):
500: 5xx statuses are "not documented by default" and nothing in this API justifies an exception. Removed in the rc. Implementations may still return 500; the mandatory "Additional CAMARA error responses" text in the description covers that.
429: in the tier that "SHOULD be documented based on the API design", so it is the subproject's call. We keep it, since rate limiting will apply to this API in practice and most CAMARA APIs document it. With the move to Commonalities r4.4 it becomes the minimal TooManyRequests429 catalogue response (TOO_MANY_REQUESTS only, QUOTA_EXCEEDED dropped).
Activity
maheshc01 commentedon Sep 14, 2026
Thanks @Kevsy. Checked against the CAMARA API Design Guide (section 3.2.1, same wording in r4.3 and r4.4):
500: 5xx statuses are "not documented by default" and nothing in this API justifies an exception. Removed in the rc. Implementations may still return 500; the mandatory "Additional CAMARA error responses" text in the description covers that.
429: in the tier that "SHOULD be documented based on the API design", so it is the subproject's call. We keep it, since rate limiting will apply to this API in practice and most CAMARA APIs document it. With the move to Commonalities r4.4 it becomes the minimal
TooManyRequests429catalogue response (TOO_MANY_REQUESTSonly,QUOTA_EXCEEDEDdropped).Will close this issue with the r4.4 alignment PR.
Address r1.1 review findings in the API definition and test definitions
Address r1.1 review findings in the API definition and test definitions