Skip to content

veb: fix #26091#26112

Merged
spytheman merged 2 commits into
vlang:masterfrom
enghitalo:fix/26091
Dec 25, 2025
Merged

veb: fix #26091#26112
spytheman merged 2 commits into
vlang:masterfrom
enghitalo:fix/26091

Conversation

@enghitalo

Copy link
Copy Markdown
Contributor

fix #26091

@enghitalo

Copy link
Copy Markdown
Contributor Author

net module needs a deep cleaning 😢

@Jengro777

Copy link
Copy Markdown
Contributor

This might be an opportunity to optimize the net library.

@enghitalo

Copy link
Copy Markdown
Contributor Author

This might be an opportunity to optimize the net library.

Unfortunately, this could take weeks or months.

@Jengro777

Jengro777 commented Dec 25, 2025

Copy link
Copy Markdown
Contributor

#26098
This can make neb_veb work properly, and I am using it locally.

Optimize the net well so that it is worth it, and lay a solid foundation for the future.

@enghitalo

Copy link
Copy Markdown
Contributor Author

IMHO, new_veb can work properly by avoiding the net module as much as possible

Comment thread vlib/net/http/request.v

@spytheman spytheman left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Excellent. Thanks for the tests 🙇🏻 .

@spytheman spytheman merged commit bfb9b65 into vlang:master Dec 25, 2025
74 of 75 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[new_veb] net.http.HeaderKeyError when parsing JSON request body

3 participants