Fix for #6320

[grogi]

Closes #6320

This introduces additional sorting function - sortAddress - that extends the behaviour of existing sortIp. There were two options effectively here

• expand the sortIp and (although unlikely) potentially modify the existing behaviour
• create a new sorting function.

The second approach was selected. The unit tests for the previous implementation pass 100%, there is a new set of tests for the new function as well.

I also cleaned up a bit the code around sorting, added some additional type safety and removed a need for es-lint overrides.

[windsurf-bot]

💡 To request another review, post a new comment with "/windsurf-review".

[windsurf-bot]

Consider adding more context to the error when throwing in strict mode (allowOnlyIpAddresses=true). This would make debugging easier.

Suggested change

	if (allowOnlyIpAddresses) {
	throw e
	}
	if (allowOnlyIpAddresses) {
	throw new Error(`Invalid IP address or CIDR: ${address}`);
	}

[grogi]

Added more comment around the throw e to explain why this is the exactly desired behaviour.

[windsurf-bot]

The wildcard pattern '*.home.fritz.net' is placed before other domain names in the sorted result, which seems inconsistent with the alphabetical sorting of other non-IP strings. Consider either sorting wildcard patterns alphabetically along with other domain names or documenting the special ordering rule if this is intentional behavior.

[grogi]

Yes, this is expected order, consistent with the regular string comparison and what is observed in the application in many places.