[beats receivers] beatsauth extension used in translation doesn't understand the proxy_url type of type *url.URL

[cmacknz]

• Relates Add proxy tests to beatsauth extension  beats#46791

// proxy_url on newConfig is of type *url.URL which is not understood by beatsauth extension
	// this logic here converts it into string type similar to what a user would set on filebeat config
	if defaultTransportSettings.Proxy.URL != nil {

Follow up from elastic/beats#46791 to make sure the translation in Elastic Agent is handled in the necessary way.

[elasticmachine]

Pinging @elastic/elastic-agent-data-plane (Team:Elastic-Agent-Data-Plane)

[cmacknz]

@khushijain21 what are the implications of this problem for users that want to write a beatsauth configuration by hand? In that case we are using the native collector configuration parsing so I assume it's not a problem? Do we have a test for that case?

We will want to allow copy/paste migration to beats receivers and beatsauth helps that quite a lot to avoid needing people to manually translate configuration so just double checking.

[khushijain21]

There would be no implications when the config is written by hand - the proxy_url is of string type in both beat and beatreceiver config.

This problem is unique to our translation logic because of how we unpack the defaults here https://github.com/elastic/beats/blob/main/libbeat/otelbeat/beatconverter/beatconverter.go#L257. Unpack method returns proxy_url of *url.URL type and hence the additonal logic to convert it to sring type.

[khushijain21]

This will now be closed by #10343 via 3616b35

[pkoutsovasilis]

@khushijain21 #10343 (comment)

[cmacknz]

Yes we'll have to bring this back separately and it's important it has a test. In agent we have pretty thorough proxy tests for the connection to Fleet server that could be used for inspiration

elastic-agent/testing/integration/ess/proxy_url_test.go

Line 37 in f04ecd3

func TestProxyURL(t *testing.T) {