Skip to content

feat(extensions): add Webhook extension for automated RSS notifications#456

Merged
Alkarex merged 17 commits intoFreshRSS:mainfrom
IAM-marco:feat/freshrss-1515-webhook-extension
May 3, 2026
Merged

feat(extensions): add Webhook extension for automated RSS notifications#456
Alkarex merged 17 commits intoFreshRSS:mainfrom
IAM-marco:feat/freshrss-1515-webhook-extension

Conversation

@IAM-marco
Copy link
Copy Markdown
Contributor

This is a copy of #324 with the linting fixed.

All credits to @Ryahn (and @Alkarex for the fixes)


🎯 Purpose

Implements the long-requested webhook feature from issue #1513, allowing users to automatically send webhook notifications when RSS entries match specified keywords.

🚀 Features

  • Automated webhook notifications for matching RSS entries
  • Flexible pattern matching (titles, feeds, authors, content)
  • Support for multiple HTTP methods and request formats
  • Template-based webhook payloads with placeholders
  • Comprehensive error handling and logging
  • Built-in test functionality

🔧 Technical Details

  • Complete PHPDoc documentation
  • Optimized performance with minimal RSS processing impact

🧪 Testing

  • Syntax validation passed
  • Configuration interface tested
  • Webhook functionality verified
  • Error handling validated

📚 Documentation

  • Comprehensive README with examples
  • PHPDoc documentation throughout
  • Configuration guide included
  • Troubleshooting section provided

Resolves FreshRSS/FreshRSS#1513

Alkarex added a commit to Alkarex/FreshRSS that referenced this pull request Apr 19, 2026
@Alkarex
Copy link
Copy Markdown
Member

Alkarex commented Apr 19, 2026

I have made a bunch of changes and will continue later, but tests already welcome.
Will need to merge FreshRSS/FreshRSS#8705 first

Alkarex added a commit to FreshRSS/FreshRSS that referenced this pull request Apr 20, 2026
Comment thread xExtension-Webhook/LICENSE Outdated
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This license is different from the one in the readme.
I suggest removing this file also for uniformity with the rest of the repo

And more simplification
Alkarex added a commit to FreshRSS/FreshRSS that referenced this pull request Apr 28, 2026
@Alkarex Alkarex merged commit 353a7c9 into FreshRSS:main May 3, 2026
1 check passed
@IAM-marco
Copy link
Copy Markdown
Contributor Author

Hey @Alkarex

Apologies for having let this PR go unmaintained.

Thank you for taking care of it, it's really appreciated!

@Alkarex
Copy link
Copy Markdown
Member

Alkarex commented May 3, 2026

No worries, and tests / feedback welcome.
I have tried to make it more uniform with #458

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.

[Feature Request] Share to Webhook

2 participants