Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feature request : Allow limiting by topic #9

Open
shanness opened this issue Oct 16, 2018 · 4 comments
Open

Feature request : Allow limiting by topic #9

shanness opened this issue Oct 16, 2018 · 4 comments

Comments

@shanness
Copy link

I've got some use cases (say notifications with lights, noise, and phone push messages) where I want to rate limit, but only per topic.

Would you consider adding a "group by" or similar that you can set to topic or any field?

Without this, it's very messy, having to break out in my reusable alert flow to one throttle node per topic, then back into the flow, and maintain all the possible callers.

Cheers

@shanness
Copy link
Author

shanness commented Nov 9, 2018

I ended up building my own that does that, just a simple node up on the flow site.

https://flows.nodered.org/flow/80390da72f235b760f853a4bda79668b

@meeki007
Copy link

have you looked at node-red-contrib-topic-timeframe-trigger
https://flows.nodered.org/node/node-red-contrib-topic-timeframe-trigger

@shanness
Copy link
Author

shanness commented Nov 11, 2018

Ahh, no, hadn't, thanks, looks pretty ideal!

This looks pretty good too

https://flows.nodered.org/node/node-red-contrib-timeframerlt

@shanness
Copy link
Author

The one you mentioned dies with a weird error I couldn't debug, and doesn't allow passing on the topic and other data (only fixed stuff), so I've updated mine to have a required min count.

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

No branches or pull requests

2 participants