-
-
Notifications
You must be signed in to change notification settings - Fork 242
Add AllowHTTP, set StreamID value #205
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
base: master
Are you sure you want to change the base?
Add AllowHTTP, set StreamID value #205
Conversation
Add extra params to NewClientProfile
Add extra logics to expose AllowHTTP and StreamID
Add extra logics to expose AllowHTTP and StreamID
Add extra logics to expose AllowHTTP and StreamID
Add extra logics to expose AllowHTTP and StreamID
Update internal_browser_profiles.go - Back to previous state
Add extra params to NewClientProfile to be compatible with new changes
gunir
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Wrong file, should be internal_custom_profiles.go
|
@gunir it looks like you mixed up the roundtripper file and the client_options file here? |
Oh I see the roundtripper |
Fix to use the right roundtripper.go file with all the new features
|
There will be some other improvement I will make in the next few days to improve the window size logics |
bogdanfinn
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In addition it seems like you are not using AllowHTTP at all right?
Update client.go - Remove redundant params
Update roundtripper.go - remove redundant params
Yeah, AllowHTTP is just a duplicate way to set initialStreamID to 3, just to fully mimic Firefox, if you don't want allowHTTP I will remove it |
This add new feature to tls-client to be able to configure AllowHTTP and StreamID, there's 2 ways to set:
Or:
This is useful because Firefox 147's HTTP Frame use stream_id = 3 instead of 1, completely match Firefox Fingerprint instead of nearly matching it:
https://tls.peet.ws/api/all
This PR depends on another PR bogdanfinn/fhttp#15 from https://github.com/gunir/fhttp-allowhttp-streamid/tree/fhttp-streamid-fixsmallwindowsizehangfirefox