From b9407aa46c738f88b1ca4667119be564efa918ac Mon Sep 17 00:00:00 2001 From: Tony Grosinger Date: Sat, 12 Aug 2023 14:54:32 -0700 Subject: [PATCH] ListWebhooks bug when listing all webhooks When listing all webhooks it should not provide the MessageStream query parameter at all. When the parameter is empty Postmark returns an error. --- webhooks.go | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/webhooks.go b/webhooks.go index 2839a11..bab90c2 100644 --- a/webhooks.go +++ b/webhooks.go @@ -73,12 +73,17 @@ type Webhook struct { // is empty it will return all webhooks for the server. A non-existent message // stream will result in an error. func (client *Client) ListWebhooks(ctx context.Context, messageStream string) ([]Webhook, error) { + msgStreamParam := "" + if messageStream != "" { + msgStreamParam = fmt.Sprintf("?MessageStream=%s", messageStream) + } + var res struct { Webhooks []Webhook } err := client.doRequest(ctx, parameters{ Method: http.MethodGet, - Path: fmt.Sprintf("webhooks?MessageStream=%s", messageStream), + Path: "webhooks" + msgStreamParam, TokenType: serverToken, }, &res) return res.Webhooks, err