OpenCTI-Platform / opencti

Open Cyber Threat Intelligence Platform
https://opencti.io
Other
5.12k stars 813 forks source link

[backend] Change log level for S3 download file errors (#7467) #7468

Closed aHenryJard closed 3 days ago

aHenryJard commented 3 days ago

Proposed changes

I did not change the behavior (like throwing exception instead of just log it) because it's use at many places and I don't want to change it all. I just made the current behavior explicit in tests.

Related issues

Closes 7467

Checklist

Further comments

codecov[bot] commented 3 days ago

Codecov Report

Attention: Patch coverage is 70.00000% with 3 lines in your changes missing coverage. Please review.

Project coverage is 67.37%. Comparing base (8590220) to head (6a8701f). Report is 2 commits behind head on master.

Files Patch % Lines
...tform/opencti-graphql/src/database/file-storage.js 70.00% 3 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #7468 +/- ## ======================================= Coverage 67.36% 67.37% ======================================= Files 563 563 Lines 69643 69652 +9 Branches 5886 5888 +2 ======================================= + Hits 46916 46927 +11 + Misses 22727 22725 -2 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.