-
Notifications
You must be signed in to change notification settings - Fork 21
fix(experimental): Add ldmiddleware library, utilities for creating scoped clients, contexts, events automatically #306
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
Merged
Changes from all commits
Commits
Show all changes
23 commits
Select commit
Hold shift + click to select a range
6d61262
Add http middleware file
aengelberg 4984b0f
Add request params to request context
aengelberg 250be0f
Separate AddRequestScopedClientWithKeyFn
aengelberg 08c3d3a
Add TrackTiming, TrackErrorResponses, httpsnoop dependency
aengelberg 58d96bb
bump v7 dependency
aengelberg 0e07729
rename fns
aengelberg b203ea2
Add test
aengelberg 7fe5bdb
Use testify assert
aengelberg 91f7acd
Test params better
aengelberg 26fb7bd
rewrite tests
aengelberg b2fb6d6
Add TestAddScopedClientForRequest_WithKeyFn
aengelberg 4bdcdb4
Add README.md
aengelberg e8441f1
ld_request context kind
aengelberg 5d49f2f
add WriteHeader
aengelberg f1b3576
Add ldmiddleware ci yml
aengelberg ed4d55a
Add to makefile
aengelberg 9685988
Update ldmiddleware/README.md
aengelberg 888764c
Add ldmiddleware to Makefile ALL_BUILD_TARGETS
aengelberg 60bbb59
lint
aengelberg 0c34467
lint
aengelberg e76927c
Merge branch 'v7' into ae/RO-1799/middleware
aengelberg 340e664
Bump ldmiddleware's go-server-sdk dependency to v7.13.4
aengelberg 6c5b532
Fix unit tests to handle scoped usage
aengelberg File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,40 @@ | ||
name: Build and Test ldmiddleware | ||
permissions: | ||
contents: read | ||
on: | ||
push: | ||
branches: [ 'v7', 'feat/**' ] | ||
paths-ignore: | ||
- '**.md' # Don't run CI on markdown changes. | ||
pull_request: | ||
branches: [ 'v7', 'feat/**' ] | ||
paths-ignore: | ||
- '**.md' | ||
|
||
jobs: | ||
go-versions: | ||
uses: ./.github/workflows/go-versions.yml | ||
|
||
# Runs the common tasks (unit tests, lint, contract tests) for each Go version. | ||
test-linux: | ||
name: ${{ format('ldmiddleware Linux, Go {0}', matrix.go-version) }} | ||
needs: go-versions | ||
strategy: | ||
# Let jobs fail independently, in case it's a single version that's broken. | ||
fail-fast: false | ||
matrix: | ||
go-version: ${{ fromJSON(needs.go-versions.outputs.matrix) }} | ||
runs-on: ubuntu-latest | ||
steps: | ||
- uses: actions/checkout@v4 | ||
- name: Setup Go ${{ inputs.go-version }} | ||
uses: actions/setup-go@v5 | ||
with: | ||
go-version: ${{ matrix.go-version }} | ||
- uses: ./.github/actions/unit-tests | ||
with: | ||
lint: 'true' | ||
test-target: ldmiddleware-test | ||
- uses: ./.github/actions/coverage | ||
with: | ||
enforce: 'false' |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,5 +1,6 @@ | ||
{ | ||
".": "7.13.4", | ||
"ldotel": "1.3.0", | ||
"ldai": "0.7.1" | ||
"ldai": "0.7.1", | ||
"ldmiddleware": "0.1.0" | ||
} |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,57 @@ | ||
LaunchDarkly HTTP Middleware | ||
=============================== | ||
|
||
# ⛔️⛔️⛔️⛔️ | ||
> [!CAUTION] | ||
> This library is an alpha version and should not be considered ready for production use while this message is visible. | ||
# ☝️☝️☝️☝️☝️☝️ | ||
|
||
[](https://github.com/launchdarkly/go-server-sdk/actions/workflows/ldmiddleware-ci.yml) | ||
|
||
This package provides a set of HTTP middleware functions that speed up the process of instrumenting your application with LaunchDarkly. | ||
|
||
## Usage | ||
|
||
```go | ||
import ( | ||
ld "github.com/launchdarkly/go-server-sdk/v7" | ||
ldmiddleware "github.com/launchdarkly/go-server-sdk/v7/ldmiddleware" | ||
"net/http" | ||
"time" | ||
"github.com/gorilla/mux" | ||
) | ||
|
||
func main() { | ||
client, err := ld.MakeClient("your-sdk-key", 5*time.Second) | ||
if err != nil { | ||
log.Fatal(err) | ||
} | ||
|
||
// Add the LaunchDarkly middleware functions to your middleware chain. | ||
// The order of the middleware functions is important. | ||
// The AddScopedClientForRequest function must be called before the TrackTiming and TrackErrorResponses functions. | ||
|
||
r := mux.NewRouter() | ||
r.Use(ldmiddleware.AddScopedClientForRequest(client)) | ||
r.Use(ldmiddleware.TrackTiming) | ||
r.Use(ldmiddleware.TrackErrorResponses) | ||
|
||
r.Handle("/", http.HandlerFunc(myHandler)) | ||
http.ListenAndServe(":8080", r) | ||
} | ||
|
||
func myHandler(w http.ResponseWriter, r *http.Request) { | ||
// Thanks to `AddScopedClientForRequest`, a scoped client is available in the request context. | ||
// We can use it to evaluate feature flags, track analytics, etc. without having to provide the LaunchDarkly context. | ||
var enableBetaFeatures bool | ||
if client, ok := ld.GetScopedClient(r.Context()); ok { | ||
enableBetaFeatures = client.BoolVariation("your-feature-key", false) | ||
} | ||
|
||
if enableBetaFeatures { | ||
// Do something | ||
} | ||
|
||
w.WriteHeader(200) | ||
} | ||
``` |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,28 @@ | ||
module github.com/launchdarkly/go-server-sdk/ldmiddleware | ||
|
||
go 1.23.0 | ||
|
||
require ( | ||
github.com/felixge/httpsnoop v1.0.4 | ||
github.com/google/uuid v1.1.1 | ||
github.com/launchdarkly/go-sdk-common/v3 v3.4.0 | ||
github.com/launchdarkly/go-server-sdk/v7 v7.13.4 | ||
github.com/stretchr/testify v1.9.0 | ||
) | ||
|
||
require ( | ||
github.com/davecgh/go-spew v1.1.1 // indirect | ||
github.com/gregjones/httpcache v0.0.0-20171119193500-2bcd89a1743f // indirect | ||
github.com/josharian/intern v1.0.0 // indirect | ||
github.com/launchdarkly/ccache v1.1.0 // indirect | ||
github.com/launchdarkly/eventsource v1.10.0 // indirect | ||
github.com/launchdarkly/go-jsonstream/v3 v3.1.0 // indirect | ||
github.com/launchdarkly/go-sdk-events/v3 v3.5.0 // indirect | ||
github.com/launchdarkly/go-semver v1.0.3 // indirect | ||
github.com/launchdarkly/go-server-sdk-evaluation/v3 v3.0.1 // indirect | ||
github.com/mailru/easyjson v0.7.7 // indirect | ||
github.com/patrickmn/go-cache v2.1.0+incompatible // indirect | ||
github.com/pmezard/go-difflib v1.0.0 // indirect | ||
golang.org/x/sync v0.8.0 // indirect | ||
gopkg.in/yaml.v3 v3.0.1 // indirect | ||
) |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,57 @@ | ||
github.com/davecgh/go-spew v1.1.0/go.mod h1:J7Y8YcW2NihsgmVo/mv3lAwl/skON4iLHjSsI+c5H38= | ||
github.com/davecgh/go-spew v1.1.1 h1:vj9j/u1bqnvCEfJOwUhtlOARqs3+rkHYY13jYWTU97c= | ||
github.com/davecgh/go-spew v1.1.1/go.mod h1:J7Y8YcW2NihsgmVo/mv3lAwl/skON4iLHjSsI+c5H38= | ||
github.com/felixge/httpsnoop v1.0.4 h1:NFTV2Zj1bL4mc9sqWACXbQFVBBg2W3GPvqp8/ESS2Wg= | ||
github.com/felixge/httpsnoop v1.0.4/go.mod h1:m8KPJKqk1gH5J9DgRY2ASl2lWCfGKXixSwevea8zH2U= | ||
github.com/google/uuid v1.1.1 h1:Gkbcsh/GbpXz7lPftLA3P6TYMwjCLYm83jiFQZF/3gY= | ||
github.com/google/uuid v1.1.1/go.mod h1:TIyPZe4MgqvfeYDBFedMoGGpEw/LqOeaOT+nhxU+yHo= | ||
github.com/gregjones/httpcache v0.0.0-20171119193500-2bcd89a1743f h1:kOkUP6rcVVqC+KlKKENKtgfFfJyDySYhqL9srXooghY= | ||
github.com/gregjones/httpcache v0.0.0-20171119193500-2bcd89a1743f/go.mod h1:FecbI9+v66THATjSRHfNgh1IVFe/9kFxbXtjV0ctIMA= | ||
github.com/josharian/intern v1.0.0 h1:vlS4z54oSdjm0bgjRigI+G1HpF+tI+9rE5LLzOg8HmY= | ||
github.com/josharian/intern v1.0.0/go.mod h1:5DoeVV0s6jJacbCEi61lwdGj/aVlrQvzHFFd8Hwg//Y= | ||
github.com/karlseguin/expect v1.0.2-0.20190806010014-778a5f0c6003 h1:vJ0Snvo+SLMY72r5J4sEfkuE7AFbixEP2qRbEcum/wA= | ||
github.com/karlseguin/expect v1.0.2-0.20190806010014-778a5f0c6003/go.mod h1:zNBxMY8P21owkeogJELCLeHIt+voOSduHYTFUbwRAV8= | ||
github.com/kr/pretty v0.3.1 h1:flRD4NNwYAUpkphVc1HcthR4KEIFJ65n8Mw5qdRn3LE= | ||
github.com/kr/pretty v0.3.1/go.mod h1:hoEshYVHaxMs3cyo3Yncou5ZscifuDolrwPKZanG3xk= | ||
github.com/kr/text v0.2.0 h1:5Nx0Ya0ZqY2ygV366QzturHI13Jq95ApcVaJBhpS+AY= | ||
github.com/kr/text v0.2.0/go.mod h1:eLer722TekiGuMkidMxC/pM04lWEeraHUUmBw8l2grE= | ||
github.com/launchdarkly/ccache v1.1.0 h1:voD1M+ZJXR3MREOKtBwgTF9hYHl1jg+vFKS/+VAkR2k= | ||
github.com/launchdarkly/ccache v1.1.0/go.mod h1:TlxzrlnzvYeXiLHmesMuvoZetu4Z97cV1SsdqqBJi1Q= | ||
github.com/launchdarkly/eventsource v1.10.0 h1:H9Tp6AfGu/G2qzBJC26iperrvwhzdbiA/gx7qE2nDFI= | ||
github.com/launchdarkly/eventsource v1.10.0/go.mod h1:J3oa50bPvJesZqNAJtb5btSIo5N6roDWhiAS3IpsKck= | ||
github.com/launchdarkly/go-jsonstream/v3 v3.1.0 h1:U/7/LplZO72XefBQ+FzHf6o4FwLHVqBE+4V58Ornu/E= | ||
github.com/launchdarkly/go-jsonstream/v3 v3.1.0/go.mod h1:2Pt4BR5AwWgsuVTCcIpB6Os04JFIKWfoA+7faKkZB5E= | ||
github.com/launchdarkly/go-sdk-common/v3 v3.4.0 h1:GTRulE0G43xdWY1QdjAXJ7QnZ8PMFU8pOWZICCydEtM= | ||
github.com/launchdarkly/go-sdk-common/v3 v3.4.0/go.mod h1:6MNeeP8b2VtsM6I3TbShCHW/+tYh2c+p5dB+ilS69sg= | ||
github.com/launchdarkly/go-sdk-events/v3 v3.5.0 h1:Yav8Thm70dZbO8U1foYwZPf3w60n/lNBRaYeeNM/qg4= | ||
github.com/launchdarkly/go-sdk-events/v3 v3.5.0/go.mod h1:oepYWQ2RvvjfL2WxkE1uJJIuRsIMOP4WIVgUpXRPcNI= | ||
github.com/launchdarkly/go-semver v1.0.3 h1:agIy/RN3SqeQDIfKkl+oFslEdeIs7pgsJBs3CdCcGQM= | ||
github.com/launchdarkly/go-semver v1.0.3/go.mod h1:xFmMwXba5Mb+3h72Z+VeSs9ahCvKo2QFUTHRNHVqR28= | ||
github.com/launchdarkly/go-server-sdk-evaluation/v3 v3.0.1 h1:rTgcYAFraGFj7sBMB2b7JCYCm0b9kph4FaMX02t4osQ= | ||
github.com/launchdarkly/go-server-sdk-evaluation/v3 v3.0.1/go.mod h1:fPS5d+zOsgFnMunj+Ki6jjlZtFvo4h9iNbtNXxzYn58= | ||
github.com/launchdarkly/go-server-sdk/v7 v7.13.4 h1:Jn4HQDkmV0DhbUKLz7gFbNrhVrE3xSx8D6FTKEDheis= | ||
github.com/launchdarkly/go-server-sdk/v7 v7.13.4/go.mod h1:EEUSX/bc1mVq+3pwrRzTfu8LFRWRI1UL4XMgzsKWmbE= | ||
github.com/launchdarkly/go-test-helpers/v3 v3.1.0 h1:E3bxJMzMoA+cJSF3xxtk2/chr1zshl1ZWa0/oR+8bvg= | ||
github.com/launchdarkly/go-test-helpers/v3 v3.1.0/go.mod h1:Ake5+hZFS/DmIGKx/cizhn5W9pGA7pplcR7xCxWiLIo= | ||
github.com/mailru/easyjson v0.7.7 h1:UGYAvKxe3sBsEDzO8ZeWOSlIQfWFlxbzLZe7hwFURr0= | ||
github.com/mailru/easyjson v0.7.7/go.mod h1:xzfreul335JAWq5oZzymOObrkdz5UnU4kGfJJLY9Nlc= | ||
github.com/patrickmn/go-cache v2.1.0+incompatible h1:HRMgzkcYKYpi3C8ajMPV8OFXaaRUnok+kx1WdO15EQc= | ||
github.com/patrickmn/go-cache v2.1.0+incompatible/go.mod h1:3Qf8kWWT7OJRJbdiICTKqZju1ZixQ/KpMGzzAfe6+WQ= | ||
github.com/pmezard/go-difflib v1.0.0 h1:4DBwDE0NGyQoBHbLQYPwSUPoCMWR5BEzIk/f1lZbAQM= | ||
github.com/pmezard/go-difflib v1.0.0/go.mod h1:iKH77koFhYxTK1pcRnkKkqfTogsbg7gZNVY4sRDYZ/4= | ||
github.com/rogpeppe/go-internal v1.9.0 h1:73kH8U+JUqXU8lRuOHeVHaa/SZPifC7BkcraZVejAe8= | ||
github.com/rogpeppe/go-internal v1.9.0/go.mod h1:WtVeX8xhTBvf0smdhujwtBcq4Qrzq/fJaraNFVN+nFs= | ||
github.com/stretchr/objx v0.1.0/go.mod h1:HFkY916IF+rwdDfMAkV7OtwuqBVzrE8GR6GFx+wExME= | ||
github.com/stretchr/testify v1.6.1/go.mod h1:6Fq8oRcR53rry900zMqJjRRixrwX3KX962/h/Wwjteg= | ||
github.com/stretchr/testify v1.9.0 h1:HtqpIVDClZ4nwg75+f6Lvsy/wHu+3BoSGCbBAcpTsTg= | ||
github.com/stretchr/testify v1.9.0/go.mod h1:r2ic/lqez/lEtzL7wO/rwa5dbSLXVDPFyf8C91i36aY= | ||
github.com/wsxiaoys/terminal v0.0.0-20160513160801-0940f3fc43a0 h1:3UeQBvD0TFrlVjOeLOBz+CPAI8dnbqNSVwUwRrkp7vQ= | ||
github.com/wsxiaoys/terminal v0.0.0-20160513160801-0940f3fc43a0/go.mod h1:IXCdmsXIht47RaVFLEdVnh1t+pgYtTAhQGj73kz+2DM= | ||
golang.org/x/sync v0.8.0 h1:3NFvSEYkUoMifnESzZl15y791HH1qU2xm6eCJU5ZPXQ= | ||
golang.org/x/sync v0.8.0/go.mod h1:Czt+wKu1gCyEFDUtn0jG5QVvpJ6rzVqr5aXyt9drQfk= | ||
gopkg.in/check.v1 v0.0.0-20161208181325-20d25e280405/go.mod h1:Co6ibVJAznAaIkqp8huTwlJQCZ016jof/cbN4VW5Yz0= | ||
gopkg.in/check.v1 v1.0.0-20201130134442-10cb98267c6c h1:Hei/4ADfdWqJk1ZMxUNpqntNwaWcugrBjAiHlqqRiVk= | ||
gopkg.in/check.v1 v1.0.0-20201130134442-10cb98267c6c/go.mod h1:JHkPIbrfpd72SG/EVd6muEfDQjcINNoR0C8j2r3qZ4Q= | ||
gopkg.in/yaml.v3 v3.0.0-20200313102051-9f266ea9e77c/go.mod h1:K4uyk7z7BCEPqu6E+C64Yfv1cQ7kz7rIZviUmN+EgEM= | ||
gopkg.in/yaml.v3 v3.0.1 h1:fxVm/GzAzEWqLHuvctI91KS9hhNmmWOoWu0XTYJS7CA= | ||
gopkg.in/yaml.v3 v3.0.1/go.mod h1:K4uyk7z7BCEPqu6E+C64Yfv1cQ7kz7rIZviUmN+EgEM= |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,110 @@ | ||
package ldmiddleware | ||
|
||
import ( | ||
"net/http" | ||
"time" | ||
|
||
"github.com/felixge/httpsnoop" | ||
"github.com/google/uuid" | ||
|
||
"github.com/launchdarkly/go-sdk-common/v3/ldcontext" | ||
"github.com/launchdarkly/go-sdk-common/v3/ldvalue" | ||
ld "github.com/launchdarkly/go-server-sdk/v7" | ||
) | ||
|
||
// RequestKeyFunc allows callers to override the request context key for the LDContext. | ||
// Return (key, true) to use the provided key; return ("", false) to fall back to the default UUID key. | ||
type RequestKeyFunc func(r *http.Request) (string, bool) | ||
|
||
// AddScopedClientForRequest returns a net/http middleware that, for each incoming request, | ||
// creates an LDScopedClient seeded with a `request`-kind LDContext populated with useful | ||
// HTTP request attributes (e.g., method, path, host, userAgent), and stores it in the | ||
// request's Go context. Downstream handlers can retrieve it via ld.GetScopedClient. | ||
func AddScopedClientForRequest(client *ld.LDClient) func(next http.Handler) http.Handler { | ||
return AddScopedClientForRequestWithKeyFn(client, nil) | ||
} | ||
|
||
// AddScopedClientForRequestWithKeyFn is like AddScopedClientForRequest, but allows providing a function to override | ||
// the context key used for the `request`-kind LDContext. If the function returns ok=false or an empty key, | ||
// a random UUID will be used. | ||
func AddScopedClientForRequestWithKeyFn( | ||
client *ld.LDClient, keyFn RequestKeyFunc, | ||
) func(next http.Handler) http.Handler { | ||
return func(next http.Handler) http.Handler { | ||
return http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) { | ||
// Determine request context key | ||
requestKey := "" | ||
if keyFn != nil { | ||
if k, ok := keyFn(r); ok && k != "" { | ||
requestKey = k | ||
} | ||
} | ||
if requestKey == "" { | ||
requestKey = uuid.New().String() | ||
} | ||
|
||
b := ldcontext.NewBuilder(requestKey).Kind("ld_request").Anonymous(true) | ||
aengelberg marked this conversation as resolved.
Show resolved
Hide resolved
|
||
b.SetString("method", r.Method) | ||
b.SetString("host", r.Host) | ||
b.SetString("userAgent", r.UserAgent()) | ||
if r.URL != nil { | ||
b.SetString("path", r.URL.Path) | ||
b.SetString("scheme", r.URL.Scheme) | ||
b.SetString("query", r.URL.RawQuery) | ||
} | ||
b.SetString("proto", r.Proto) | ||
b.SetString("remoteAddr", r.RemoteAddr) | ||
requestCtx := b.Build() | ||
|
||
scoped := ld.NewScopedClient(client, requestCtx) | ||
ctxWithScoped := ld.GoContextWithScopedClient(r.Context(), scoped) | ||
|
||
next.ServeHTTP(w, r.WithContext(ctxWithScoped)) | ||
}) | ||
} | ||
} | ||
|
||
// TrackTiming sends a LD event "http.request.duration_ms" with the duration of the request in milliseconds. | ||
// This middleware must be after AddScopedClientForRequest in the middleware chain, as it uses the scoped client | ||
// from the Go context. | ||
// | ||
// The timing event will include all LaunchDarkly contexts added to the scoped client. You may add more | ||
// contexts to the scoped client _during_ the request, and they will be included in the timing event sent | ||
// when the request completes. | ||
func TrackTiming(next http.Handler) http.Handler { | ||
return http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) { | ||
startTime := time.Now() | ||
next.ServeHTTP(w, r) | ||
duration := time.Since(startTime) | ||
scoped, ok := ld.GetScopedClient(r.Context()) | ||
if !ok { | ||
return | ||
} | ||
_ = scoped.TrackMetric("http.request.duration_ms", float64(duration.Milliseconds()), ldvalue.Null()) | ||
}) | ||
} | ||
|
||
// TrackErrorResponses sends a LD event "http.response.4xx" or "http.response.5xx" if the response code is 4xx or 5xx. | ||
// This middleware must be after AddScopedClientForRequest in the middleware chain, as it uses the scoped client | ||
// from the Go context. | ||
// | ||
// The error event will include all LaunchDarkly contexts added to the scoped client. You may add more | ||
// contexts to the scoped client _during_ the request, and they will be included in the error event sent | ||
// when the request completes. | ||
func TrackErrorResponses(next http.Handler) http.Handler { | ||
return http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) { | ||
metrics := httpsnoop.CaptureMetrics(next, w, r) | ||
if metrics.Code < 400 { | ||
return | ||
} | ||
scoped, ok := ld.GetScopedClient(r.Context()) | ||
if !ok { | ||
return | ||
} | ||
if metrics.Code < 500 { | ||
_ = scoped.TrackEvent("http.response.4xx") | ||
return | ||
} | ||
_ = scoped.TrackEvent("http.response.5xx") | ||
}) | ||
} |
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Bug: Incorrect Go Version Reference in Step Name
The 'Setup Go' step name references
inputs.go-version
. This workflow uses a matrix strategy for Go versions, somatrix.go-version
is the correct context. While this doesn't affect functionality, the step name is misleading.