You are correct and it should be so.
But I want to defer this change as an future issue because there are other same usages of `string()` and `non_neg_integer()` for hostnames and port numbers.