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.

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

CrowdStrike Logscale UDI Connector #1631

Conversation

SharmilaMS-Hcl
Copy link
Contributor

Added UDI connector module for CrowdStrike Logscale data source.

Added UDI connector for CrowdStrike Logscale data source
Copy link

codecov bot commented Dec 27, 2023

Codecov Report

Attention: Patch coverage is 96.38932% with 46 lines in your changes are missing coverage. Please review.

Project coverage is 86.52%. Comparing base (3bae101) to head (184636f).
Report is 3 commits behind head on develop.

❗ Current head 184636f differs from pull request most recent head b440a9a. Consider uploading reports for the commit b440a9a to get more accurate results

Files Patch % Lines
...ike_logscale/stix_translation/query_constructor.py 94.20% 20 Missing ⚠️
...ke_logscale/stix_transmission/results_connector.py 94.81% 7 Missing ⚠️
...wdstrike_logscale/stix_translation/transformers.py 75.00% 6 Missing ⚠️
...ike_logscale/stix_transmission/delete_connector.py 83.33% 5 Missing ⚠️
...rike_logscale/stix_transmission/query_connector.py 87.87% 4 Missing ⚠️
...dstrike_logscale/stix_transmission/error_mapper.py 90.00% 2 Missing ⚠️
...trike_logscale/stix_transmission/ping_connector.py 96.29% 1 Missing ⚠️
...slation/test_crowdstrike_logscale_stix_to_query.py 99.50% 1 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #1631      +/-   ##
===========================================
+ Coverage    86.26%   86.52%   +0.25%     
===========================================
  Files          586      600      +14     
  Lines        50040    51314    +1274     
===========================================
+ Hits         43166    44397    +1231     
- Misses        6874     6917      +43     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Updated the code to fix python 3.8 test failure
Added a boolean parameter - combined_observation to handle different qualifiers to handle filter functions.
SharmilaMS-Hcl and others added 3 commits January 25, 2024 10:01
1. Updated the dialect name crowdstrik_edr to crowdstrikeedr in mapping files.
2. Updated unit test case files, config file, read me, entry point files  with respect to change in dialect name.
mdazam1942 and others added 3 commits February 6, 2024 09:14
Modified the transformer of MAC- address and updated the config files with respect to repository.
Changes corresponding to config has been done in transmit files and read-me files.
Updated except class to reuse the exception_response function in connector.py file
SharmilaMS-Hcl and others added 4 commits February 19, 2024 15:20
updated transformer and readme file based on review comments
Updated the error codes in read me and added back the connector extension details.
SharmilaMS-Hcl and others added 4 commits March 12, 2024 08:47
added connector name in logger error
Updated the connector with Pagination logic and updated unit test cases, readme files.
Copy link
Collaborator

@DerekRushton DerekRushton left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The code looks good. Thanks for taking the time to implement the suggested changes. I'll approve the PR.

@DerekRushton DerekRushton merged commit af474b6 into opencybersecurityalliance:develop Apr 24, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants