Skip to content

Navigation Menu

Appearance settings

Explore
By company size
By use case
By industry
View all solutions
Topics
- AI
- DevOps
- Security
- Software Development
- View all
Explore
- GitHub Sponsors
  Fund open source developers
- The ReadME Project
  GitHub community articles
Repositories
- Enterprise platform
  AI-powered developer platform
Available add-ons
Pricing

Search code, repositories, users, issues, pull requests...

Search

Clear

Search syntax tips

Provide feedback

We read every piece of feedback, and take your input very seriously.

Include my email address so I can be contacted

Saved searches

Use saved searches to filter your results more quickly

Name

Query

To see all available qualifiers, see our documentation.

Appearance settings

You signed in with another tab or window. Reload to refresh your session. You signed out in another tab or window. Reload to refresh your session. You switched accounts on another tab or window. Reload to refresh your session.

Dismiss alert

redis / go-redis Public

Notifications You must be signed in to change notification settings
Fork 2.5k
Star 21k

Code
Issues 62
Pull requests 38
Discussions
Actions
Projects
Security
Insights

Additional navigation options

Code
Issues
Pull requests
Discussions
Actions
Projects
Security
Insights

[wip] resp3 notification handlers#3418

Open

ndyakov wants to merge 30 commits intomasterredis/go-redis:masterfrom

ndyakov/CAE-1088-resp3-notification-handlersredis/go-redis:ndyakov/CAE-1088-resp3-notification-handlersCopy head branch name to clipboard

Conversation Commits30 (30)Checks Files changed

Open

[wip] resp3 notification handlers#3418
ndyakov wants to merge 30 commits intomasterfrom
ndyakov/CAE-1088-resp3-notification-handlers

Commits

Commits on Jun 26, 2025

feat: add general push notification system
ndyakov
committed
feat: enforce single handler per notification type
ndyakov
committed
feat: remove global handlers and enable push notifications by default
ndyakov
committed
feat: fix connection health check interference with push notifications
ndyakov
committed
fix: remove unused fields and ensure push notifications work in cloned clients
ndyakov
committed
test: add comprehensive unit tests for 100% coverage
ndyakov
committed
refactor: simplify push notification interface
ndyakov
committed
fix: resolve data race in PushNotificationProcessor
ndyakov
committed
remove: push-notification-demo
ndyakov
committed
feat: add protected handler support and rename command to pushNotificationName
ndyakov
committed
feat: add VoidPushNotificationProcessor for disabled push notifications
ndyakov
committed
refactor: remove unnecessary enabled field and IsEnabled/SetEnabled methods
ndyakov
committed
fix: ensure push notification processor is never nil in newConn
ndyakov
committed
fix: initialize push notification processor in SentinelClient
ndyakov
committed
fix: copy push notification processor to transaction baseClient
ndyakov
committed

Commits on Jun 27, 2025

fix: initialize push notification processor in NewFailoverClient
ndyakov
committed
feat: add GetHandler method and improve push notification API encapsulation
ndyakov
committed
feat: enable real push notification processors for SentinelClient and FailoverClient
ndyakov
committed
feat: remove GetRegistry from PushNotificationProcessorInterface for better encapsulation
ndyakov
committed
fix: add nil reader check in ProcessPendingNotifications to prevent panic
ndyakov
committed
refactor: move push notification logic to pusnotif package
ndyakov
committed
refactor: remove handlerWrapper and use separate maps in registry
ndyakov
committed
Fixes tests:
ndyakov
committed
fix: update coverage test to expect errors for disabled push notifications
ndyakov
committed
fix: address pr review
ndyakov
committed
fix: simplify api
ndyakov
committed
test: add comprehensive test coverage for pushnotif package
ndyakov
committed
feat: add proactive push notification processing to WithReader
ndyakov
committed
feat: add pub/sub message filtering to push notification processor
ndyakov
committed
feat: expand notification filtering to include streams, keyspace, and client tracking
ndyakov
committed

Footer

© 2025 GitHub, Inc.

Footer navigation

Terms
Privacy
Security
Status
Docs
Contact

You can’t perform that action at this time.