-
Notifications
You must be signed in to change notification settings - Fork 10
Open
Labels
cliRelating to the CLI onlyRelating to the CLI onlylow prioLow priority. Something that doesn't affect core functionality.Low priority. Something that doesn't affect core functionality.
Description
This doesn't matter too much since the CLI runs once and returns, but we are making multiple requests per instance due to the health check that we do prior to sending the "real" request.
Defining http.Client{} once and passing it around would be more efficient (not that it matters too much) but it's more of a code quality thing.
Low prio, mostly writing this so I don't forget about if this become more relevant in the future.
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
cliRelating to the CLI onlyRelating to the CLI onlylow prioLow priority. Something that doesn't affect core functionality.Low priority. Something that doesn't affect core functionality.