nickva commented on issue #2241: URL: https://github.com/apache/couchdb/issues/2241#issuecomment-1808527119
400 is more correct, however 403 has been traditionally used by CouchDB as a response from validate doc update (VDU) functions. At some point `_replicator` docs were also validated by an auto-injected design doc. Now they are re-written as an Erlang function, however the validation response was preserved for backwards compatibility. Another aspect here is that a 403 is also a signal to a replicator to skip over a reject document on the target. That might be in the odd case or replicating `_replicator` dbs themselves. At some point we used to skip over any 4xx error, but that's not the case [any longer](https://github.com/apache/couchdb/commit/d4d49077d3ecca7d5122f4ba0961fa38f6fa597a). That sort of argues that we should just preserve the 403 response. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: notifications-unsubscr...@couchdb.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
