Skip to content
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.

Sign up for GitHub

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

Jump to bottom

rc-cts-urlscanio-1.0.1 #70

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

rc-cts-urlscanio-1.0.1 #70

wants to merge 2 commits into from

Conversation

lmahoney1
Copy link

@lmahoney1 lmahoney1 commented Mar 8, 2021 •
edited
Loading

Description

Changed the variable the CTS uses to determine if a result from urlscan.io is malicious or not.

Motivation and Context

Fixes #69

How Has This Been Tested?

Submitted the url mentioned in the issue and the CTS returns a hit.

Checklist:

  • I have added a Signed-off-by
  • Either no new documentation is required by this change, OR I added new documentation
  • Either no new tests are required by this change, OR I added new tests
  • I have run pep8 and pylint. I have cleaned up all valid errors and warnings in code I have added or modified. These tools may generate false positives. Don't be worried about ignoring some errors or warnings. The goal is clean, consistent, and readable code.

Signed-off-by: Liam Mahoney lmahoney@oshkoshcorp.com

@lmahoney1 lmahoney1 mentioned this pull request Apr 20, 2021
Closed
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

rc-cts-urlscanio Not Returning Hits
1 participant
@lmahoney1