Closed wdbaruni closed 1 week ago
[!CAUTION]
Review failed
The pull request is closed.
The changes in this pull request involve modifications to the pkg/version/update.go
file. The default endpoint for version checks has been updated from an HTTP URL to a more secure HTTPS URL. Additionally, the error handling within the CheckForUpdate
function has been improved by adding a check for the HTTP response status code. This ensures that the function only reads the response body if the request is successful, thus enhancing the robustness of the update checking process.
File | Change Summary |
---|---|
pkg/version/update.go | Updated default endpoint from HTTP to HTTPS for version checks. Improved error handling in CheckForUpdate by adding a status code check before reading the response body. |
defaultUpdateCheckerEndpoint
constant and enhances the error handling in the CheckForUpdate
function, making it closely related to the changes in the main PR.🐇 In the meadow where updates play,
A rabbit hops, "Hooray, hooray!
From HTTP to HTTPS we glide,
Securely now, we take our stride.
With checks in place, no fear to roam,
The latest version finds its home!" 🌼
also introduces better error handling when failing to connect to the update server
Closes ENG-336
Summary by CodeRabbit
New Features
Bug Fixes