Skip to content

Conversation

seer-by-sentry[bot]
Copy link

👋 Hi there! This PR was automatically generated by Autofix 🤖

This fix was triggered by Philip Niedertscheider

Fixes ONLAUNCH-1F

  • Introduces a RedisConnectionStatus singleton to track Redis connection state.
  • Adds explicit connection and command timeouts to Redis client options.
  • Modifies retry strategy to be more conservative and reduces max retries.
  • Disables ready check for faster connection.
  • Adds connection event handlers for connect, error, ready, and close events.
  • Modifies global singleton initialization to be more resilient with initialization retry and timeout.
  • Returns a disabled state instead of throwing an error if Redis instance creation fails.

If you have any questions or feedback for the Sentry team about this fix, please email autofix@sentry.io with the Run ID: 4549.

Copy link

codecov bot commented Feb 11, 2025

Codecov Report

Attention: Patch coverage is 16.66667% with 30 lines in your changes missing coverage. Please review.

Project coverage is 6.38%. Comparing base (4c30f1d) to head (d46c14e).

Files with missing lines Patch % Lines
src/services/redis.ts 26.08% 17 Missing ⚠️
src/services/redis-status.ts 0.00% 13 Missing ⚠️
Additional details and impacted files
@@           Coverage Diff            @@
##            main   #1505      +/-   ##
========================================
+ Coverage   6.34%   6.38%   +0.03%     
========================================
  Files        304     305       +1     
  Lines       6238    6267      +29     
  Branches     678     680       +2     
========================================
+ Hits         396     400       +4     
- Misses      5836    5861      +25     
  Partials       6       6              
Flag Coverage Δ
unittests 6.38% <16.66%> (+0.03%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@philprime philprime self-assigned this Feb 11, 2025
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