CFE-4648: cf-execd SMTP port configurable

[basvandervlies]

CFE-4648: cf-execd SMTP port configurable

[CLAassistant]

All committers have signed the CLA.

[craigcomstock]

seems reasonable, I do wonder if you shouldn't use the getservbyname() after all just to hedge bets that it couldn't possible be changed in the services database? It does seem prudent to prefer using the "API" as it were instead of hard-coding and assuming.

Squash the commits then good to go I'd say.

[basvandervlies]

I have adjusted the code to try fitrst getservbyname is this ok? If yes then I will squash the commits

[craigcomstock]

@basvandervlies this looks great! A few unit tests are failing though:

test_exec_config_empty: Starting test
test_exec_config_empty: Test failed.
test_exec_config_full: Starting test
test_exec_config_full: Test failed.
test_exec_config_copy: Starting test
test_exec_config_copy: Test failed.
FAIL: exec-config-test

I tried it out myself and the failure looks a bit mysterious currently.

[basvandervlies]

Yes It compiles and runs fines, but maybe I have overseen something. It look for me unrelated

[craigcomstock]

@basvandervlies looks great. Please squash the commits and while you are doing that, maybe nudge the one commit left to conform to our template https://github.com/cfengine/core/blob/master/CONTRIBUTING.md#commit-message-example

something like:

Added a new smtpport body executor control attribute to specify on which port to send mail

Changelog: Title
Ticket: ENT-4648

[craigcomstock]

@basvandervlies I would prefer the commit to be more like what I suggested so that the changelog makes it more clear where you configure the smtp port for cf-execd:

Added a new smtpport body executor control attribute to specify on which port to send mail

Changelog: Title
Ticket: ENT-4648

Thanks for the contribution and the squash!