[pkg/stanza] syslog input not initializing OctetMultiLineBuilder

[timannguyen]

Component(s)

pkg/stanza

What happened?

Description

the following change in https://github.com/open-telemetry/opentelemetry-collector-contrib/blob/main/pkg/stanza/operator/input/syslog/syslog.go#L64-L66 is being over written on https://github.com/open-telemetry/opentelemetry-collector-contrib/blob/main/pkg/stanza/operator/input/syslog/syslog.go#L68. the lines

if syslogParserCfg.EnableOctetCounting {
	tcpInputCfg.MultiLineBuilder = OctetMultiLineBuilder
}

should be moved to set after

tcpInputCfg.BaseConfig = *c.TCP

to prevent being overwritten.

Steps to Reproduce

enable enable_octet_counting = true and the multiline builder is using the default.

Expected Result

Actual Result

Collector version

v0.81.0

Environment information

Environment

OS: (e.g., "Ubuntu 20.04")
Compiler(if manually compiled): (e.g., "go 14.2")
go 20.0

OpenTelemetry Collector configuration

No response

Log output

No response

Additional context

No response

[github-actions]

Pinging code owners:

• pkg/stanza: @djaglowski

See Adding Labels via Comments if you do not have permissions to add labels yourself.

[timannguyen]

i'll push a fix for it. if someone can assign this to me. :)

[djaglowski]

Thanks @timannguyen. I can't assign the issue to you because you are not yet a member of the OpenTelemetry org, but I'll happily review your PR.