Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 4 additions & 4 deletions providers/dns/allinkl/internal/client.go
Original file line number Diff line number Diff line change
Expand Up @@ -86,23 +86,23 @@ func (c *Client) AddDNSSettings(ctx context.Context, record DNSRequest) (string,
}

// DeleteDNSSettings Deleting a DNS Resource Record.
func (c *Client) DeleteDNSSettings(ctx context.Context, recordID string) (bool, error) {
func (c *Client) DeleteDNSSettings(ctx context.Context, recordID string) (string, error) {
requestParams := map[string]string{"record_id": recordID}

req, err := c.newRequest(ctx, "delete_dns_settings", requestParams)
if err != nil {
return false, err
return "", err
}

var g DeleteDNSSettingsAPIResponse
err = c.do(req, &g)
if err != nil {
return false, err
return "", err
}

c.updateFloodTime(g.Response.KasFloodDelay)

return g.Response.ReturnInfo, nil
return g.Response.ReturnString, nil
}

func (c *Client) newRequest(ctx context.Context, action string, requestParams any) (*http.Request, error) {
Expand Down
2 changes: 1 addition & 1 deletion providers/dns/allinkl/internal/client_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -131,7 +131,7 @@ func TestClient_DeleteDNSSettings(t *testing.T) {
r, err := client.DeleteDNSSettings(mockContext(), "57347450")
require.NoError(t, err)

assert.True(t, r)
assert.Equal(t, "TRUE", r)
}

func testHandler(filename string) http.HandlerFunc {
Expand Down
2 changes: 1 addition & 1 deletion providers/dns/allinkl/internal/types_api.go
Original file line number Diff line number Diff line change
Expand Up @@ -89,6 +89,6 @@ type DeleteDNSSettingsAPIResponse struct {

type DeleteDNSSettingsResponse struct {
KasFloodDelay float64 `json:"KasFloodDelay"`
ReturnInfo bool `json:"ReturnInfo"`
ReturnString string `json:"ReturnString"`
// NOTE: ReturnInfo (!= ReturnString) doesn't seem to have a stable type
}