Skip to content
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

Support to configure TLS for the redis connection #130

Merged
merged 2 commits into from
Aug 3, 2023

Conversation

dominik-cnx
Copy link
Collaborator

No description provided.

@codecov
Copy link

codecov bot commented Jul 31, 2023

Codecov Report

Patch coverage: 80.00% and project coverage change: +0.02% 🎉

Comparison is base (afb16f3) 64.02% compared to head (9358497) 64.04%.
Report is 8 commits behind head on develop.

❗ Current head 9358497 differs from pull request most recent head b080092. Consider uploading reports for the commit b080092 to get more accurate results

Additional details and impacted files
@@              Coverage Diff              @@
##             develop     #130      +/-   ##
=============================================
+ Coverage      64.02%   64.04%   +0.02%     
- Complexity       329      331       +2     
=============================================
  Files             25       25              
  Lines           2157     2161       +4     
  Branches         263      264       +1     
=============================================
+ Hits            1381     1384       +3     
  Misses           637      637              
- Partials         139      140       +1     
Files Changed Coverage Δ
...rg/swisspush/reststorage/DefaultRedisProvider.java 76.47% <50.00%> (-2.32%) ⬇️
...wisspush/reststorage/util/ModuleConfiguration.java 79.83% <100.00%> (+0.52%) ⬆️

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

@dominik-cnx dominik-cnx merged commit bc309c8 into develop Aug 3, 2023
3 checks passed
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.

3 participants