Skip to content

Feat/vulnerability correlation#426

Open
0x0elliot wants to merge 4 commits into
mainfrom
feat/vulnerability-correlation
Open

Feat/vulnerability correlation#426
0x0elliot wants to merge 4 commits into
mainfrom
feat/vulnerability-correlation

Conversation

@0x0elliot

Copy link
Copy Markdown
Member

No description provided.

@socket-security

socket-security Bot commented Jul 7, 2026

Copy link
Copy Markdown

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff Package Supply Chain
Security
Vulnerability Quality Maintenance License
Addedgithub.com/​microcosm-cc/​bluemonday@​v1.0.27100100100100100

View full report

Comment thread blobs.go Outdated

@frikky frikky left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

come on...

Comment thread blobs.go
// That means each algorithm needs to be written as if-statements to
// replace a specific part of a workflow :thinking:

func getVulnerabilityCorrelationWorkflow(actionType, orgId, startActionId, actionEnv string, categoryAction CategoryAction) Workflow {

@frikky frikky Jul 25, 2026

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

The whole point is that putting a single Python script within a bunch of Golang code is absolutely horrendous.

Do not write code without looking at how things are already please. You are not the first to do this, so we have a structure already. If you disagree, at least tell me, so we can find the correct solution and make it identical everywhere. This is just sloppy.

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