Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

✨ Config option to configure the https adaptive window size for requests #529

Conversation

ddotthomas
Copy link
Contributor

@ddotthomas ddotthomas commented Feb 19, 2024

What does this PR do?

Provided a new config option to allow user enable/disable the https window size setting for the requests fetching the search results from the upstream search engines.

Why is this change important?

These change is essential as it allows the user to enable https adaptive window sizes option for requests which improves the performance of requests on reliable/stable internet connections which in turn improves user experience ny reducing latency for responses.

How to test this PR locally?

Check TCP SYN and ACK packets in wireshark for TCP Window Size scaling

Related issues

Closes #527

@neon-mmd neon-mmd changed the title Added config option to enable the reqwest client adaptive window ✨ Config option to configure the https adaptive window size for requests Feb 20, 2024
Copy link
Owner

@neon-mmd neon-mmd left a comment

Choose a reason for hiding this comment

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

Thanks ❤️ for working on this PR. We really appreciate it 👍 Keep it up!! But before we merge this PR, we would like to suggest a few changes. 🙂

websurfx/config.lua Outdated Show resolved Hide resolved
ddotthomas and others added 2 commits February 20, 2024 12:11
Co-authored-by: neon_arch <mustafadhuleb53@gmail.com>
Copy link
Owner

@neon-mmd neon-mmd left a comment

Choose a reason for hiding this comment

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

Sorry for the late reply.
Thanks ❤️ again for working on this. Since everything looks good. We will merge this PR as soon as possible. 🚀 🙂

@neon-mmd neon-mmd merged commit 41ab8a2 into neon-mmd:rolling Feb 28, 2024
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

✨ Config option to use adaptive http window size for requests
3 participants