[receiver/sqlqueryreceiver]: break up `datasource` parameter into separate fields

[justinianvoss22]

Component(s)

receiver/sqlqueryreceiver

Is your feature request related to a problem? Please describe.

Right now, the datasource string does not have strong validation for the values in this string. We cannot directly pass encrypted passwords, send usernames, send additional options, without assembling the connection string with all of them. The current implementation breaks down when we start using encrypted sensitive values.

Describe the solution you'd like

My solution is to add an additional datasource_config parameter. This will have a struct DataSourceConfig struct that will have the fields: host, port, database, username, password, and additional_params. The struct will be parsed into the connection string in URL format. It is parsed differently depending on what driver is used.

Note: The receiver will use the datasource parameter if it is specified. If datasource is empty, then it will use the datasource_config struct parsed into a string.

Describe alternatives you've considered

An alternative would be to replace the existing datasource string value with this new struct. However, it makes sense to keep the same functionality, and check if datasource is empty in order to use the datasource_config struct.

Additional context

PR for this issue:

[github-actions]

Pinging code owners for receiver/sqlquery: @dmitryax @crobert-1. See Adding Labels via Comments if you do not have permissions to add labels yourself. For example, comment '/label priority:p2 -needs-triaged' to set the priority and remove the needs-triaged label.

[crobert-1]

Hello @justinianvoss22, sorry for the delayed response. My strong preference is to not make this a breaking change, but a supplemental way to configure the receiver. 👍

[justinianvoss22]

@crobert-1 Sounds good, I agree.