fix(fcm): correct the iidEndpoint endpoints used for topic management

rueian · rueian · commit 74bda9ff9662 · 2020-02-21T00:13:51.000+08:00
According to the document https://developers.google.com/instance-id/reference/server, the endpoints should be: https://iid.googleapis.com/iid/v1:batchAdd https://iid.googleapis.com/iid/v1:batchRemove NOT: https://iid.googleapis.com/iid/v1/:batchAdd https://iid.googleapis.com/iid/v1/:batchRemove
diff --git a/messaging/topic_mgt.go b/messaging/topic_mgt.go
@@ -25,9 +25,9 @@ import (
 )
 
 const (
-	iidEndpoint    = "https://iid.googleapis.com/iid/v1"
-	iidSubscribe   = ":batchAdd"
-	iidUnsubscribe = ":batchRemove"
+	iidEndpoint    = "https://iid.googleapis.com/iid"
+	iidSubscribe   = "v1:batchAdd"
+	iidUnsubscribe = "v1:batchRemove"
 )
 
 var iidErrorCodes = map[string]struct{ Code, Msg string }{
@@ -164,7 +164,7 @@ func (c *iidClient) makeTopicManagementRequest(ctx context.Context, req *iidRequ
 
 	request := &internal.Request{
 		Method: http.MethodPost,
-		URL:    fmt.Sprintf("%s/%s", c.iidEndpoint, req.op),
+		URL:    fmt.Sprintf("%s%s", c.iidEndpoint, req.op),
 		Body:   internal.NewJSONEntity(req),
 	}
 	var result iidResponse

Original file line number	Diff line number	Diff line change
`@@ -25,9 +25,9 @@ import (`
`25`	`25`	`)`
`26`	`26`
`27`	`27`	`const (`
`28`		`- iidEndpoint = "https://iid.googleapis.com/iid/v1"`
`29`		`- iidSubscribe = ":batchAdd"`
`30`		`- iidUnsubscribe = ":batchRemove"`
	`28`	`+ iidEndpoint = "https://iid.googleapis.com/iid"`
	`29`	`+ iidSubscribe = "v1:batchAdd"`
	`30`	`+ iidUnsubscribe = "v1:batchRemove"`
`31`	`31`	`)`
`32`	`32`
`33`	`33`	`var iidErrorCodes = map[string]struct{ Code, Msg string }{`
`@@ -164,7 +164,7 @@ func (c iidClient) makeTopicManagementRequest(ctx context.Context, req iidRequ`
`164`	`164`
`165`	`165`	`request := &internal.Request{`
`166`	`166`	`Method: http.MethodPost,`
`167`		`- URL: fmt.Sprintf("%s/%s", c.iidEndpoint, req.op),`
	`167`	`+ URL: fmt.Sprintf("%s%s", c.iidEndpoint, req.op),`
`168`	`168`	`Body: internal.NewJSONEntity(req),`
`169`	`169`	`}`
`170`	`170`	`var result iidResponse`