Skip to content

feat: add http-timeout option - #1445

Merged
phm07 merged 3 commits into
mainfrom
http-timeout-option
Jul 2, 2026
Merged

feat: add http-timeout option#1445
phm07 merged 3 commits into
mainfrom
http-timeout-option

Conversation

@phm07

@phm07 phm07 commented Jul 2, 2026

Copy link
Copy Markdown
Contributor

This PR adds a configurable http-timeout option.
Since the default Go HTTP client has a timeout of 0, this is the proposed default for the new option as well to keep behavior consistent. A value of 0 means no timeout.

@phm07 phm07 self-assigned this Jul 2, 2026
@phm07
phm07 requested a review from a team as a code owner July 2, 2026 09:04
@codecov

codecov Bot commented Jul 2, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 66.66667% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 72.86%. Comparing base (b0dd537) to head (949d4e0).
⚠️ Report is 10 commits behind head on main.

Files with missing lines Patch % Lines
internal/state/state.go 66.66% 1 Missing and 1 partial ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1445      +/-   ##
==========================================
+ Coverage   72.80%   72.86%   +0.06%     
==========================================
  Files         300      300              
  Lines       11104    11124      +20     
==========================================
+ Hits         8084     8106      +22     
+ Misses       2120     2119       -1     
+ Partials      900      899       -1     
Flag Coverage Δ
e2e 51.35% <66.66%> (+0.10%) ⬆️
unit 67.75% <0.00%> (+0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Comment thread internal/state/state.go Outdated
Co-authored-by: Jonas L. <jooola@users.noreply.github.com>
Comment thread internal/state/config/options.go Outdated
@phm07
phm07 merged commit 96a1bae into main Jul 2, 2026
8 checks passed
@phm07
phm07 deleted the http-timeout-option branch July 2, 2026 14:48
phm07 pushed a commit that referenced this pull request Jul 24, 2026
<!-- section-start changelog -->
### Removed deprecated Datacenter from Server and Primary IP

Removed the deprecated Datacenter property from the Server and Primary
IP resources. Since the property was already removed from the Hetzner
Cloud API, we do not consider this a breaking change (see [changelog
entry](https://docs.hetzner.cloud/changelog#2026-07-01-removing-datacenters)).

The `hcloud primary-ip create --datacenter` and `hcloud server create
--datacenter` flags have been removed in favor of `--location`.
Additionally, `hcloud primary-ip describe` and `hcloud server describe`
will no longer show the Datacenter property.

In place of the `--datacenter` flag the `--location` flag can be used.
The Location name is the first part of the Datacenter name, e.g. the
part before `-`. Example:

**Before:**

```sh
$ hcloud server create --datacenter fsn1-dc14
$ hcloud primary-ip create --datacenter hel1-dc2
```

**After:**

```sh
$ hcloud server create --location fsn1
$ hcloud primary-ip create --location hel1
```

### Features

- add http-timeout option (#1445)
- remove datacenter property from server and primary_ip (#1460)

### Bug Fixes

- **primary-ip**: format IPv6 properly in describe (#1451)

<!-- section-end changelog -->

---

<details>
<summary><h4>PR by <a
href="https://github.com/apricote/releaser-pleaser">releaser-pleaser</a>
🤖</h4></summary>

If you want to modify the proposed release, add you overrides here. You
can learn more about the options in the docs.

## Release Notes

### Prefix / Start

This will be added to the start of the release notes.

~~~~rp-prefix
### Removed deprecated Datacenter from Server and Primary IP

Removed the deprecated Datacenter property from the Server and Primary
IP resources. Since the property was already removed from the Hetzner
Cloud API, we do not consider this a breaking change (see [changelog
entry](https://docs.hetzner.cloud/changelog#2026-07-01-removing-datacenters)).

The `hcloud primary-ip create --datacenter` and `hcloud server create
--datacenter` flags have been removed in favor of `--location`.
Additionally, `hcloud primary-ip describe` and `hcloud server describe`
will no longer show the Datacenter property.

In place of the `--datacenter` flag the `--location` flag can be used.
The Location name is the first part of the Datacenter name, e.g. the
part before `-`. Example:

**Before:**

```sh
$ hcloud server create --datacenter fsn1-dc14
$ hcloud primary-ip create --datacenter hel1-dc2
```

**After:**

```sh
$ hcloud server create --location fsn1
$ hcloud primary-ip create --location hel1
```
~~~~

### Suffix / End

This will be added to the end of the release notes.

~~~~rp-suffix
~~~~

</details>

Co-authored-by: Hetzner Cloud Bot <>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants