Skip to content

Commit 6a02985

Browse files
committed
Fix region config not set at eventsapi compiletime
1 parent 5cf1368 commit 6a02985

2 files changed

Lines changed: 6 additions & 6 deletions

File tree

pkg/eventsapi/v1.go

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -2,13 +2,12 @@ package eventsapi
22

33
import (
44
"context"
5-
"fmt"
65
"net/http"
76

87
"github.com/PagerDuty/go-pdagent/pkg/common"
98
)
109

11-
var endpointV1 = fmt.Sprintf("%v/generic/2010-04-15/create_event.json", common.PdEventsUrl())
10+
const endpointV1 = "/generic/2010-04-15/create_event.json"
1211

1312
// EventV1 corresponds to a V1 event object.
1413
type EventV1 struct {
@@ -76,6 +75,7 @@ type ResponseV1 struct {
7675
// service's own.
7776
func CreateV1(context context.Context, client *http.Client, event *EventV1) (*ResponseV1, error) {
7877
var response ResponseV1
79-
err := enqueueEvent(context, client, endpointV1, event, &response)
78+
requestUrl := common.PdEventsUrl() + endpointV1
79+
err := enqueueEvent(context, client, requestUrl, event, &response)
8080
return &response, err
8181
}

pkg/eventsapi/v2.go

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -2,13 +2,12 @@ package eventsapi
22

33
import (
44
"context"
5-
"fmt"
65
"net/http"
76

87
"github.com/PagerDuty/go-pdagent/pkg/common"
98
)
109

11-
var endpointV2 = fmt.Sprintf("%v/v2/enqueue", common.PdEventsUrl())
10+
var endpointV2 = "/v2/enqueue"
1211

1312
// EventV2 corresponds to a V2 event object.
1413
type EventV2 struct {
@@ -81,6 +80,7 @@ type ResponseV2 struct {
8180
// EnqueueV2 sends an event explicitly to the Events API V2.
8281
func EnqueueV2(context context.Context, client *http.Client, event *EventV2) (*ResponseV2, error) {
8382
var response ResponseV2
84-
err := enqueueEvent(context, client, endpointV2, event, &response)
83+
requestUrl := common.PdEventsUrl() + endpointV2
84+
err := enqueueEvent(context, client, requestUrl, event, &response)
8585
return &response, err
8686
}

0 commit comments

Comments
 (0)