[BBOT] New Integration - Bighuge BLS OSINT Tool ASM Data#9427
[BBOT] New Integration - Bighuge BLS OSINT Tool ASM Data#9427CarsonHrusovsky wants to merge 24 commits intoelastic:mainfrom CarsonHrusovsky:main
Conversation
|
❌ Author of the following commits did not sign a Contributor Agreement: Please, read and sign the above mentioned agreement if you want to contribute to this project |
|
@jamiehynds - Can we check with Black Lantern Security for usage of their logo for the integration? |
|
This is ready to be reviewed and tested. 👍 |
|
Pinging @elastic/security-service-integrations (Team:Security-Service Integrations) |
efd6
left a comment
There was a problem hiding this comment.
Please add a line to .github/CODEOWNERS.
packages/bbot/_dev/build/build.yml
Outdated
| dependencies: | ||
| ecs: | ||
| reference: git@v8.11.0 | ||
| import_mappings: true No newline at end of file |
| volumes: | ||
| - ./sample_logs:/sample_logs:ro | ||
| - ${SERVICE_LOGS_DIR}:/var/log | ||
| command: /bin/sh -c "cp /sample_logs/* /var/log/" No newline at end of file |
packages/bbot/data_stream/asm_intel/_dev/test/pipeline/test-bbot-ndjson.log
Outdated
Show resolved
Hide resolved
| @@ -0,0 +1,3 @@ | |||
| fields: | |||
| tags: | |||
| - preserve_original_event No newline at end of file | |||
packages/bbot/data_stream/asm_intel/elasticsearch/ingest_pipeline/default.yml
Outdated
Show resolved
Hide resolved
packages/bbot/data_stream/asm_intel/elasticsearch/ingest_pipeline/default.yml
Show resolved
Hide resolved
| # ECS Fields # | ||
| ############## | ||
| - rename: | ||
| field: bbot.data.PROTOCOL.port |
| fields: | ||
| - name: data.DNS_NAME | ||
| type: keyword | ||
| description: > |
…descriptions to fields. Updated README.md. Removed READ.me. Added cleanup processor.
|
This is still being worked on. |
|
Okay this is ready for review again. A few points I will make here as far as changes:
|
|
@efd6 Any updates on this? Looks like it's ready for another review. |
efd6
left a comment
There was a problem hiding this comment.
I have left the keys in the format they are in (bbot.data.THISISCAPS.fnding). My reasoning is that within BBOT, the capitalization is a reference for an event type, so having this data is useful for an analyst. If this is something that cannot be ignored and must be lowercased, I will do this however.
From looking at the BBOT docs and the events here, I can't find any cases where they are lower cased. Given this, there is apparently no information provided by the case. If this is the case, please lower-case them. If it's not the case, please point me to documentation explaining the semantics.
packages/bbot/data_stream/asm_intel/_dev/test/pipeline/test-common-config.yml
Outdated
Show resolved
Hide resolved
packages/bbot/validation.yml
Outdated
| @@ -0,0 +1,3 @@ | |||
| errors: | |||
| exclude_checks: | |||
| - SVR00004 # Added search to dashboard. No newline at end of file | |||
|
On the file path, would you recommend that in the integration readme it states an example of executing bbot with an example path and use that? BBOT docs https://www.blacklanternsecurity.com/bbot/scanning/output/ You are correct that the agent is running a different user than the integration which is usually system. Example in the readme today: Perhaps it can be added in docs with the --output to read as: Then the path could align with the example? The problem is in the BBOT docs they drop it by default in the user that ran the scan. |
Co-authored-by: Dan Kortschak <dan.kortschak@elastic.co>
Co-authored-by: Dan Kortschak <dan.kortschak@elastic.co>
…mmon-config.yml Co-authored-by: Dan Kortschak <dan.kortschak@elastic.co>
Co-authored-by: Dan Kortschak <dan.kortschak@elastic.co>
Co-authored-by: Dan Kortschak <dan.kortschak@elastic.co>
Co-authored-by: Dan Kortschak <dan.kortschak@elastic.co>
|
Fields have been lowercased. Marked for review once more :D - Thanks for all the help on this @efd6 |
|
/test |
|
Please fix up the cases where there is no final new line in the files (all except json for generated test expectations and sample events). |
|
/test |
💔 Build Failed
Failed CI StepsHistory
|
|
Please run |
I spoke with Jamie. We're good having our logo used for the integration. I'm one of the principals behind BLS, we appreciate the integration and team work! |
|
Lets give it another go! |
|
Awesome, thank you @TheTechromancer ! Do you happen to have SVG versions of your logos? The icon usage on the integrations website and in Kibana requires an SVG according to the package manifest spec. Unless @efd6 knows of a way to use a PNG for the icon (today you see the rising Phoenix populated in the initial screenshot). |
|
Also, Techro, do y'all have a preference of which logo to use for the integration? It will only be 1 logo that is needed. The Phoenix I referenced earlier is where it will live. Plus it will be here in time: https://www.elastic.co/integrations/data-integrations |
|
Proof of concept currently. |
|
There is a commit here that has a name that has not signed the CLA. Can all the contributors to this PR please check that they have signed with the email addresses that appear in the author email field of the commit they contributed. |
Looks like the local git blew away Carson's user info so it doesn't recognize his email. |
|
@efd6 Is something like this possible? Seems like the dark_mode: tag for icons doesn't work / is unpredictable. |
This reverts commit 698ae39. This commit broke the PR. Attempting to fix.
I think this is not a feature or it is broken. In the Okta Entity Analytics, it is not using the dark mode SVG. icons:
The solution is here is applying a white background to the img which seems trivial to do in SVG. |
…official BBOT svg" This reverts commit a94f1f9. This broke CLI
|
I think this PR is dead. I will open another and mention this one, I am afraid if I keep trying to undo commits I will break something else. |
|








Enhancement
Added BBOT Integration
WHAT: BBOT is an OSINT ASM tool that is available to the general public. The link for this tool can be found here: https://github.com/blacklanternsecurity/bbot
WHY: This allows ingest of ASM findings into Elastic, allowing for review and analysis.
Checklist
changelog.ymlfile.Author's Checklist
Screenshots