Skip to content

ari: Only check if cert is not expired, and use proper context#328

Merged
mholt merged 6 commits intomasterfrom
ari-debug
Jan 8, 2025
Merged

ari: Only check if cert is not expired, and use proper context#328
mholt merged 6 commits intomasterfrom
ari-debug

Conversation

@mholt
Copy link
Member

@mholt mholt commented Jan 8, 2025

Using a context that expires after the handshake cancels, for a background task, is no good!

Also avoid unnecessary ARI maintenance, if cert is expired, it'll try to be renewed regardless.

@mholt mholt merged commit 4d5c08f into master Jan 8, 2025
6 checks passed
DennisRasey pushed a commit to DennisRasey/forgejo that referenced this pull request Jan 9, 2025
…#6510)

This PR contains the following updates:

| Package | Type | Update | Change |
|---|---|---|---|
| [github.com/caddyserver/certmagic](https://github.com/caddyserver/certmagic) | require | patch | `v0.21.5` -> `v0.21.6` |

---

### Release Notes

<details>
<summary>caddyserver/certmagic (github.com/caddyserver/certmagic)</summary>

### [`v0.21.6`](https://github.com/caddyserver/certmagic/releases/tag/v0.21.6)

[Compare Source](caddyserver/certmagic@v0.21.5...v0.21.6)

Minor patch for ARI handshake maintenance.

#### What's Changed

-   Fix typos by [@&#8203;deining](https://github.com/deining) in caddyserver/certmagic#324
-   Replace interface{} with any by [@&#8203;deining](https://github.com/deining) in caddyserver/certmagic#325
-   ari: Only check if cert is not expired, and use proper context by [@&#8203;mholt](https://github.com/mholt) in caddyserver/certmagic#328

#### New Contributors

-   [@&#8203;deining](https://github.com/deining) made their first contribution in caddyserver/certmagic#324

**Full Changelog**: caddyserver/certmagic@v0.21.5...v0.21.6

</details>

---

### Configuration

📅 **Schedule**: Branch creation - "* 0-3 * * *" (UTC), Automerge - "* 0-3 * * *" (UTC).

🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update again.

---

 - [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check this box

---

This PR has been generated by [Renovate Bot](https://github.com/renovatebot/renovate).
<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzOS45My4wIiwidXBkYXRlZEluVmVyIjoiMzkuOTMuMCIsInRhcmdldEJyYW5jaCI6ImZvcmdlam8iLCJsYWJlbHMiOlsiZGVwZW5kZW5jeS11cGdyYWRlIiwidGVzdC9ub3QtbmVlZGVkIl19-->

Reviewed-on: https://codeberg.org/forgejo/forgejo/pulls/6510
Reviewed-by: Gusted <gusted@noreply.codeberg.org>
Co-authored-by: Renovate Bot <forgejo-renovate-action@forgejo.org>
Co-committed-by: Renovate Bot <forgejo-renovate-action@forgejo.org>
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.

1 participant