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

Adding Sysdig connector #1630

Merged

Conversation

thangaraj-ramesh
Copy link
Contributor

No description provided.

thangaraj-ramesh and others added 7 commits October 27, 2023 18:24
Updating log analytics review comments.
1. Added transformer for converting int to float for latitude.
2.Updated TimestampConversion transformer to handle without milliseconds and added mappings for first and last observed.
3. Updated transformer to handle ConfidenceScore value is 'nan'.
Adding Sysdig connector
Enabled cert_verification only when self signed cert is passed
Copy link

codecov bot commented Dec 12, 2023

Codecov Report

Attention: 60 lines in your changes are missing coverage. Please review.

Comparison is base (038129a) 86.01% compared to head (251c4e5) 86.19%.

Files Patch % Lines
...dules/sysdig/stix_translation/query_constructor.py 89.11% 27 Missing ⚠️
...fter_modules/sysdig/stix_transmission/connector.py 88.48% 16 Missing ⚠️
...er_modules/sysdig/stix_translation/transformers.py 71.79% 11 Missing ⚠️
...r_modules/sysdig/stix_transmission/error_mapper.py 85.00% 3 Missing ⚠️
...ests/stix_translation/test_sysdig_stix_to_query.py 98.75% 3 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #1630      +/-   ##
===========================================
+ Coverage    86.01%   86.19%   +0.17%     
===========================================
  Files          572      582      +10     
  Lines        48733    49794    +1061     
===========================================
+ Hits         41919    42920    +1001     
- Misses        6814     6874      +60     

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

changes are below:
1. x_ancestor_names combined as list and moved to x_parent_names under parent process
2. x_pcmdline is moved to command_line of parent process.
thangaraj-ramesh and others added 3 commits January 8, 2024 09:58
Added connector name in logger messages.

Documentation reference mentioned in the doc string.

x_direction field is removed.
Converting severity 1-100 value to data source allowed severity range 0-7 in query constructor.
@mdazam1942 mdazam1942 merged commit e8066a4 into opencybersecurityalliance:develop Jan 11, 2024
5 checks passed
DerekRushton pushed a commit that referenced this pull request Jan 16, 2024
DerekRushton added a commit that referenced this pull request Jul 22, 2024
* CP4S-39527 Initial Translation Code - Draft

* Tanium Threat Response

* Fix Azure log analytics results translation. (#1612)

Updating azure log analytics review comments.
1. Added transformer for converting int to float for latitude.
2.Updated TimestampConversion transformer to handle without milliseconds and added mappings for first and last observed.
3. Updated transformer to handle ConfidenceScore value is 'nan'.

* Bump aioboto3 from 11.3.1 to 12.0.0 in /stix_shifter (#1611)

Bumps [aioboto3](https://github.com/terrycain/aioboto3) from 11.3.1 to 12.0.0.
- [Changelog](https://github.com/terrycain/aioboto3/blob/main/CHANGELOG.rst)
- [Commits](terricain/aioboto3@v11.3.1...v12.0.0)

---
updated-dependencies:
- dependency-name: aioboto3
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump pyopenssl from 23.2.0 to 23.3.0 in /stix_shifter (#1610)

Bumps [pyopenssl](https://github.com/pyca/pyopenssl) from 23.2.0 to 23.3.0.
- [Changelog](https://github.com/pyca/pyopenssl/blob/main/CHANGELOG.rst)
- [Commits](pyca/pyopenssl@23.2.0...23.3.0)

---
updated-dependencies:
- dependency-name: pyopenssl
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* table of mapping script update for to-stix dialects (#1609)

* Bump azure-identity from 1.14.1 to 1.15.0 in /stix_shifter (#1614)

Bumps [azure-identity](https://github.com/Azure/azure-sdk-for-python) from 1.14.1 to 1.15.0.
- [Release notes](https://github.com/Azure/azure-sdk-for-python/releases)
- [Changelog](https://github.com/Azure/azure-sdk-for-python/blob/main/doc/esrp_release.md)
- [Commits](Azure/azure-sdk-for-python@azure-identity_1.14.1...azure-identity_1.15.0)

---
updated-dependencies:
- dependency-name: azure-identity
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump flatten-json from 0.1.13 to 0.1.14 in /stix_shifter (#1613)

Bumps [flatten-json](https://github.com/amirziai/flatten) from 0.1.13 to 0.1.14.
- [Release notes](https://github.com/amirziai/flatten/releases)
- [Commits](https://github.com/amirziai/flatten/commits)

---
updated-dependencies:
- dependency-name: flatten-json
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Update CHANGELOG.md for 6.3.0

* Cisco secure email added readme detailed file. (#1615)

* Added tested communication code for Tanium

* Added suggestions from Azam.

* Fix parameter assignment in error handling function (#1616)

* Remove future timestamp qualifier conditions (#1619)

* Make sure certificate is verified when required by RestApiClientAsync (#1620)

Deprecates selfSignedCert: false bypasss

* Update CHANGELOG.md for 7.0.0

* add email-message translation to ecs (#1621)

* Update group_ref keyword documenation (#1622)

* Initial To Stix mapping - Event and Transformers

* Another temporary commit to hold x-oca-event form

* Finished up the to_stix mapping + test.

* Removed additional event data.

* Fixing the unittest failure

* Another Attempt

* Added the missing fields to the Tanium API response and request.

* Updated toStix and fromStix

* Update CHANGELOG.md for 7.0.1

* second half of email.* mapping for elastic_ecs (#1632)

* Sysdig connector (#1630)

* Update machine ID field in QRadar module (#1634)

Co-authored-by: Kane Brennan <Kane.Brennan@ibm.com>

* Sysdig Connector - Formatting issue in sysdig_supported_stix.md file corrected  (#1635)

* Added the readme (WIP)

* Undid an unintended change.

* Another Attempt to undo the change.

* Removing one more unintended change.

* One more unintended change.

* Updated the sample for the unit test.

* Azam's suggestions.

* Cleaned out the testing code I had left.

* Clean-up - Fixed up the readme.

* Added Azam's suggestions

* Cleaned the Json so it's standardized.

* Removed the total size from the meta data as it's not needed.

* Cleaning up some comments+fixed observation queries.

Signed-off-by: DerekRushton <derek.rushton1@ibm.com>

* Making the config values consistent.

Signed-off-by: DerekRushton <derek.rushton1@ibm.com>

---------

Signed-off-by: dependabot[bot] <support@github.com>
Signed-off-by: DerekRushton <derek.rushton1@ibm.com>
Co-authored-by: thangaraj-ramesh <92723742+thangaraj-ramesh@users.noreply.github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Danny Elliott <danny.elliott@ibm.com>
Co-authored-by: Md Azam <mdazam@ca.ibm.com>
Co-authored-by: Xiaokui Shu <subbyte@gmail.com>
Co-authored-by: Alex-Kidston <113187177+Alex-Kidston@users.noreply.github.com>
Co-authored-by: Kane Brennan <Kane.Brennan@ibm.com>
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.

2 participants