#6603 Update Flyte to Go 1.24
Pull request opened by
Sovietaced
## Why are the changes needed?
New versions of go include new features, performance improvements, and security fixes.
## What changes were proposed in this pull request?
Updates Flyte to use Go 1.24. Changes can be summarized as follows:
1. Go 1.24 introduces additional checks with
go vet which get upset when constants are passed to methods designed to take string formats. Most of the code changes address go vet errors.
2. Updates mockery to a version that supports Go 1.24. As such generated code has minor changes
3. They changed the behavior such that seeding rand is a no-op. I decided to refactor CSRF tokens with increased entropy (10 bytes -> 32 bytes) and decided to use a more secure random. Also, I eliminated seedings the CSRF token based on the wall clock timestamp so the values are less predictable for security. This is a change in behavior, but one that should be transparent to the user.
## How was this patch tested?
Not tested yet but I can land this in Stack AV's fork.
## Check all the applicable boxes
• I updated the documentation accordingly.
• All new and existing tests passed.
• All commits are signed-off.
## Related PRs
## Docs link
https://bito.ai#summarystart|
### Summary by Bito
• This pull request updates the Flyte project to Go version 1.24, addressing various security and compatibility issues, including refactoring CSRF token generation for improved security compliance.
• The mockery dependency has been updated to align with Go 1.24 standards, ensuring that generated code remains functional and secure.
• The pull request resolves go vet errors across multiple files, enhancing error handling and overall project stability.
• Overall, the changes touch on security practices, dependency updates, and error handling improvements, ensuring compliance with the latest Go standards and enhancing project robustness.
flyteorg/flyte