21649486448

Committed 03 Feb 2026 09:59PM UTC coverage: 77.593% (+4.3%) from 73.276%

Build # 21649486448

Build Type

Pull #325

github

Committed by

joohoi

Commit Message

Fix linter and umask setting

Pull Request Pull Request #325: Refactoring

Run Details

578 of 754 new or added lines in 20 files covered. (76.66%)

3 existing lines in 1 file now uncovered.

793 of 1022 relevant lines covered (77.59%)

7.23 hits per line

Source File
Press 'n' to go to next uncovered line, 'b' for previous

77.5

/pkg/api/update.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)
}

1	package api
2
3	import (
4	"net/http"
5
6	"github.com/joohoi/acme-dns/pkg/acmedns"
7
8	"github.com/julienschmidt/httprouter"
9	)
10
11	func (a AcmednsAPI) webUpdatePost(w http.ResponseWriter, r http.Request, _ httprouter.Params) {	12✔
12	var updStatus int	12✔
13	var upd []byte	12✔
14	// Get user	12✔
15	atxt, ok := r.Context().Value(ACMETxtKey).(acmedns.ACMETxt)	12✔
16	if !ok {	12✔
NEW 17	a.Logger.Errorw("Context error",	×
NEW 18	"error", "context")	×
NEW 19	}	×
20	// NOTE: An invalid subdomain should not happen - the auth handler should
21	// reject POSTs with an invalid subdomain before this handler. Reject any
22	// invalid subdomains anyway as a matter of caution.
23	if !validSubdomain(atxt.Subdomain) {	12✔
NEW 24	a.Logger.Errorw("Bad update data",	×
NEW 25	"error", "subdomain",	×
NEW 26	"subdomain", atxt.Subdomain,	×
NEW 27	"txt", atxt.Value)	×
NEW 28	updStatus = http.StatusBadRequest	×
NEW 29	upd = jsonError("bad_subdomain")	×
30	} else if !validTXT(atxt.Value) {	15✔
31	a.Logger.Errorw("Bad update data",	3✔
32	"error", "txt",	3✔
33	"subdomain", atxt.Subdomain,	3✔
34	"txt", atxt.Value)	3✔
35	updStatus = http.StatusBadRequest	3✔
36	upd = jsonError("bad_txt")	3✔
37	} else if validSubdomain(atxt.Subdomain) && validTXT(atxt.Value) {	21✔
38	err := a.DB.Update(atxt.ACMETxtPost)	9✔
39	if err != nil {	10✔
40	a.Logger.Errorw("Error while trying to update record",	1✔
41	"error", err.Error())	1✔
42	updStatus = http.StatusInternalServerError	1✔
43	upd = jsonError("db_error")	1✔
44	} else {	9✔
45	a.Logger.Debugw("TXT record updated",	8✔
46	"subdomain", atxt.Subdomain,	8✔
47	"txt", atxt.Value)	8✔
48	updStatus = http.StatusOK	8✔
49	upd = []byte("{\"txt\": \"" + atxt.Value + "\"}")	8✔
50	}	8✔
51	}
52	w.Header().Set("Content-Type", "application/json")	12✔
53	w.WriteHeader(updStatus)	12✔
54	_, _ = w.Write(upd)	12✔
55	}

joohoi / acme-dns / 21649486448

Source File Press 'n' to go to next uncovered line, 'b' for previous

Source File
Press 'n' to go to next uncovered line, 'b' for previous