I noted that the error handling in restrict-overwrite example seems unsuitable for production use:
|
console.log(`AQL query failed: ${x.message}`); |
On an occasion where the AQL query were to fail, the error is spuriously silenced, and code proceeds with return { status: UploadStatus.UPLOAD_PROCEED };
The expected behavior would to be fail safely, that is, the error should lead to upload being stopped.
I noted that the error handling in restrict-overwrite example seems unsuitable for production use:
workers-sample/samples/artifactory/BEFORE_UPLOAD/restrict-overwrite/worker.ts
Line 49 in 12b0d34
On an occasion where the AQL query were to fail, the error is spuriously silenced, and code proceeds with
return { status: UploadStatus.UPLOAD_PROCEED };The expected behavior would to be fail safely, that is, the error should lead to upload being stopped.