Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Upstream revert revert auth token fix #5407

Merged
merged 15 commits into from
May 31, 2024

Merge remote-tracking branch 'upstream/master' into upstream-revert-r…

6ac430b
Select commit
Loading
Failed to load commit list.
Merged

Upstream revert revert auth token fix #5407

Merge remote-tracking branch 'upstream/master' into upstream-revert-r…
6ac430b
Select commit
Loading
Failed to load commit list.
Codecov / codecov/patch failed May 31, 2024 in 0s

34.61% of diff hit (target 61.09%)

View this Pull Request on Codecov

34.61% of diff hit (target 61.09%)

Annotations

Check warning on line 37 in flytectl/pkg/pkce/token_cache_keyring.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flytectl/pkg/pkce/token_cache_keyring.go#L30-L37

Added lines #L30 - L37 were not covered by tests

Check warning on line 41 in flytectl/pkg/pkce/token_cache_keyring.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flytectl/pkg/pkce/token_cache_keyring.go#L40-L41

Added lines #L40 - L41 were not covered by tests

Check warning on line 45 in flytectl/pkg/pkce/token_cache_keyring.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flytectl/pkg/pkce/token_cache_keyring.go#L44-L45

Added lines #L44 - L45 were not covered by tests

Check warning on line 49 in flytectl/pkg/pkce/token_cache_keyring.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flytectl/pkg/pkce/token_cache_keyring.go#L48-L49

Added lines #L48 - L49 were not covered by tests

Check warning on line 54 in flytectl/pkg/pkce/token_cache_keyring.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flytectl/pkg/pkce/token_cache_keyring.go#L53-L54

Added lines #L53 - L54 were not covered by tests

Check warning on line 77 in flytectl/pkg/pkce/token_cache_keyring.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flytectl/pkg/pkce/token_cache_keyring.go#L74-L77

Added lines #L74 - L77 were not covered by tests

Check warning on line 82 in flytectl/pkg/pkce/token_cache_keyring.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flytectl/pkg/pkce/token_cache_keyring.go#L81-L82

Added lines #L81 - L82 were not covered by tests

Check warning on line 130 in flytectl/pkg/pkce/token_cache_keyring.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flytectl/pkg/pkce/token_cache_keyring.go#L123-L130

Added lines #L123 - L130 were not covered by tests

Check warning on line 46 in flyteidl/clients/go/admin/auth_interceptor.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flyteidl/clients/go/admin/auth_interceptor.go#L46

Added line #L46 was not covered by tests

Check warning on line 51 in flyteidl/clients/go/admin/auth_interceptor.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flyteidl/clients/go/admin/auth_interceptor.go#L51

Added line #L51 was not covered by tests

Check warning on line 163 in flyteidl/clients/go/admin/auth_interceptor.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flyteidl/clients/go/admin/auth_interceptor.go#L162-L163

Added lines #L162 - L163 were not covered by tests

Check warning on line 170 in flyteidl/clients/go/admin/auth_interceptor.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flyteidl/clients/go/admin/auth_interceptor.go#L169-L170

Added lines #L169 - L170 were not covered by tests

Check warning on line 178 in flyteidl/clients/go/admin/auth_interceptor.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flyteidl/clients/go/admin/auth_interceptor.go#L176-L178

Added lines #L176 - L178 were not covered by tests

Check warning on line 186 in flyteidl/clients/go/admin/auth_interceptor.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flyteidl/clients/go/admin/auth_interceptor.go#L186

Added line #L186 was not covered by tests

Check warning on line 191 in flyteidl/clients/go/admin/token_source_provider.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flyteidl/clients/go/admin/token_source_provider.go#L191

Added line #L191 was not covered by tests

Check warning on line 237 in flyteidl/clients/go/admin/token_source_provider.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flyteidl/clients/go/admin/token_source_provider.go#L237

Added line #L237 was not covered by tests

Check warning on line 64 in flyteidl/clients/go/admin/tokenorchestrator/base_token_orchestrator.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flyteidl/clients/go/admin/tokenorchestrator/base_token_orchestrator.go#L64

Added line #L64 was not covered by tests

Check warning on line 80 in flyteidl/clients/go/admin/tokenorchestrator/base_token_orchestrator.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flyteidl/clients/go/admin/tokenorchestrator/base_token_orchestrator.go#L80

Added line #L80 was not covered by tests