Also please edit your initial comment above to includeFixes: #1914 as the last line in the comment. This instructsgit to close that Issue upon the merge of this PR. Thanks.

Edited the comment, thank you.

Is removing thetrying another port part of the log a good idea, since that would be consistent withPermission to listen on port 403 denied? Or would you suggest having this clause in both the cases?

I'm only suggesting the clause's removal whenself.port_retries == 0. When non-zero we should indicate other ports will be attempted, but not when retries are disabled. I believe the permission message is fine as-is - although I suppose one could argue that it too should include the 'trying another port` clause (only when retries are enabled). That said, you typically want to provide only the necessary information in security-related messaging, so perhaps leaving that one alone is probably best given it didn't include the clause in the first place.

@kevin-bates - I have added the changes you suggested, in the PR. I made a new commit instead of rebasing, in order to highlight my new changes. I will rebase the commits if needed after you approve it. Thank you.