potential bug in newAppRun

[rittneje]

I came across the following code while trying to diagnose an oddity we are seeing with "expected status codes".

go-agent/v3/newrelic/app_run.go

Lines 102 to 108 in 424d175

	if run.Config.ErrorCollector.IgnoreStatusCodes != nil {
	run.mu.Lock()
	for _, errorCode := range run.Config.ErrorCollector.ExpectStatusCodes {
	run.expectErrorCodesCache[errorCode] = true
	}
	run.mu.Unlock()
	}

Should that instead say run.Config.ErrorCollector.ExpectStatusCodes != nil?

[iamemilio]

Seems like a likely bug, we will make a ticket for it.

[mirackara]

Released fix with #924. Closing this out for now!