Skip to content
This repository has been archived by the owner on Feb 14, 2024. It is now read-only.

2- Create and implement Cybersixgill views #195

Draft
wants to merge 4 commits into
base: develop
Choose a base branch
from

Conversation

aloftus23
Copy link
Contributor

@aloftus23 aloftus23 commented May 3, 2022

fixes #182

This PR should be merged after these updates are in develop:

🗣 Description

Creates views for all of the cybersixgill tables:

  1. vw_darkweb_mentionsbydate
  2. vw_darkweb_socmedia_mostactposts
  3. vw_darkweb_mostactposts
  4. vw_darkweb_assetalerts
  5. vw_darkweb_execalerts
  6. vw_darkweb_threatactors
  7. vw_darkweb_potentialthreats
  8. vw_darkweb_sites
  9. vw_darkweb_inviteonlymarkets
  10. vw_darkweb_topcves

💭 Motivation and context

These views make adhoc investigations more efficient and limits the amount of clean-up needed on the python side.

🧪 Testing

Passes all pre-commits and pytests

✅ Pre-approval checklist

  • This PR has an informative and human-readable title.
  • Changes are limited to a single goal - eschew scope creep!
  • All future TODOs are captured in issues, which are referenced
    in code comments.
  • All relevant type-of-change labels have been added.
  • I have read the CONTRIBUTING document.
  • These code changes follow cisagov code standards.
  • All relevant repo and/or project documentation has been updated
    to reflect the changes in this PR.
  • Tests have been added and/or modified to cover the changes in this PR.
  • All new and existing tests pass.

@coveralls
Copy link

coveralls commented May 3, 2022

Coverage Status

Coverage decreased (-1.05%) to 17.937% when pulling 0b88d9e on AL-cybersixgill-views into 78ccbfd on develop.

@lgtm-com
Copy link

lgtm-com bot commented May 3, 2022

This pull request introduces 3 alerts when merging 69b16b4 into c926100 - view on LGTM.com

new alerts:

  • 2 for Wrong name for an argument in a class instantiation
  • 1 for Wrong number of arguments in a call

@aloftus23 aloftus23 mentioned this pull request May 23, 2022
1 task
@aloftus23 aloftus23 changed the title Create and implement Cybersixgill views 2- Create and implement Cybersixgill views Sep 26, 2022
@lgtm-com
Copy link

lgtm-com bot commented Sep 26, 2022

This pull request introduces 1 alert when merging 0b88d9e into 78ccbfd - view on LGTM.com

new alerts:

  • 1 for Mismatch in multiple assignment

@aloftus23 aloftus23 self-assigned this Oct 25, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Create Cybersixgill Views
2 participants