Skip to content

Commit eca0288

Browse files
authored
Merge pull request #32 from setoru/fix-name
refactor unused function
2 parents 3634e0c + c443dcf commit eca0288

1 file changed

Lines changed: 13 additions & 21 deletions

File tree

main.go

Lines changed: 13 additions & 21 deletions
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,6 @@ import (
55
"encoding/json"
66
"fmt"
77
"os"
8-
"strings"
98

109
"github.com/cert-manager/cert-manager/pkg/acme/webhook/apis/acme/v1alpha1"
1110
"github.com/cert-manager/cert-manager/pkg/acme/webhook/cmd"
@@ -143,7 +142,7 @@ func (h *huaweiDNSProviderSolver) CleanUp(ch *v1alpha1.ChallengeRequest) error {
143142
return err
144143
}
145144

146-
records, err := h.getRecords(zoneId, ch.ResolvedZone, ch.ResolvedFQDN)
145+
records, err := h.getRecords(zoneId, ch.ResolvedFQDN)
147146
if err != nil {
148147
return err
149148
}
@@ -153,9 +152,10 @@ func (h *huaweiDNSProviderSolver) CleanUp(ch *v1alpha1.ChallengeRequest) error {
153152
if r[0] != ch.Key {
154153
continue
155154
}
156-
req := &dnsMdl.DeleteRecordSetRequest{}
157-
req.ZoneId = zoneId
158-
req.RecordsetId = *record.Id
155+
req := &dnsMdl.DeleteRecordSetRequest{
156+
ZoneId: zoneId,
157+
RecordsetId: *record.Id,
158+
}
159159
resp, err := h.huaweiClient.DeleteRecordSet(req)
160160
if err != nil {
161161
return errors.Wrap(err, "failed to delete record")
@@ -245,9 +245,10 @@ func (h *huaweiDNSProviderSolver) getHuaweiClient(ch *v1alpha1.ChallengeRequest,
245245
func (h *huaweiDNSProviderSolver) getZoneId(resolvedZone string) (string, error) {
246246
zoneList := make([]dnsMdl.PrivateZoneResp, 0)
247247

248-
req := &dnsMdl.ListPrivateZonesRequest{}
249-
req.Offset = ptr.To[int32](0)
250-
req.Limit = ptr.To[int32](50)
248+
req := &dnsMdl.ListPrivateZonesRequest{
249+
Offset: ptr.To[int32](0),
250+
Limit: ptr.To[int32](50),
251+
}
251252
totalCount := int32(50)
252253

253254
for *req.Offset < totalCount {
@@ -299,20 +300,11 @@ func (h *huaweiDNSProviderSolver) addTxtRecord(zoneId, fqdn, key string) error {
299300
return nil
300301
}
301302

302-
func extractRecordName(fqdn, zone string) string {
303-
if idx := strings.Index(fqdn, "."+zone); idx != -1 {
304-
return fqdn[:idx]
303+
func (h *huaweiDNSProviderSolver) getRecords(zoneId, fqdn string) ([]dnsMdl.ListRecordSets, error) {
304+
req := &dnsMdl.ListRecordSetsByZoneRequest{
305+
ZoneId: zoneId,
306+
Name: &fqdn,
305307
}
306-
307-
return util.UnFqdn(fqdn)
308-
}
309-
310-
func (h *huaweiDNSProviderSolver) getRecords(zoneId, zone, fqdn string) ([]dnsMdl.ListRecordSets, error) {
311-
recordName := extractRecordName(fqdn, zone)
312-
req := &dnsMdl.ListRecordSetsByZoneRequest{}
313-
req.ZoneId = zoneId
314-
req.Name = &recordName
315-
316308
req.Offset = ptr.To[int32](0)
317309
req.Limit = ptr.To[int32](50)
318310
totalCount := int32(50)

0 commit comments

Comments
 (0)