Skip to content

Neo4j fix - serialize host_metadata#3060

Merged
TheTechromancer merged 3 commits intoblacklanternsecurity:devfrom
cnnrshd:neo4j-fix
May 1, 2026
Merged

Neo4j fix - serialize host_metadata#3060
TheTechromancer merged 3 commits intoblacklanternsecurity:devfrom
cnnrshd:neo4j-fix

Conversation

@cnnrshd
Copy link
Copy Markdown
Contributor

@cnnrshd cnnrshd commented Apr 25, 2026

Add "host_metadata" to the nested field serialization loop. Prior to this, I was getting Neo4j insertion failures, seemingly on cloud providers:

Neo.ClientError.Statement.TypeError: Property values can only be of primitive types or arrays thereof

The error looked like:

2026-04-24 13:06:16,306 [ERROR] bbot.modules.output.neo4j base.py:1738 Error inserting Neo4j nodes (label:DNS_NAME): {neo4j_code: Neo.ClientError.Statement.TypeError} {message: Property values can only be of primitive types or arrays thereof. Encountered: Map{1.1.1.1 -> Map{cloud_providers -> Map{microsoft -> Map{match -> String("ip"), types -> List{String("cloud")}}, github -> Map{match -> String("ip"), types -> List{String("cdn")}}}}, 1.1.1.2 -> Map{cloud_providers -> Map{microsoft -> Map{match -> String("ip"), types -> List{String("cloud")}}, github -> Map{match -> String("ip"), types -> List{String("cdn")}}}}}.} {gql_status: 22G03} {gql_status_description: error: data exception - invalid value type}

Verified that this fix works locally by cloning the branch on 3.0, patching the file, building a docker container, and running another scan.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 25, 2026


Thank you for your submission, we really appreciate it. Like many open-source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution. You can sign the CLA by just posting a Pull Request Comment same as the below format.


I have read the CLA Document and I hereby sign the CLA


1 out of 2 committers have signed the CLA.
✅ (cnnrshd)[https://github.com/cnnrshd]
@TheTechromancer
You can retrigger this bot by commenting recheck in this Pull Request. Posted by the CLA Assistant Lite bot.

@cnnrshd
Copy link
Copy Markdown
Contributor Author

cnnrshd commented Apr 25, 2026

I have read the CLA Document and I hereby sign the CLA

bls-cla-bot Bot added a commit to blacklanternsecurity/CLA that referenced this pull request Apr 25, 2026
@TheTechromancer TheTechromancer changed the base branch from 3.0 to dev April 27, 2026 16:16
@TheTechromancer
Copy link
Copy Markdown
Collaborator

recheck

@TheTechromancer TheTechromancer self-requested a review April 27, 2026 16:17
@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 27, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 91%. Comparing base (45aa36c) to head (7d01014).
⚠️ Report is 9 commits behind head on dev.

Additional details and impacted files
@@          Coverage Diff          @@
##             dev   #3060   +/-   ##
=====================================
+ Coverage     91%     91%   +1%     
=====================================
  Files        437     437           
  Lines      37431   37431           
=====================================
+ Hits       33874   33880    +6     
+ Misses      3557    3551    -6     

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@TheTechromancer TheTechromancer self-assigned this Apr 30, 2026
@TheTechromancer
Copy link
Copy Markdown
Collaborator

recheck

@TheTechromancer TheTechromancer merged commit 1f93c7a into blacklanternsecurity:dev May 1, 2026
16 of 19 checks passed
@cnnrshd cnnrshd deleted the neo4j-fix branch May 1, 2026 15:22
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