-
Notifications
You must be signed in to change notification settings - Fork 52
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
update Plugin to work with new Notification/Alert System in Graylog 3.1 #89
Comments
is not working with Graylog 3.1.2 as well: |
I'm sorely tempted to have a go at this, it would be extremely useful for us (the existing version seems to duplicate some backlog items in 3.1). This would be my first foray into Graylog development, is there an example 3.1 alert plugin that anyone can point me at? |
@tabascoterrier That would be awesome! 😃 There is no external plugin yet but you can take a look at the HTTP event notification in the server repository. All the internal notifications are implemented as plugins as well: |
Thanks @bernd, I'll have a look! |
As i had the same problem i simply wrote a new slack plugin for graylog 3.1 using the new notification system. https://marketplace.graylog.org/addons/d87f19f5-4a91-4099-8179-c4700b6c8f6d |
Is anyone working on this? If not I will take care of it as I really need it |
@radykal-com Feel free to work on this! 😃 Thank you! 👍 |
@radykal-com I would recommend you the plugin I wrote and mentioned three comments above. |
@sboeingSPA why dont contribute your changes back to the original plugin? Its much easier to maintain one plugin instead of 2 different ones that actually are doing the same. |
Because all the code of the original plugin is deprecated. Meaning i would deleted all of it and replace it with new one. But the original plugin currently supports Mattermost which i don't know and don't want to explicitly support and test. Also this project looks quite unmaintained to me. The last release is more than a year old and there are merge requests no one takes care of. |
@sboeingSPA Please feel free to submit your work as a PR, regardless how much code you changed. I do my best to check in on this repo from time to time for critical issues, but I have a full time career not affiliated with Graylog and a family. There is only so much time I can contribute. If you would like to take over repository maintenance, I’m happy to pass the torch. |
@sboeingSPA @Aenima4six2 Thanks folks for taking care! 👍 We don't need to delete the legacy plugin. I think it's good to keep it until everyone migrated off to the new event notification. So it would be enough to just add the new event notification code to the existing structure. |
@bernd @Aenima4six2 i will take another look into it the next weeks during my vacation |
Hi everyone, Any news on this? We miss this plugin in Graylog :-/ |
This Plugin needs to be updated to work with the new alerting/notification system.
The text was updated successfully, but these errors were encountered: