-
Notifications
You must be signed in to change notification settings - Fork 2
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
ANY.RUN: Bugfix - Fix type validation errors in get_intelligence
action
#3
ANY.RUN: Bugfix - Fix type validation errors in get_intelligence
action
#3
Conversation
Thank you for your submission! We have a total of 22 PRs open right now, and we are working hard on all of them! We will take a look as soon as we can. |
@anyrun-integrations Static Tests results for commit 0213275 - https://drive.google.com/file/d/1-RxAUXM5Bx1-6UcfHZCkq6GRxNMaxZSu/view?usp=drivesdk |
@anyrun-integrations Compile Tests results for commit 0213275 - https://drive.google.com/file/d/1hRxaWGfRRyY8DndHF_JTNHkagl27pk4C/view?usp=drivesdk |
@ishans-crest Static Tests results for commit 6af170f - https://drive.google.com/file/d/1FoNt-ouKVh9bL4gi4CnLWGyTx7bwxCy-/view?usp=drivesdk |
@ishans-crest Compile Tests results for commit 6af170f - https://drive.google.com/file/d/1ndimUNrVNB9MC66yn5nBKo0Sp7K_KR_D/view?usp=drivesdk |
@ishans-crest Static Tests results for commit cd919ca - https://drive.google.com/file/d/1us7FuYOKROLXKTpiv0ZLufIGCr2F_L65/view?usp=drivesdk |
@ishans-crest Static Tests results for commit 657de02 - https://drive.google.com/file/d/19gLb2gsU0IY4Xk2GQAxebgV71TRU1qNK/view?usp=drivesdk |
@ishans-crest Compile Tests results for commit 657de02 - https://drive.google.com/file/d/1rXtsHI0r_aYgsY4U6X4bLe5iBoVnPViu/view?usp=drivesdk |
@ishans-crest Static Tests results for commit 791af1d - https://drive.google.com/file/d/1kJUbJytXh-aSR-XkuXFl5_r-5aBCmL4q/view?usp=drivesdk |
@ishans-crest Compile Tests results for commit 791af1d - https://drive.google.com/file/d/1JIUnh4zOYHifB1OCEjXkjj2PqDdiAh23/view?usp=drivesdk |
@ishans-crest Static Tests results for commit 316b417 - https://drive.google.com/file/d/1Q1GWbIXeEEW_Rrbnf_7H4yjt-tx-uzu4/view?usp=drivesdk |
@ishans-crest Compile Tests results for commit 316b417 - https://drive.google.com/file/d/1-Y3wa7hLtBgfuaDBiI68qaeA0j6s3bBL/view?usp=drivesdk |
We have made the necessary changes in the application code. One of the changes is upgrading the 'aiohttp' library version from 3.9.5 to 3.10.2. Let us know your feedback on this change. |
@anyrun-integrations Static Tests results for commit 3b3a699 - https://drive.google.com/file/d/1JF0tdNN7CApvTU_p3Mbn9hOpvBvmL0TY/view?usp=drivesdk |
@anyrun-integrations Compile Tests results for commit 3b3a699 - https://drive.google.com/file/d/1IRD_yKItNT8hLKloIHwP2hLP_J8rY7eY/view?usp=drivesdk |
Hello @ishans-crest, We have updated 'anyrun-py' lib to work with aiohttp 3.10.2, otherwise every action would've failed with 'RuntimeError: no running event loop' error. After that no errors were encountered, so it should be good now. |
f591772
into
splunk-soar-connectors:next
Please ensure your pull request (PR) adheres to the following guidelines:
Pull Request Checklist
Please check if your PR fulfills the following requirements:
<App Name>: <PR Type> - <PR Description>
next
branch of the forked repo. Create separate feature branch for raising the PR.Pull Request Type
Please check the type of change your PR introduces:
Security Considerations (REQUIRED)
please document them in the
manual_readme_content.md
.manual_readme_content.md
all methods (eg, OAuth) used to authenticatewith the service that the connector is integrating with.
manual_readme_content.md
.the Vault API.
nosemgrep
?If yes, please provide justification in an additional comment next to the ignored code.
Release Notes (REQUIRED)
get_intelligence
actionWhat is the current behavior? (OPTIONAL)
get_intelligence
returns an error "Error in validating Threat Intelligence parameters. Error Message: '< param >' must be str, not < param_type >.", when any parameter with type other than 'string' is used.What is the new behavior? (OPTIONAL)
get_intelligence
now performs type casting for all used parameters.Other information (OPTIONAL)
Pay close attention to (OPTIONAL)
Screenshots (if relevant)
Thanks for contributing!