fix(net/ghttp): Server Domain if is empty str, bind handler pattern will add @ which is not expect #4100

[elonnzhang]

#4100

[gqcn]

It might have something to do with the regular expression at

gf/net/ghttp/ghttp_server.go

Line 367 in 00e2f6b

array, _ := gregex.MatchString(`(.*?)%([A-Z]+):(.+)@(.+)`, k)

The regular expression might better be (.*?)%([A-Z]+):(.+?)@(.+).
You also need commit associated unit testing case for this update.

[Copilot]

Pull Request Overview

This PR fixes an issue where binding handler patterns would incorrectly include an "@" character when the server domain is an empty string. The changes refactor the string concatenation logic by introducing a helper function that conditionally adds the "@" separator.

• Replaces inline concatenations with the new helper function patternBindDomain.
• Updates all handler binding methods to use the helper function.
• Introduces patternBindDomain to conditionally format the handler pattern.

Comments suppressed due to low confidence (2)

net/ghttp/ghttp_server_domain.go:170

• [nitpick] Consider adding a brief documentation comment for the helper function 'patternBindDomain' to explain its behavior, especially its handling of empty domain strings.

func patternBindDomain(pattern, domain string) string {

net/ghttp/ghttp_server_domain.go:170

• Consider adding unit tests for 'patternBindDomain', ensuring that the function behaves correctly with both empty and non-empty domain strings.

func patternBindDomain(pattern, domain string) string {