Skip to content

SK-3133: Use the generated RetryInterceptor + fix logging/dotenv latency bottlenecks - #424

Open
Devesh-Skyflow wants to merge 3 commits into
mainfrom
devesh/sk-3133
Open

SK-3133: Use the generated RetryInterceptor + fix logging/dotenv latency bottlenecks#424
Devesh-Skyflow wants to merge 3 commits into
mainfrom
devesh/sk-3133

Make console logging async and memoize .env lookups on the request ho…

82d0cc7
Select commit
Loading
Failed to load commit list.
Codecov / codecov/project succeeded Sep 8, 2026 in 1s

91.89% (-0.11%) compared to d0dea2f

View this Pull Request on Codecov

91.89% (-0.11%) compared to d0dea2f

Details

Codecov Report

❌ Patch coverage is 85.26316% with 14 lines in your changes missing coverage. Please review.
✅ Project coverage is 91.89%. Comparing base (d0dea2f) to head (82d0cc7).

Files with missing lines Patch % Lines
...mon/src/main/java/com/skyflow/utils/BaseUtils.java 73.91% 2 Missing and 4 partials ⚠️
.../com/skyflow/utils/logger/AsyncConsoleHandler.java 87.09% 3 Missing and 1 partial ⚠️
.../com/skyflow/utils/logger/AsyncConsoleHandler.java 87.09% 3 Missing and 1 partial ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##               main     #424      +/-   ##
============================================
- Coverage     91.99%   91.89%   -0.11%     
- Complexity        0      493     +493     
============================================
  Files           158      159       +1     
  Lines          6631     6671      +40     
  Branches        893      890       -3     
============================================
+ Hits           6100     6130      +30     
- Misses          351      358       +7     
- Partials        180      183       +3     
Components Coverage Δ
Module: common 88.19% <82.14%> (-0.23%) ⬇️
Module: skyvault 94.86% <87.87%> (-0.09%) ⬇️
Module: flowvault 90.21% <100.00%> (-0.05%) ⬇️
Service Account 87.36% <ø> (ø)
Vault Data 92.62% <ø> (ø)
Vault Tokens 99.03% <ø> (ø)
Vault Connection 100.00% <ø> (ø)
Vault Controller 85.47% <100.00%> (+0.11%) ⬆️
Detect 100.00% <ø> (ø)
Audit 100.00% <ø> (ø)
BIN Lookup 100.00% <ø> (ø)
Config 96.26% <ø> (ø)
Utils 90.34% <84.09%> (-0.45%) ⬇️
Errors 100.00% <ø> (ø)
Enums 100.00% <ø> (ø)
Logs 95.60% <ø> (ø)
Files with missing lines Coverage Δ
...mon/src/main/java/com/skyflow/BaseVaultClient.java 87.75% <100.00%> (+1.21%) ⬆️
...rc/main/java/com/skyflow/utils/logger/LogUtil.java 83.01% <100.00%> (ø)
...owvault/src/main/java/com/skyflow/VaultClient.java 97.53% <100.00%> (+0.09%) ⬆️
...owvault/src/main/java/com/skyflow/utils/Utils.java 92.78% <100.00%> (+0.15%) ⬆️
.../com/skyflow/vault/controller/VaultController.java 77.54% <100.00%> (+0.15%) ⬆️
...lt/src/main/java/com/skyflow/ConnectionClient.java 89.13% <100.00%> (-0.24%) ⬇️
...rc/main/java/com/skyflow/utils/logger/LogUtil.java 79.24% <100.00%> (ø)
.../com/skyflow/utils/logger/AsyncConsoleHandler.java 87.09% <87.09%> (ø)
.../com/skyflow/utils/logger/AsyncConsoleHandler.java 87.09% <87.09%> (ø)
...mon/src/main/java/com/skyflow/utils/BaseUtils.java 76.85% <73.91%> (-0.70%) ⬇️

Continue to review full report in Codecov by Harness.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d0dea2f...82d0cc7. Read the comment docs.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.