-
Notifications
You must be signed in to change notification settings - Fork 1.6k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat(export): add API export system #6878
Conversation
3ba6457
to
e8476ef
Compare
e8476ef
to
d5e2d75
Compare
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #6878 +/- ##
==========================================
- Coverage 92.68% 89.30% -3.39%
==========================================
Files 76 1278 +1202
Lines 6698 41887 +35189
==========================================
+ Hits 6208 37406 +31198
- Misses 490 4481 +3991
Flags with carried forward coverage won't be shown. Click here to find out more.
|
60a1767
to
747b97f
Compare
5743b35
to
cbf8cf7
Compare
cdb9d8b
to
84e66ce
Compare
84e66ce
to
cf761ab
Compare
b6f124a
to
f5f62f8
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Finally this get to an end, super happy about this new functionality, it required a great effort but it is done 👏 awesome work 🚀
b803f4e
Description
In this PR we have refactored the code in charge of performing the scans from the API side, now all the report files will always be generated, they will be compressed and can be uploaded to S3 or just keep them locally. This happens in a separate task.
We have added an endpoint to get the files from local and from the S3 bucket.
Checklist
API
License
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.