mirror of
https://github.com/acme-dns/acme-dns.git
synced 2026-02-22 01:25:36 -07:00
* Refactor core * Re-added tests * Small fixes * Add tests for acmetxt cidrslice and util funcs * Remove the last dangling reference to old logging package * Refactoring (#327) * chore: enable more linters and fix linter issues * ci: enable linter checks on all branches and disable recurring checks recurring linter checks don't make that much sense. The code & linter checks should not change on their own over night ;) * chore: update packages * Revert "chore: update packages" This reverts commit 30250bf28c4b39e9e5b3af012a4e28ab036bf9af. * chore: manually upgrade some packages * Updated dependencies, wrote changelog entry and fixed namespace for release * Refactoring - improving coverage (#371) * Increase code coverage in acmedns * More testing of ReadConfig() and its fallback mechanism * Found that if someone put a '"' double quote into the filename that we configure zap to log to, it would cause the the JSON created to be invalid. I have replaced the JSON string with proper config * Better handling of config options for api.TLS - we now error on an invalid value instead of silently failing. added a basic test for api.setupTLS() (to increase test coverage) * testing nameserver isOwnChallenge and isAuthoritative methods * add a unit test for nameserver answerOwnChallenge * fix linting errors * bump go and golangci-lint versions in github actions * Update golangci-lint.yml Bumping github-actions workflow versions to accommodate some changes in upstream golanci-lint * Bump Golang version to 1.23 (currently the oldest supported version) Bump golanglint-ci to 2.0.2 and migrate the config file. This should resolve the math/rand/v2 issue * bump golanglint-ci action version * Fixing up new golanglint-ci warnings and errors --------- Co-authored-by: Joona Hoikkala <5235109+joohoi@users.noreply.github.com> * Minor refactoring, error returns and e2e testing suite * Add a few tests * Fix linter and umask setting * Update github actions * Refine concurrency configuration for GitHub actions * HTTP timeouts to API, and self-validation mutex to nameserver ops --------- Co-authored-by: Florian Ritterhoff <32478819+fritterhoff@users.noreply.github.com> Co-authored-by: Jason Playne <jason@jasonplayne.com>
56 lines
1.6 KiB
Go
56 lines
1.6 KiB
Go
package api
|
|
|
|
import (
|
|
"net/http"
|
|
|
|
"github.com/joohoi/acme-dns/pkg/acmedns"
|
|
|
|
"github.com/julienschmidt/httprouter"
|
|
)
|
|
|
|
func (a *AcmednsAPI) webUpdatePost(w http.ResponseWriter, r *http.Request, _ httprouter.Params) {
|
|
var updStatus int
|
|
var upd []byte
|
|
// Get user
|
|
atxt, ok := r.Context().Value(ACMETxtKey).(acmedns.ACMETxt)
|
|
if !ok {
|
|
a.Logger.Errorw("Context error",
|
|
"error", "context")
|
|
}
|
|
// NOTE: An invalid subdomain should not happen - the auth handler should
|
|
// reject POSTs with an invalid subdomain before this handler. Reject any
|
|
// invalid subdomains anyway as a matter of caution.
|
|
if !validSubdomain(atxt.Subdomain) {
|
|
a.Logger.Errorw("Bad update data",
|
|
"error", "subdomain",
|
|
"subdomain", atxt.Subdomain,
|
|
"txt", atxt.Value)
|
|
updStatus = http.StatusBadRequest
|
|
upd = jsonError("bad_subdomain")
|
|
} else if !validTXT(atxt.Value) {
|
|
a.Logger.Errorw("Bad update data",
|
|
"error", "txt",
|
|
"subdomain", atxt.Subdomain,
|
|
"txt", atxt.Value)
|
|
updStatus = http.StatusBadRequest
|
|
upd = jsonError("bad_txt")
|
|
} else if validSubdomain(atxt.Subdomain) && validTXT(atxt.Value) {
|
|
err := a.DB.Update(atxt.ACMETxtPost)
|
|
if err != nil {
|
|
a.Logger.Errorw("Error while trying to update record",
|
|
"error", err.Error())
|
|
updStatus = http.StatusInternalServerError
|
|
upd = jsonError("db_error")
|
|
} else {
|
|
a.Logger.Debugw("TXT record updated",
|
|
"subdomain", atxt.Subdomain,
|
|
"txt", atxt.Value)
|
|
updStatus = http.StatusOK
|
|
upd = []byte("{\"txt\": \"" + atxt.Value + "\"}")
|
|
}
|
|
}
|
|
w.Header().Set("Content-Type", "application/json")
|
|
w.WriteHeader(updStatus)
|
|
_, _ = w.Write(upd)
|
|
}
|