Skip to content

Add Notify#51

Merged
f8thl3ss merged 2 commits intocaido:mainfrom
MDGDSS:patch-4
Feb 5, 2026
Merged

Add Notify#51
f8thl3ss merged 2 commits intocaido:mainfrom
MDGDSS:patch-4

Conversation

@MDGDSS
Copy link
Contributor

@MDGDSS MDGDSS commented Nov 5, 2025

Added Notify plugin to send findings to notification platforms.

I am submitting a new Plugin Package

Repository URL

Link to my plugin package: https://github.com/MDGDSS/caido-notify

Release Checklist

  • I have tested the plugin on
    • Windows
    • macOS
    • Linux
  • My GitHub release contains all required files
    • plugin_package.zip
    • plugin_package.zip.sig
  • Release immutability is enabled
  • GitHub Tag name matches the version number specified in my caido.config.json
  • The id in my caido.config.json matches the id in the plugin_packages.json file.
  • My README.md describes the plugin package purpose and provides clear usage instructions.
  • I have read the developer policy at https://developer.caido.io/policy.html, and have assessed my plugin package adherence to this policy.
  • I have added a license in the LICENSE file and it matches the license field in the plugin_packages.json file.
  • My project respects and is compatible with the original license of any third-party code that I'm using.
    I have given proper attribution to these other projects in my README.md and/or LICENSE.

Added Notify plugin to send findings to notification platforms.
@f8thl3ss
Copy link
Member

f8thl3ss commented Feb 2, 2026

Hey @MDGDSS, thanks for the plugin! :) I think it still needs a bit of work, but we are almost there! There are a couple of things that I think are needed, mostly around communicating with user when something is wrong. I tried to install and use it to send a request to a webhook, but couldn't make it work. You should use the sdk.window.showToast or, at a minimum, add some log with sdk.console.log when a notification fails. I think it would also be useful to add a test notification feature in the page which lets you configure the providers.

@MDGDSS
Copy link
Contributor Author

MDGDSS commented Feb 2, 2026

Hey @f8thl3ss, thanks for the detailed feedback!
Debugging notifications was a bit rough. I’ve added better logging for failures and a simple test feature to make things easier to troubleshoot.
The plugin was originally built for the CLI version, so logs were either removed or commented out. Now we have both sdk.log and toast notifications.
Let me know if anything’s still confusing or if you’ve got more ideas!

@f8thl3ss f8thl3ss self-requested a review as a code owner February 4, 2026 18:41
@f8thl3ss
Copy link
Member

f8thl3ss commented Feb 4, 2026

Thanks for the improvements! Last thing before we merge this @MDGDSS, can you make sure to enable immutable releases and do a new release ?
https://developer.caido.io/guides/repository.html#_3-enable-immutable-releases

@MDGDSS
Copy link
Contributor Author

MDGDSS commented Feb 4, 2026

Hey @f8thl3ss Thanks, I had forgotten everything should be good now.

@github-actions
Copy link

github-actions bot commented Feb 5, 2026

Plugin Validation Results for Notify v1.0.3

Repository: MDGDSS/caido-notify

✅ Version Match
✅ ID Match
✅ README Present
✅ Required Artifacts
✅ Signature Validation

@f8thl3ss f8thl3ss merged commit 0acb7b3 into caido:main Feb 5, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants