Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

certmagic : Change ManageSync() to take a context #18119

Closed
azhai opened this issue Dec 29, 2021 · 0 comments · Fixed by #18138
Closed

certmagic : Change ManageSync() to take a context #18119

azhai opened this issue Dec 29, 2021 · 0 comments · Fixed by #18138
Labels
topic/build PR changes how Gitea is built, i.e. regarding Docker or the Makefile

Comments

@azhai
Copy link

azhai commented Dec 29, 2021

Gitea Version

e4e3df6

Git Version

2.34

Operating System

CentOS 7

How are you running Gitea?

In go 1.18 beta1
github.com/caddyserver/certmagic v0.15.2

Database

No response

Can you reproduce the bug on the Gitea demo site?

No

Log Gist

No response

Description

certmagic v0.15.2 Change ManageSync() to take a context

import "context"
#cmd/web_letsencrypt.go line 50
err := magic.ManageSync(context.Background(), []string{domain})

Screenshots

No response
@lunny lunny added the topic/build PR changes how Gitea is built, i.e. regarding Docker or the Makefile label Dec 31, 2021
@lunny lunny mentioned this issue Dec 31, 2021
Merged
@go-gitea go-gitea locked and limited conversation to collaborators Apr 28, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
topic/build PR changes how Gitea is built, i.e. regarding Docker or the Makefile
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Upgrade certmagic from v0.14.1 to v0.15.2 lunny/gitea
2 participants
@azhai @lunny