Closed JQvickWester closed 5 months ago
Thank you for your submission! We have a total of 16 PRs open right now, and we are working hard on all of them! We will take a look as soon as we can.
@JQvickWester Static Tests results for commit f22a48483082e749b4644969196d22158b97521f - https://drive.google.com/file/d/1MfHu-f1w-nKBPx0feWykTkyOqvGkggtv/view?usp=drivesdk
@JQvickWester Compile Tests results for commit f22a48483082e749b4644969196d22158b97521f - https://drive.google.com/file/d/1__yHS3uO8yKOlf8bI8pLzc4vxIazz33K/view?usp=drivesdk
@JQvickWester Static Tests results for commit 88a996b68408391249631cba0a4f72187a7b4663 - https://drive.google.com/file/d/1xwXYWU5Hi-uoA6fZL9kDXFV7FbHQH9k-/view?usp=drivesdk
@JQvickWester Compile Tests results for commit 88a996b68408391249631cba0a4f72187a7b4663 - https://drive.google.com/file/d/1jD6czO5-F8SWXN3rSAl8V_Sl55DJ7Ccy/view?usp=drivesdk
@JQvickWester Static Tests results for commit 8f365d367823fe15d70b66c3a37c308a7eaac80a - https://drive.google.com/file/d/18GNx4f1E0MZBgrRZXuyp-BjEL0uMWm2T/view?usp=drivesdk
@JQvickWester Compile Tests results for commit 8f365d367823fe15d70b66c3a37c308a7eaac80a - https://drive.google.com/file/d/1oOjnQqhmH350OStyThDqayTl-l9LiCnI/view?usp=drivesdk
Hi @dhwanis-crest
Looking to get this merged, but I'm a bit confused about the AWS CodeBuild us-west-2 (prodsec-scans)
action that appears to be failing. Clicking details redirect me to an AWS login page.
@nestoor22 suggested that I reach out to you.
Do you have any more details as to why this is failing?
Thanks!
The prodsec fail is caused by requests package, please remove it from requirements, as it is supplied by SOAR platform @JQvickWester
@JQvickWester Static Tests results for commit 8efa1d5cb00f1ba16a6d1034811a702a3792841a - https://drive.google.com/file/d/1NNix-1BruPQvr5ZiBhIXcMNUYstmnKdz/view?usp=drivesdk
@JQvickWester Compile Tests results for commit 8efa1d5cb00f1ba16a6d1034811a702a3792841a - https://drive.google.com/file/d/1_x84k78mbjySh0oXPBkIG9IHx2y-QIzb/view?usp=drivesdk
@bb-splunk Thank you for you assistance, that cleared it up - have a great day!
@nestoor22 looks like this is ready to merge, if you'd be so kind!
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)
manual_readme_content.md
.manual_readme_content.md
all methods (eg, OAuth) used to authenticate with the service that the connector is integrating with.manual_readme_content.md
.nosemgrep
? If yes, please provide justification in an additional comment next to the ignored code. API key fingerprint is logged for debugging purposes.Release Notes (REQUIRED)
What is the current behavior? (OPTIONAL)
What is the new behavior? (OPTIONAL)
Other information (OPTIONAL)
Pay close attention to (OPTIONAL)
Screenshots (if relevant)
Thanks for contributing!