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.

Sign up for GitHub

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

Split access token into half and store to avoid "securecookie: the value is too long" error #4863

Merged
merged 7 commits into from
Mar 29, 2024
+154 −19

correct spelling...

3d11035
Select commit
Loading
Failed to load commit list.
Merged

Split access token into half and store to avoid "securecookie: the value is too long" error #4863

correct spelling...
3d11035
Select commit
Loading
Failed to load commit list.
Codecov / codecov/patch succeeded Mar 29, 2024 in 0s

65.78% of diff hit (target 58.56%)

View this Pull Request on Codecov

65.78% of diff hit (target 58.56%)

Annotations

Check warning on line 60 in flyteadmin/auth/cookie_manager.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flyteadmin/auth/cookie_manager.go#L59-L60 

Added lines #L59 - L60 were not covered by tests

Check warning on line 65 in flyteadmin/auth/cookie_manager.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flyteadmin/auth/cookie_manager.go#L64-L65 

Added lines #L64 - L65 were not covered by tests

Check warning on line 69 in flyteadmin/auth/cookie_manager.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flyteadmin/auth/cookie_manager.go#L68-L69 

Added lines #L68 - L69 were not covered by tests

Check warning on line 170 in flyteadmin/auth/cookie_manager.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flyteadmin/auth/cookie_manager.go#L168-L170 

Added lines #L168 - L170 were not covered by tests
View more details on Codecov