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
Add API Metrics Reporting Support to CommonServer #16724
Conversation
@yuvalbenshalom @anara123 This was more or less my idea as to how to handle the rate limited requests. We set a new property called |
…-GSB-CommonServer � Conflicts: � Packs/Base/ReleaseNotes/1_18_0.md � Packs/Base/pack_metadata.json � Packs/GoogleSafeBrowsing/pack_metadata.json
Packs/GoogleSafeBrowsing/Integrations/GoogleSafeBrowsingV2/GoogleSafeBrowsingV2.py
Outdated
Show resolved
Hide resolved
…-GSB-CommonServer � Conflicts: � Packs/UrlScan/Integrations/UrlScan/UrlScan.py � Packs/UrlScan/pack_metadata.json
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.
see my comments
Packs/GoogleSafeBrowsing/Integrations/GoogleSafeBrowsingV2/GoogleSafeBrowsingV2.py
Outdated
Show resolved
Hide resolved
demisto.results([{
'Type': entryTypes['note'],
'ContentsFormat': formats['markdown'],
'Contents': "This is an Error",
'HumanReadable': f"This command returned 60 ratelimit errors",
'ErrorType': "RateLimited",
'APICalls': 60
}]) @anara123 @yuvalbenshalom I have the API Call count working, but we really should think of a way to pretty this up. |
…-GSB-CommonServer
…-GSB-CommonServer
This pull request introduces 1 alert when merging 6a5d311 into c0aa406 - view on LGTM.com new alerts:
|
…-GSB-CommonServer � Conflicts: � Packs/Base/ReleaseNotes/1_19_0.md � Packs/Base/pack_metadata.json � Packs/VirusTotal/ReleaseNotes/2_2_1.md
…-GSB-CommonServer
…-GSB-CommonServer
@yuvalbenshalom @anara123 Validations fail due to moving UrlScan to Py3, otherwise I believe we are good to go. |
…-GSB-CommonServer � Conflicts: � Packs/Base/pack_metadata.json
Link to the unit tests coverage report: |
Contributing to Cortex XSOAR Content
Make sure to register your contribution by filling the contribution registration form
The Pull Request will be reviewed only after the contribution registration form is filled.
Status
Related Issues
https://github.com/demisto/etc/issues/32115
Description
This PR adds the rate limit handling to CommonServerPython, UrlScan, and to VirusTotal
Minimum version of Cortex XSOAR
Does it break backward compatibility?
Must have