404 response to /regulator/{id}/ when not all ids could be mapped.

[alecKent]

I am curious about the logic for using 404 as the response if not all ids were able to be attached.

404 is defined as "The server has not found anything matching the Request-URI".

If I am reading the docs correctly in the case that not all ids could be mapped the request URI /regulator/{id}/ is valid and a matching resource was found, the issue exists within the array of ids to map.

Since the /regulator/{id}/ transaction doesn't seem to follow ACID I think the request should return a 200 response code since it is possible that some ids were mapped successfully.
Then it is on the caller to check if the failures array has data / exists at all.

Thoughts on this?

[elibosley]

This is a really great point. I am happy to change to a 200 with a failure array.

[alecKent]

Nice. I think a 200 would result in the least amount of confusion.
That way 404 can be reserved for when the requested resource is not found.

[danaspiegel]

#13

[danaspiegel]

Closing issue as this will be handled with issue #13