search

kubeguard / guard

🔑 Kubernetes Authentication & Authorization WebHook Server
https://kubeguard.dev
Apache License 2.0
590 stars 81 forks source link

Use retryable http client in Azure authz provider #372

Closed bingosummer closed 5 months ago

bingosummer commented 1 year ago

We see the timeout error in the checkaccess requrests sometimes.

Error occured during authorization check. Please retry again. Error: Checkaccess requests have timed out.: context deadline exceeded

This PR moves withRetryableHttpClient from Azure authn provider to utils, and leverage it in Azure authz provider's RefreshToken and CheckAccess.

bingosummer commented 1 year ago

@weinong @AzureMarker could you please have a review? Thanks!

bingosummer commented 9 months ago

@AzureMarker Could you please have a review again? Thanks

bingosummer commented 7 months ago

@weinong could you please have a review? Thanks

bingosummer commented 6 months ago

The PR is rebased. @weinong could you please have a review? Please let me know if anything is missing. Thanks!

bingosummer commented 5 months ago

@weinong could you please have another round of review? Thanks

weinong commented 5 months ago

@julienstroheker PTAL