Update for alert center GA.

- Use alerting/v1 in REST calls.
- Update changelog and client version.
- Minor fix to example to create secondary zone correctly.

Author: hhellyer

CHANGELOG.md

@@ -1,4 +1,4 @@
-## 2.13.0 (Oct 28th, 2024)
+## 2.13.0 (Dec 5th, 2024)
 
 FEATURES:
 

mockns1/alert.go

@@ -15,7 +15,7 @@ type mockAlertListResponse struct {
 	TotalResults *int64            `json:"total_results,omitempty"`
 }
 
-const alertPath = "../alerting/v1beta1/alerts"
+const alertPath = "../alerting/v1/alerts"
 
 // AddAlertListTestCase sets up a test case for the api.Client.Alert.List()
 // function

rest/_examples/alerts.go

@@ -31,7 +31,6 @@ func init() {
 
 func prettyPrint(header string, v interface{}) {
 	fmt.Println(header)
-	fmt.Printf("%#v \n", v)
 	b, _ := json.MarshalIndent(v, "", "  ")
 	fmt.Println(string(b))
 }
@@ -66,6 +65,11 @@ func main() {
 
 	z := dns.NewZone(domain)
 	z.NxTTL = 3600
+	z.Secondary = &dns.ZoneSecondary{
+		Enabled:     true,
+		PrimaryIP:   "127.0.0.1",
+		PrimaryPort: 67,
+	}
 	_, err = client.Zones.Create(z)
 	if err != nil {
 		// Ignore if zone already exists
@@ -75,10 +79,10 @@ func main() {
 			log.Println("Zone already exists, continuing...")
 		}
 	}
-
 	prettyPrint("Zone:", z)
-	fmt.Printf("Creating alert...\n")
+
 	alert := alerting.NewZoneAlert("myalerttest.com - transfer failed", "transfer_failed", []string{webhookList.ID}, []string{domain})
+	prettyPrint("Creating alert: ", alert)
 	_, err = client.Alerts.Create(alert)
 	if err != nil {
 		if err == api.ErrAlertExists {

rest/alert.go

@@ -13,7 +13,7 @@ type AlertsService service
 
 // The base for the alerting api relative to /v1
 // client.NewRequest will call ResolveReference and remove /v1/../
-const alertingRelativeBase = "../alerting/v1beta1"
+const alertingRelativeBase = "../alerting/v1"
 
 type alertListResponse struct {
 	Limit        *int64            `json:"limit,omitempty"`

rest/alert_test.go

@@ -45,7 +45,7 @@ func TestAlert(t *testing.T) {
 			defer mock.ClearTestCases()
 
 			linkHeader := http.Header{}
-			linkHeader.Set("Link", `</alerting/v1beta1/alerts?next=`+*alertList[1].Name+`>; rel="next"`)
+			linkHeader.Set("Link", `</alerting/v1/alerts?next=`+*alertList[1].Name+`>; rel="next"`)
 			require.Nil(t, mock.AddAlertListTestCase("", nil, linkHeader, alertList[0:1]))
 			require.Nil(t, mock.AddAlertListTestCase("next="+*alertList[1].Name, nil, nil, alertList[2:3]))
 

rest/client.go

@@ -13,9 +13,9 @@ import (
 )
 
 const (
-	clientVersion = "2.12.2"
+	clientVersion = "2.13.0"
 
-	defaultBase                   = "http://localhost:80"
+	defaultBase                   = "https://api.nsone.net"
 	defaultEndpoint               = defaultBase + "/v1/"
 	defaultShouldFollowPagination = true
 	defaultUserAgent              = "go-ns1/" + clientVersion