Yet another IPv6 bug fix.

Instead of using the helper functions here, I've found a regex in the code, it looks like this:

cut_port_re=re.compile(r":\d+$",re.ASCII)

This could perfectly solve the bug. caz currently, we are doing this:

defrequest_port(request):host=request.hosti=host.find(':')ifi>=0:port=host[i+1:]try:int(port)exceptValueError:_debug("nonnumeric port: '%s'",port)returnNoneelse:port=DEFAULT_HTTP_PORTreturnport

And the problem is if the input is IPv6 addr like[::1]:1234, the program will treat:1]:1234 as a port and raise a ValueError when trying toint() it.

Now, since we've got the regex, we could use it directly in this func to make sure that port numberscan only be numbers and fix the bug.

Something like this:

defrequest_port(request):match=cut_port_re.search(request.host)ifmatch:port=match.group(0).removeprefix(':')try:int(port)exceptValueError:_debug("nonnumeric port: '%s'",port)returnNoneelse:port=DEFAULT_HTTP_PORTreturnport

I've also added the tests.

cc@picnixz . Looking forward to your review, thank you for your time and effort in advance.

• Issue:CookieDefaultPolicy.set_port_ok and return_ok_port func does not work for IPv6 addresses #135993

📚 Documentation preview 📚:https://cpython-previews--136076.org.readthedocs.build/