Skip to content

fix: bugfix health provider#281

Closed
juandav wants to merge 2 commits intomainfrom
bugfix/health-indicator
Closed

fix: bugfix health provider#281
juandav wants to merge 2 commits intomainfrom
bugfix/health-indicator

Conversation

@juandav
Copy link
Copy Markdown
Member

@juandav juandav commented Feb 24, 2024

No description provided.

juandav added a commit that referenced this pull request Mar 22, 2026
…dling

- Add OnApplicationShutdown to RedisCoreModule for graceful Redis disconnection (PR #289)
- Fix health provider to use configured Redis connection instead of default localhost (PR #281)
- Add null-safe error handling in RedisHealthIndicator (PR #288)
- Export redisHealthIndicatorProvider from index

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@juandav
Copy link
Copy Markdown
Member Author

juandav commented Mar 22, 2026

This health provider bugfix has been incorporated into main (commit 04b2720). The redisHealthIndicatorProvider now properly injects the configured Redis connection instead of creating a new Redis() instance that defaults to localhost.

The provider is also now exported from the package index for consumers who need direct access.

Closing as merged.

@juandav juandav closed this Mar 22, 2026
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.

1 participant