Skip to content

Conversation

conitrade-as
Copy link

This change set adds the ability to push JSON events into a Valkey / Redis stream. This allows for the decoupling of downstream processors of these events.

@AGWA
Copy link
Member

AGWA commented Oct 3, 2025

Thanks for your interest in improving certspotter! I would like to avoid adding third-party dependencies. Is there a reason this functionality couldn't be implemented as a hook?

@AGWA AGWA added the feature label Oct 3, 2025
@conitrade-as
Copy link
Author

Thanks for your feedback and this great CT log monitoring tool. I can understand that you want to avoid third-party dependencies.

Using a hook works great, as long as your volume of notifications is below a certain level (we are monitoring some cloud applications which issue a lot of certificates). At one point the system becomes saturated and a lot of CPU cycles are used to set up the environment and read from the file system parsing .json files. Thus the reason for using a queuing system comes down to a performance optimization. With this in place, a very modest machine can do all the work required without the overhead of calling an external hook.

May be this functionality could be implemented as a loadable module? https://pkg.go.dev/plugin#pkg-overview However, that may create additional headaches. May be you see another way of implementing the proposed functionality?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants