diff --git a/models/t_parse.go b/models/t_parse.go index ac444208d..00aba0812 100644 --- a/models/t_parse.go +++ b/models/t_parse.go @@ -12,43 +12,43 @@ import ( // // If this doesn't work for all rtypes, process the special cases then // call this for the remainder. -func (rc *RecordConfig) PopulateFromString(rtype, contents, origin string) error { - if rc.Type != "" && rc.Type != rtype { - panic(fmt.Errorf("assertion failed: rtype already set (%s) (%s)", rtype, rc.Type)) +func (r *RecordConfig) PopulateFromString(rtype, contents, origin string) error { + if r.Type != "" && r.Type != rtype { + panic(fmt.Errorf("assertion failed: rtype already set (%s) (%s)", rtype, r.Type)) } - switch rc.Type = rtype; rtype { // #rtype_variations + switch r.Type = rtype; rtype { // #rtype_variations case "A": ip := net.ParseIP(contents) if ip == nil || ip.To4() == nil { return fmt.Errorf("invalid IP in A record: %s", contents) } - return rc.SetTargetIP(ip) // Reformat to canonical form. + return r.SetTargetIP(ip) // Reformat to canonical form. case "AAAA": ip := net.ParseIP(contents) if ip == nil || ip.To16() == nil { return fmt.Errorf("invalid IP in AAAA record: %s", contents) } - return rc.SetTargetIP(ip) // Reformat to canonical form. + return r.SetTargetIP(ip) // Reformat to canonical form. case "AKAMAICDN", "ALIAS", "ANAME", "CNAME", "NS", "PTR": - return rc.SetTarget(contents) + return r.SetTarget(contents) case "CAA": - return rc.SetTargetCAAString(contents) + return r.SetTargetCAAString(contents) case "DS": - return rc.SetTargetDSString(contents) + return r.SetTargetDSString(contents) case "MX": - return rc.SetTargetMXString(contents) + return r.SetTargetMXString(contents) case "NAPTR": - return rc.SetTargetNAPTRString(contents) + return r.SetTargetNAPTRString(contents) case "SRV": - return rc.SetTargetSRVString(contents) + return r.SetTargetSRVString(contents) case "SOA": - return rc.SetTargetSOAString(contents) + return r.SetTargetSOAString(contents) case "SSHFP": - return rc.SetTargetSSHFPString(contents) + return r.SetTargetSSHFPString(contents) case "TLSA": - return rc.SetTargetTLSAString(contents) + return r.SetTargetTLSAString(contents) case "SPF", "TXT": - return rc.SetTargetTXTString(contents) + return r.SetTargetTXTString(contents) default: return fmt.Errorf("unknown rtype (%s) when parsing (%s) domain=(%s)", rtype, contents, origin) diff --git a/pkg/acme/acme.go b/pkg/acme/acme.go index 06a496bb4..31de71d4a 100644 --- a/pkg/acme/acme.go +++ b/pkg/acme/acme.go @@ -181,9 +181,7 @@ func getCertInfo(pemBytes []byte) (names []string, remaining float64, err error) if err != nil { return nil, 0, err } - //lint:ignore S1024 I think this would be - // I think that would be this code but without unit tests I don't want to risk it: - // var daysLeft = float64(time.Until(cert.NotAfter)) / float64(time.Hour*24) + // FIXME(tlim): should use time.Until instead of t.Sub(time.Now()) (S1024) var daysLeft = float64(cert.NotAfter.Sub(time.Now())) / float64(time.Hour*24) return cert.DNSNames, daysLeft, nil } diff --git a/pkg/js/js.go b/pkg/js/js.go index 9660055c9..8a31ee257 100644 --- a/pkg/js/js.go +++ b/pkg/js/js.go @@ -149,7 +149,7 @@ func require(call otto.FunctionCall) otto.Value { throw(call.Otto, err.Error()) } - var value = otto.TrueValue() + var value otto.Value = otto.TrueValue() // If its a json file return the json value, else default to true if strings.HasSuffix(filepath.Ext(relFile), "json") { @@ -194,7 +194,7 @@ func listFiles(call otto.FunctionCall) otto.Value { } // Second: Recursive? - var recursive = true + var recursive bool = true if call.Argument(1).IsDefined() && !call.Argument(1).IsNull() { if call.Argument(1).IsBoolean() { recursive, _ = call.Argument(1).ToBoolean() // If it should be recursive @@ -204,7 +204,7 @@ func listFiles(call otto.FunctionCall) otto.Value { } // Third: File extension filter. - var fileExtension = ".js" + var fileExtension string = ".js" if call.Argument(2).IsDefined() && !call.Argument(2).IsNull() { if call.Argument(2).IsString() { fileExtension = call.Argument(2).String() // Which file extension to filter for. diff --git a/providers/cloudflare/rest.go b/providers/cloudflare/rest.go index 79b6638e2..d25692dd1 100644 --- a/providers/cloudflare/rest.go +++ b/providers/cloudflare/rest.go @@ -352,9 +352,7 @@ func (c *cloudflareProvider) createTestWorker(workerName string) error { return err } -//lint:ignore U1000 false positive due to -// https://github.com/dominikh/go-tools/issues/1137 which is a dup of -// https://github.com/dominikh/go-tools/issues/810 +// go-staticcheck lies! type pageRuleConstraint struct { Operator string `json:"operator"` Value string `json:"value"` diff --git a/providers/linode/api.go b/providers/linode/api.go index 59f72c093..c5e1ccfab 100644 --- a/providers/linode/api.go +++ b/providers/linode/api.go @@ -15,17 +15,17 @@ const ( domainsPath = "domains" ) -func (api *linodeProvider) fetchDomainList() error { - api.domainIndex = map[string]int{} +func (c *linodeProvider) fetchDomainList() error { + c.domainIndex = map[string]int{} page := 1 for { dr := &domainResponse{} endpoint := fmt.Sprintf("%s?page=%d", domainsPath, page) - if err := api.get(endpoint, dr); err != nil { + if err := c.get(endpoint, dr); err != nil { return fmt.Errorf("failed fetching domain list (Linode): %s", err) } for _, domain := range dr.Data { - api.domainIndex[domain.Domain] = domain.ID + c.domainIndex[domain.Domain] = domain.ID } if len(dr.Data) == 0 || dr.Page >= dr.Pages { break @@ -35,13 +35,13 @@ func (api *linodeProvider) fetchDomainList() error { return nil } -func (api *linodeProvider) getRecords(id int) ([]domainRecord, error) { +func (c *linodeProvider) getRecords(id int) ([]domainRecord, error) { records := []domainRecord{} page := 1 for { dr := &recordResponse{} endpoint := fmt.Sprintf("%s/%d/records?page=%d", domainsPath, id, page) - if err := api.get(endpoint, dr); err != nil { + if err := c.get(endpoint, dr); err != nil { return nil, fmt.Errorf("failed fetching record list (Linode): %s", err) } @@ -56,21 +56,21 @@ func (api *linodeProvider) getRecords(id int) ([]domainRecord, error) { return records, nil } -func (api *linodeProvider) createRecord(domainID int, rec *recordEditRequest) (*domainRecord, error) { +func (c *linodeProvider) createRecord(domainID int, rec *recordEditRequest) (*domainRecord, error) { endpoint := fmt.Sprintf("%s/%d/records", domainsPath, domainID) - req, err := api.newRequest(http.MethodPost, endpoint, rec) + req, err := c.newRequest(http.MethodPost, endpoint, rec) if err != nil { return nil, err } - resp, err := api.client.Do(req) + resp, err := c.client.Do(req) if err != nil { return nil, err } if resp.StatusCode != http.StatusOK { - return nil, api.handleErrors(resp) + return nil, c.handleErrors(resp) } record := &domainRecord{} @@ -84,52 +84,52 @@ func (api *linodeProvider) createRecord(domainID int, rec *recordEditRequest) (* return record, nil } -func (api *linodeProvider) modifyRecord(domainID, recordID int, rec *recordEditRequest) error { +func (c *linodeProvider) modifyRecord(domainID, recordID int, rec *recordEditRequest) error { endpoint := fmt.Sprintf("%s/%d/records/%d", domainsPath, domainID, recordID) - req, err := api.newRequest(http.MethodPut, endpoint, rec) + req, err := c.newRequest(http.MethodPut, endpoint, rec) if err != nil { return err } - resp, err := api.client.Do(req) + resp, err := c.client.Do(req) if err != nil { return err } if resp.StatusCode != http.StatusOK { - return api.handleErrors(resp) + return c.handleErrors(resp) } return nil } -func (api *linodeProvider) deleteRecord(domainID, recordID int) error { +func (c *linodeProvider) deleteRecord(domainID, recordID int) error { endpoint := fmt.Sprintf("%s/%d/records/%d", domainsPath, domainID, recordID) - req, err := api.newRequest(http.MethodDelete, endpoint, nil) + req, err := c.newRequest(http.MethodDelete, endpoint, nil) if err != nil { return err } - resp, err := api.client.Do(req) + resp, err := c.client.Do(req) if err != nil { return err } if resp.StatusCode != http.StatusOK { - return api.handleErrors(resp) + return c.handleErrors(resp) } return nil } -func (api *linodeProvider) newRequest(method, endpoint string, body interface{}) (*http.Request, error) { +func (c *linodeProvider) newRequest(method, endpoint string, body interface{}) (*http.Request, error) { rel, err := url.Parse(endpoint) if err != nil { return nil, err } - u := api.baseURL.ResolveReference(rel) + u := c.baseURL.ResolveReference(rel) buf := new(bytes.Buffer) if body != nil { @@ -149,24 +149,24 @@ func (api *linodeProvider) newRequest(method, endpoint string, body interface{}) return req, nil } -func (api *linodeProvider) get(endpoint string, target interface{}) error { - req, err := api.newRequest(http.MethodGet, endpoint, nil) +func (c *linodeProvider) get(endpoint string, target interface{}) error { + req, err := c.newRequest(http.MethodGet, endpoint, nil) if err != nil { return err } - resp, err := api.client.Do(req) + resp, err := c.client.Do(req) if err != nil { return err } if resp.StatusCode != http.StatusOK { - return api.handleErrors(resp) + return c.handleErrors(resp) } defer resp.Body.Close() decoder := json.NewDecoder(resp.Body) return decoder.Decode(target) } -func (api *linodeProvider) handleErrors(resp *http.Response) error { +func (c *linodeProvider) handleErrors(resp *http.Response) error { defer resp.Body.Close() decoder := json.NewDecoder(resp.Body) diff --git a/providers/linode/linodeProvider.go b/providers/linode/linodeProvider.go index 7a5414b33..d6b1926c8 100644 --- a/providers/linode/linodeProvider.go +++ b/providers/linode/linodeProvider.go @@ -293,7 +293,7 @@ func toReq(dc *models.DomainConfig, rc *models.RecordConfig) (*recordEditRequest return nil, fmt.Errorf("SRV Record must match format \"_service._protocol\" not %s", req.Name) } - var serviceName, protocol = result[1], strings.ToLower(result[2]) + var serviceName, protocol string = result[1], strings.ToLower(result[2]) req.Protocol = protocol req.Service = serviceName diff --git a/providers/msdns/corrections.go b/providers/msdns/corrections.go index a80ef4082..a166028ca 100644 --- a/providers/msdns/corrections.go +++ b/providers/msdns/corrections.go @@ -9,10 +9,10 @@ import ( ) // GetDomainCorrections gets existing records, diffs them against existing, and returns corrections. -func (client *msdnsProvider) GenerateDomainCorrections(dc *models.DomainConfig, existing models.Records) ([]*models.Correction, error) { +func (c *msdnsProvider) GenerateDomainCorrections(dc *models.DomainConfig, existing models.Records) ([]*models.Correction, error) { // Read foundRecords: - foundRecords, err := client.GetZoneRecords(dc.Name) + foundRecords, err := c.GetZoneRecords(dc.Name) if err != nil { return nil, fmt.Errorf("c.GetDNSZoneRecords(%v) failed: %v", dc.Name, err) } @@ -30,45 +30,45 @@ func (client *msdnsProvider) GenerateDomainCorrections(dc *models.DomainConfig, // Generate changes. corrections := []*models.Correction{} for _, del := range dels { - corrections = append(corrections, client.deleteRec(client.dnsserver, dc.Name, del)) + corrections = append(corrections, c.deleteRec(c.dnsserver, dc.Name, del)) } for _, cre := range creates { - corrections = append(corrections, client.createRec(client.dnsserver, dc.Name, cre)...) + corrections = append(corrections, c.createRec(c.dnsserver, dc.Name, cre)...) } for _, m := range modifications { - corrections = append(corrections, client.modifyRec(client.dnsserver, dc.Name, m)) + corrections = append(corrections, c.modifyRec(c.dnsserver, dc.Name, m)) } return corrections, nil } -func (client *msdnsProvider) deleteRec(dnsserver, domainname string, cor diff.Correlation) *models.Correction { +func (c *msdnsProvider) deleteRec(dnsserver, domainname string, cor diff.Correlation) *models.Correction { rec := cor.Existing return &models.Correction{ Msg: cor.String(), F: func() error { - return client.shell.RecordDelete(dnsserver, domainname, rec) + return c.shell.RecordDelete(dnsserver, domainname, rec) }, } } -func (client *msdnsProvider) createRec(dnsserver, domainname string, cre diff.Correlation) []*models.Correction { +func (c *msdnsProvider) createRec(dnsserver, domainname string, cre diff.Correlation) []*models.Correction { rec := cre.Desired arr := []*models.Correction{{ Msg: cre.String(), F: func() error { - return client.shell.RecordCreate(dnsserver, domainname, rec) + return c.shell.RecordCreate(dnsserver, domainname, rec) }, }} return arr } -func (client *msdnsProvider) modifyRec(dnsserver, domainname string, m diff.Correlation) *models.Correction { +func (c *msdnsProvider) modifyRec(dnsserver, domainname string, m diff.Correlation) *models.Correction { old, rec := m.Existing, m.Desired return &models.Correction{ Msg: m.String(), F: func() error { - return client.shell.RecordModify(dnsserver, domainname, old, rec) + return c.shell.RecordModify(dnsserver, domainname, old, rec) }, } } diff --git a/providers/msdns/domains.go b/providers/msdns/domains.go index 88e643335..4767cd576 100644 --- a/providers/msdns/domains.go +++ b/providers/msdns/domains.go @@ -2,7 +2,7 @@ package msdns import "github.com/StackExchange/dnscontrol/v3/models" -func (client *msdnsProvider) GetNameservers(string) ([]*models.Nameserver, error) { +func (c *msdnsProvider) GetNameservers(string) ([]*models.Nameserver, error) { // TODO: If using AD for publicly hosted zones, probably pull these from config. return nil, nil } diff --git a/providers/msdns/listzones.go b/providers/msdns/listzones.go index 9ad65f651..5168e5c8d 100644 --- a/providers/msdns/listzones.go +++ b/providers/msdns/listzones.go @@ -1,7 +1,7 @@ package msdns -func (client *msdnsProvider) ListZones() ([]string, error) { - zones, err := client.shell.GetDNSServerZoneAll(client.dnsserver) +func (c *msdnsProvider) ListZones() ([]string, error) { + zones, err := c.shell.GetDNSServerZoneAll(c.dnsserver) if err != nil { return nil, err } diff --git a/providers/namedotcom/zones.go b/providers/namedotcom/zones.go index 93f45140b..0896497a0 100644 --- a/providers/namedotcom/zones.go +++ b/providers/namedotcom/zones.go @@ -5,18 +5,18 @@ import ( ) // ListZones returns all the zones in an account -func (n *namedotcomProvider) ListZones() ([]string, error) { +func (c *namedotcomProvider) ListZones() ([]string, error) { var names []string var page int32 for { - response, err := n.client.ListDomains(&namecom.ListDomainsRequest{Page: page}) + n, err := c.client.ListDomains(&namecom.ListDomainsRequest{Page: page}) if err != nil { return nil, err } - page = response.NextPage + page = n.NextPage - for _, j := range response.Domains { + for _, j := range n.Domains { names = append(names, j.DomainName) } diff --git a/providers/ns1/ns1Provider.go b/providers/ns1/ns1Provider.go index 95c0b9d73..e609dad14 100644 --- a/providers/ns1/ns1Provider.go +++ b/providers/ns1/ns1Provider.go @@ -176,8 +176,8 @@ func buildRecord(recs models.Records, domain string, id string) *dns.Record { Rdata: []string{ fmt.Sprintf("%v", r.CaaFlag), r.CaaTag, - r.GetTargetField(), - }}) + fmt.Sprintf("%s", r.GetTargetField()), + }}) } else if r.Type == "SRV" { rec.AddAnswer(&dns.Answer{Rdata: strings.Split(fmt.Sprintf("%d %d %d %v", r.SrvPriority, r.SrvWeight, r.SrvPort, r.GetTargetField()), " ")}) } else if r.Type == "NAPTR" { @@ -216,8 +216,8 @@ func convert(zr *dns.ZoneRecord, domain string) ([]*models.RecordConfig, error) } case "CAA": //dnscontrol expects quotes around multivalue CAA entries, API doesn't add them - xAns := strings.SplitN(ans, " ", 3) - if err := rec.SetTargetCAAStrings(xAns[0], xAns[1], xAns[2]); err != nil { + x_ans := strings.SplitN(ans, " ", 3) + if err := rec.SetTargetCAAStrings(x_ans[0], x_ans[1], x_ans[2]); err != nil { return nil, fmt.Errorf("unparsable %s record received from ns1: %w", rtype, err) } default: diff --git a/providers/octodns/octodnsProvider.go b/providers/octodns/octodnsProvider.go index 27224ec67..64498fb56 100644 --- a/providers/octodns/octodnsProvider.go +++ b/providers/octodns/octodnsProvider.go @@ -52,13 +52,13 @@ func initProvider(config map[string]string, providermeta json.RawMessage) (provi if api.directory == "" { api.directory = "config" } - // Commented out because at this time api has no exported fields. - // if len(providermeta) != 0 { - // err := json.Unmarshal(providermeta, api) - // if err != nil { - // return nil, err - // } - // } + if len(providermeta) != 0 { + err := json.Unmarshal(providermeta, api) + if err != nil { + return nil, err + } + } + //api.nameservers = models.StringsToNameservers(api.DefaultNS) return api, nil } diff --git a/providers/octodns/staticcheck.conf b/providers/octodns/staticcheck.conf deleted file mode 100644 index f147677e5..000000000 --- a/providers/octodns/staticcheck.conf +++ /dev/null @@ -1 +0,0 @@ -checks = ["inherit", "-S1034"] diff --git a/providers/packetframe/api.go b/providers/packetframe/api.go index e990a52e5..069d33c87 100644 --- a/providers/packetframe/api.go +++ b/providers/packetframe/api.go @@ -57,25 +57,25 @@ type domainRecord struct { Zone string `json:"zone"` } -func (api *packetframeProvider) fetchDomainList() error { - api.domainIndex = map[string]zone{} +func (c *packetframeProvider) fetchDomainList() error { + c.domainIndex = map[string]zone{} dr := &domainResponse{} endpoint := "dns/zones" - if err := api.get(endpoint, dr); err != nil { + if err := c.get(endpoint, dr); err != nil { return fmt.Errorf("failed fetching domain list (Packetframe): %w", err) } for _, zone := range dr.Data.Zones { - api.domainIndex[zone.Zone] = zone + c.domainIndex[zone.Zone] = zone } return nil } -func (api *packetframeProvider) getRecords(zoneID string) ([]domainRecord, error) { +func (c *packetframeProvider) getRecords(zoneID string) ([]domainRecord, error) { var records []domainRecord dr := &recordResponse{} endpoint := "dns/records/" + zoneID - if err := api.get(endpoint, dr); err != nil { + if err := c.get(endpoint, dr); err != nil { return records, fmt.Errorf("failed fetching domain list (Packetframe): %w", err) } records = append(records, dr.Data.Records...) @@ -93,15 +93,15 @@ func (api *packetframeProvider) getRecords(zoneID string) ([]domainRecord, error return records, nil } -func (api *packetframeProvider) createRecord(rec *domainRecord) (*domainRecord, error) { +func (c *packetframeProvider) createRecord(rec *domainRecord) (*domainRecord, error) { endpoint := "dns/records" - req, err := api.newRequest(http.MethodPost, endpoint, rec) + req, err := c.newRequest(http.MethodPost, endpoint, rec) if err != nil { return nil, err } - _, err = api.client.Do(req) + _, err = c.client.Do(req) if err != nil { return nil, err } @@ -109,15 +109,15 @@ func (api *packetframeProvider) createRecord(rec *domainRecord) (*domainRecord, return rec, nil } -func (api *packetframeProvider) modifyRecord(rec *domainRecord) error { +func (c *packetframeProvider) modifyRecord(rec *domainRecord) error { endpoint := "dns/records" - req, err := api.newRequest(http.MethodPut, endpoint, rec) + req, err := c.newRequest(http.MethodPut, endpoint, rec) if err != nil { return err } - _, err = api.client.Do(req) + _, err = c.client.Do(req) if err != nil { return err } @@ -125,31 +125,31 @@ func (api *packetframeProvider) modifyRecord(rec *domainRecord) error { return nil } -func (api *packetframeProvider) deleteRecord(zoneID string, recordID string) error { +func (c *packetframeProvider) deleteRecord(zoneID string, recordID string) error { endpoint := "dns/records" - req, err := api.newRequest(http.MethodDelete, endpoint, deleteRequest{Zone: zoneID, Record: recordID}) + req, err := c.newRequest(http.MethodDelete, endpoint, deleteRequest{Zone: zoneID, Record: recordID}) if err != nil { return err } - resp, err := api.client.Do(req) + resp, err := c.client.Do(req) if err != nil { return err } if resp.StatusCode != http.StatusOK { - return api.handleErrors(resp) + return c.handleErrors(resp) } return nil } -func (api *packetframeProvider) newRequest(method, endpoint string, body interface{}) (*http.Request, error) { +func (c *packetframeProvider) newRequest(method, endpoint string, body interface{}) (*http.Request, error) { rel, err := url.Parse(endpoint) if err != nil { return nil, err } - u := api.baseURL.ResolveReference(rel) + u := c.baseURL.ResolveReference(rel) buf := new(bytes.Buffer) if body != nil { @@ -166,21 +166,21 @@ func (api *packetframeProvider) newRequest(method, endpoint string, body interfa req.Header.Add("Content-Type", mediaType) req.Header.Add("Accept", mediaType) - req.Header.Add("Authorization", "Token "+api.token) + req.Header.Add("Authorization", "Token "+c.token) return req, nil } -func (api *packetframeProvider) get(endpoint string, target interface{}) error { - req, err := api.newRequest(http.MethodGet, endpoint, nil) +func (c *packetframeProvider) get(endpoint string, target interface{}) error { + req, err := c.newRequest(http.MethodGet, endpoint, nil) if err != nil { return err } - resp, err := api.client.Do(req) + resp, err := c.client.Do(req) if err != nil { return err } if resp.StatusCode != http.StatusOK { - return api.handleErrors(resp) + return c.handleErrors(resp) } defer resp.Body.Close() @@ -188,7 +188,7 @@ func (api *packetframeProvider) get(endpoint string, target interface{}) error { return decoder.Decode(target) } -func (api *packetframeProvider) handleErrors(resp *http.Response) error { +func (c *packetframeProvider) handleErrors(resp *http.Response) error { body, err := ioutil.ReadAll(resp.Body) if err != nil { return err diff --git a/providers/softlayer/softlayerProvider.go b/providers/softlayer/softlayerProvider.go index ec237476c..b0ca137ea 100644 --- a/providers/softlayer/softlayerProvider.go +++ b/providers/softlayer/softlayerProvider.go @@ -150,7 +150,7 @@ func (s *softlayerProvider) getExistingRecords(domain *datatypes.Dns_Domain) ([] switch recType { case "SRV": - var service, protocol = "", "_tcp" + var service, protocol string = "", "_tcp" if record.Weight != nil { recConfig.SrvWeight = uint16(*record.Weight) @@ -190,9 +190,9 @@ func (s *softlayerProvider) getExistingRecords(domain *datatypes.Dns_Domain) ([] } func (s *softlayerProvider) createRecordFunc(desired *models.RecordConfig, domain *datatypes.Dns_Domain) func() error { - var ttl, preference, domainID = verifyMinTTL(int(desired.TTL)), int(desired.MxPreference), *domain.Id - var weight, priority, port = int(desired.SrvWeight), int(desired.SrvPriority), int(desired.SrvPort) - var host, data, newType = desired.GetLabel(), desired.GetTargetField(), desired.Type + var ttl, preference, domainID int = verifyMinTTL(int(desired.TTL)), int(desired.MxPreference), *domain.Id + var weight, priority, port int = int(desired.SrvWeight), int(desired.SrvPriority), int(desired.SrvPort) + var host, data, newType string = desired.GetLabel(), desired.GetTargetField(), desired.Type var err error srvRegexp := regexp.MustCompile(`^_(?P\w+)\.\_(?P\w+)$`) @@ -226,7 +226,7 @@ func (s *softlayerProvider) createRecordFunc(desired *models.RecordConfig, domai return fmt.Errorf("SRV Record must match format \"_service._protocol\" not %s", host) } - var serviceName, protocol = result[1], strings.ToLower(result[2]) + var serviceName, protocol string = result[1], strings.ToLower(result[2]) newSrv := datatypes.Dns_Domain_ResourceRecord_SrvType{ Dns_Domain_ResourceRecord: newRecord, @@ -260,8 +260,8 @@ func (s *softlayerProvider) deleteRecordFunc(resID int) func() error { } func (s *softlayerProvider) updateRecordFunc(existing *datatypes.Dns_Domain_ResourceRecord, desired *models.RecordConfig) func() error { - var ttl, preference = verifyMinTTL(int(desired.TTL)), int(desired.MxPreference) - var priority, weight, port = int(desired.SrvPriority), int(desired.SrvWeight), int(desired.SrvPort) + var ttl, preference int = verifyMinTTL(int(desired.TTL)), int(desired.MxPreference) + var priority, weight, port int = int(desired.SrvPriority), int(desired.SrvWeight), int(desired.SrvPort) return func() error { var changes = false diff --git a/staticcheck.conf b/staticcheck.conf deleted file mode 100644 index 5a9502dd5..000000000 --- a/staticcheck.conf +++ /dev/null @@ -1,3 +0,0 @@ -checks = ["all", "-ST1000", "-U1000"] -# U1000 is disabled due to https://github.com/dominikh/go-tools/issues/810 -# TODO(tlim): When #810 is fixed, we should enable U1000.