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

Add stakeholder full and NER #273

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

Conversation

cduhn17
Copy link
Collaborator

@cduhn17 cduhn17 commented Sep 23, 2022

🗣 Description

💭 Motivation and context

Not all stake holders require a full report, this pull request will split full and lite versions of PE_Reports. Additionally this PR will add the process of gathering executive names as it is a time consuming task and we have the need to automate the process of adding large numbers of stakeholders.This PR closes #187

✅ 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.

✅ Pre-merge checklist

  • Revert dependencies to default branches.
  • Finalize version.

✅ Post-merge checklist

  • Add a tag or create a release.

@cduhn17 cduhn17 added improvement This issue or pull request will add or improve functionality, maintainability, or ease of use upstream update This issue or pull request pulls in upstream updates labels Sep 23, 2022
@cduhn17 cduhn17 self-assigned this Sep 23, 2022
def getToken():
"""Get authorization token from Cybersixgill (CSG)."""
LOGGER.info(API_Client_ID)
LOGGER.info(API_Client_secret)

Check failure

Code scanning / CodeQL

Clear-text logging of sensitive information

[Sensitive data (secret)](1) is logged here.
@lgtm-com
Copy link

lgtm-com bot commented Sep 23, 2022

This pull request introduces 5 alerts when merging cfbed4e into 78ccbfd - view on LGTM.com

new alerts:

  • 3 for Module is imported with 'import' and 'import from'
  • 1 for Unreachable 'except' block
  • 1 for Clear-text logging of sensitive information

@coveralls
Copy link

Coverage Status

Coverage increased (+0.4%) to 19.345% when pulling e4b6dda on CD-add-stakeholder-full into 78ccbfd on develop.

@lgtm-com
Copy link

lgtm-com bot commented Sep 26, 2022

This pull request introduces 5 alerts when merging e4b6dda into 78ccbfd - view on LGTM.com

new alerts:

  • 3 for Module is imported with 'import' and 'import from'
  • 1 for Unreachable 'except' block
  • 1 for Clear-text logging of sensitive information

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
improvement This issue or pull request will add or improve functionality, maintainability, or ease of use upstream update This issue or pull request pulls in upstream updates
Projects
None yet
Development

Successfully merging this pull request may close these issues.

NER(Named Entity Recognition) stakeholder extraction
2 participants