Skip to content

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

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

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

Ryahn
Copy link

@Ryahn Ryahn commented May 30, 2025

🎯 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 #1513

Ryahn added 2 commits May 30, 2025 16:48
…ns - Implements webhook notifications for RSS entries matching keywords - Supports pattern matching in titles, feeds, authors, and content - Configurable HTTP methods and request formats - Comprehensive error handling and logging - Resolves #1513
@Alkarex
Copy link
Member

Alkarex commented May 31, 2025

FreshRSS/FreshRSS#1513

@Alkarex
Copy link
Member

Alkarex commented May 31, 2025

Thanks @Ryahn 👍🏻
I have made a few fixes. A bit more would be good - see automated tests (or run make test-all)

@Ryahn
Copy link
Author

Ryahn commented Jun 1, 2025

Thanks @Ryahn 👍🏻

I have made a few fixes. A bit more would be good - see automated tests (or run make test-all)

Will make more fixes in the next day or 2.

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.

2 participants