Skip to content

flyteplugins/v1.9.17

flyteplugins/v1.9.17: [Monorepo] Enable proxy-authorization in admin client (#4189)
@eapolinario eapolinario tagged this 11 Oct 22:20 
* Add proxyCommand to client config

Signed-off-by: Fabio Grätz <fabiogratz@googlemail.com>

* Add proxy auth unary interceptor

Signed-off-by: Fabio Grätz <fabiogratz@googlemail.com>

* Use proxy auth in http client for oauth

Signed-off-by: Fabio Grätz <fabiogratz@googlemail.com>

* Cache tokens obtained from external commands

Signed-off-by: Fabio Grätz <fabiogratz@googlemail.com>

* Make tests pass

Signed-off-by: Fabio Grätz <fabiogratz@googlemail.com>

* Add tests for proxy auth interceptor

Signed-off-by: Fabio Grätz <fabiogratz@googlemail.com>

* Make work without 2nd token cache but instead with 2nd credentials future

Signed-off-by: Fabio Grätz <fabiogratz@googlemail.com>

* Adapt existing tests to not using a 2nd token cache but a 2nd credentials future

Signed-off-by: Fabio Grätz <fabiogratz@googlemail.com>

* Adapt new tests to not using a 2nd token cache but a 2nd credentials future

Signed-off-by: Fabio Grätz <fabiogratz@googlemail.com>

* Fix number of opts in NewAdminConnection

Signed-off-by: Fabio Grätz <fabiogratz@googlemail.com>

* Don't overwrite original error in NewProxyAuthInterceptor

Signed-off-by: Fabio Grätz <fabiogratz@googlemail.com>

* Improve error message when failing to create http client for oauth

Signed-off-by: Fabio Grätz <fabiogratz@googlemail.com>

* Actually don't return any error from setHTTPClientContext at all as before

Signed-off-by: Fabio Grätz <fabiogratz@googlemail.com>

* Don't require github.com/golang-jwt/jwt anymore

Signed-off-by: Fabio Grätz <fabiogratz@googlemail.com>

* Make tests pass again

Signed-off-by: Fabio Grätz <fabiogratz@googlemail.com>

* Lint

Signed-off-by: Fabio Grätz <fabiogratz@googlemail.com>

* make -C flyteidl generate

Signed-off-by: Eduardo Apolinario <eapolinario@users.noreply.github.com>

* Fix flytepropeller's getAdminClient

Signed-off-by: Fabio Grätz <fabiogratz@googlemail.com>

---------

Signed-off-by: Fabio Grätz <fabiogratz@googlemail.com>
Signed-off-by: Eduardo Apolinario <eapolinario@users.noreply.github.com>
Co-authored-by: Fabio Grätz <fabiogratz@googlemail.com>
Co-authored-by: Eduardo Apolinario <eapolinario@users.noreply.github.com>
Assets 2
Loading