aporeto-inc / trireme-lib

Simple, scalable and secure application segmentation
https://trireme.io
Apache License 2.0
300 stars 51 forks source link

fix: update token cache considering the cert expiry #961

Closed sibicramesh closed 4 years ago

sibicramesh commented 4 years ago
// We cache the token only If we are atleast 20 seconds (SynTokenCacheValiditity + 10s)
// away from expiration. This fixes the corner case when the cert is expired and
// we still use the token we cached earlier.
codecov[bot] commented 4 years ago

Codecov Report

Merging #961 into master will decrease coverage by 0.15%. The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #961      +/-   ##
==========================================
- Coverage   54.46%   54.31%   -0.16%     
==========================================
  Files         122      122              
  Lines       11724    11724              
==========================================
- Hits         6386     6368      -18     
- Misses       4710     4733      +23     
+ Partials      628      623       -5
Impacted Files Coverage Δ
monitor/extractors/linux.go 48.78% <0%> (-21.96%) :arrow_down:

Continue to review full report at Codecov.

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

sibicramesh commented 4 years ago

/build - automatically fired by gogo with following PRs and commit SHAs v1.0.0

[
  {
    "project": "",
    "component": "enforcerd",
    "pr-id": "1575",
    "commit-sha": "2b753a4e5825f2a824c28116b241857294c89a5c",
    "pipeline": "master"
  },
  {
    "project": "",
    "component": "trireme-lib",
    "pr-id": "961",
    "commit-sha": "30c9bf44d261f1ea3b88f6114d8e728529388cef",
    "pipeline": "master"
  }
]