shotover / shotover-proxy

L7 data-layer proxy
https://docs.shotover.io
Apache License 2.0
82 stars 16 forks source link

#1667 Remove cached delegation token from AuthorizeScramOverMtls #1683

Closed justinweng-instaclustr closed 2 weeks ago

justinweng-instaclustr commented 2 weeks ago

This PR removes the field delegation_token from the struct AuthorizeScramOverMtls to avoid caching the token in AuthorizeScramOverMtls instances. Instead, we store the username used to generate delegation tokens in this struct and fetch the delegation token from the TokenTask every time it's needed.

Progress towards #1667

codspeed-hq[bot] commented 2 weeks ago

CodSpeed Performance Report

Merging #1683 will not alter performance

Comparing justinweng-instaclustr:1667-avoid-caching-scram_over_mtls (511ba93) with main (841ec95)

Summary

✅ 39 untouched benchmarks