Skip to content

Conversation

@parsiya
Copy link
Contributor

@parsiya parsiya commented Dec 7, 2025

I thought having the function name in the message will help with triage either manually or using the message as a filter.

If this is something you want to incorporate, the decision is how you want the message to look like (e.g., where does the function name appear in the message). I am open to the location so it's clearer to users.

If this useful, I can go ahead and work on some of the other rules that have a similar structure.

@0xdea
Copy link
Owner

0xdea commented Dec 13, 2025

Thank you for the PR! It's a nice idea and I didn't know that it was possible with Semgrep. I have to check how this change impacts SARIF Explorer and similar tools to make sure it doesn't break them. I'll get back to you after the holidays, during which I'll finally be AFK for some time. Cheers!

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