Adam Kafka

Results 17 comments of Adam Kafka

I had this issue as well, but I noticed that my version of `docker-credential-ecr-login` was old: ``` $ docker-credential-ecr-login -v amazon-ecr-credential-helper Version: 0.3.1 Git commit: b4a1707 ``` I updated the...

I had this issue as well, but I noticed that my version of `docker-credential-ecr-login` was old: ``` $ docker-credential-ecr-login -v amazon-ecr-credential-helper Version: 0.3.1 Git commit: b4a1707 ``` I updated the...

Hmm, it appears I found a better workaround. If I remove the "Single Page Application" completely in Azure, and instead use a "Mobile and desktop applications", the Client Secret is...

Revisiting this now... I learned recently that Azure AD applications of type `Mobile and desktop applications` are not in scope for Conditional Access Policies (see documentation [here](https://learn.microsoft.com/en-us/entra/identity/conditional-access/concept-conditional-access-cloud-apps#other-applications)), meaning in my...

In order to implement this code, I think we need to implement the change 2 libraries upstream. This library uses https://github.com/int128/oauth2cli/tree/master which in turn uses https://github.com/golang/oauth2/tree/master. The logic to actually...

I am getting the same error as @oskarm93 when using `kubelogin` with an Azure AD "Single Page Application" and PKCE. I believe the error can be resolved by including an...

> Out of these I think 1, 2, 3, and 5 are reasonable. I think 1 and 5 have the most predictable behaviour. 2 feels like it doesn't offer a...

Hey @alowde, I see this PR hasn't had any changes for a while now. What work remains? Anything I can do to help? I would love to use `least_connections` soon...

@alowde Great to hear :) I have been thinking a bit about the conversation thread we had about the downsides to this algorithm: https://github.com/kubernetes/ingress-nginx/pull/9025#issuecomment-1304728049. Specifically, the issue that with a...

> I'd like to leave this PR as is and then raise a new PR to either change the behaviour of the least_connections algorithm or (my preference) add a new...