fix_headers() now initializes Host header when needed

[karenetheridge]

RFC9112 §3.2-5 says that Host is mandatory, but can be an empty value.

resolves issue #2232.

[karenetheridge]

Looks like perltidy needs to be run in main? Or has the tidyrc been changed?

[kraih]

Rebase on main.

[karenetheridge]

rebased.

[karenetheridge]

can someone review this please?

[christopherraa]

Functionally this looks good to me. No objections these. I only had one question with regards to consistency in alignment. If some of the more senior members of mojo thinks this is unimportant then I have no issue changing this review to an approval.

[christopherraa]

A nitpick, but I see that the rest of the file is mostly has alignment of the expected values and the test description. Perhaps align these two as well?

[kraih]

And it's URL, not url.

[karenetheridge]

I'll change url to URL, but the formatting is purely what perltidy produced.

[karenetheridge]

rebased to new commits in master, and s/url/URL/ fixed. re-ran perltidy with no formatting changes from before.

[christopherraa]

👍 from me since the perltidy check doesn't complain on the formatting.

[Copilot]

Pull Request Overview

This PR ensures that the Host header is properly initialized to an empty string when it is not defined, in accordance with RFC9112 §3.2-5.

• Added tests in t/mojo/request.t to validate fix_headers behavior for the Host header.
• Updated the fix_headers method in lib/Mojo/Message/Request.pm to set the Host header to an empty string if undefined.

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File	Description
t/mojo/request.t	Added tests verifying that fix_headers correctly sets Host.
lib/Mojo/Message/Request.pm	Modified fix_headers to initialize the Host header as needed.