jjkiers/layer4-proxy
1
0
Fork 0
Code Issues 4 Pull Requests Packages Projects Releases 4 Activity

Fix fragmented packets #11

Closed
jjkiers wants to merge 0 commits from 10-handle-fragmented-packets into main
pull from: 10-handle-fragmented-packets
merge into: jjkiers:main
Conversation 0 Commits - Files Changed -
jjkiers commented 2025-01-09 19:49:44 +00:00
Owner
Copy Link

Should address #10.

Should address #10.
jjkiers added 4 commits 2025-01-09 19:49:44 +00:00 
Signed-off-by: Jacob Kiers <code@kiers.eu>
This also fixes a build failure which would otherwise happen due to the
time crate.

Signed-off-by: Jacob Kiers <code@kiers.eu>
When a listener is configured to deal with TLS upstreams, we use the SNI
field of the TLS ClientHello message to decide where to send the traffic.

Therefore, a buffer of 1024 bytes was used to temporarily store this
message. However, a TLS ClientHello message can be larger than that, up
to 16K bytes.

So now the first few bytes are read and manually parsed to find out how
long the message is. And then the entire ClientHello message is
retrieved.

So hopefully that will fix the issue causing the ClientHello
determination to fail.

Closes #10

Signed-off-by: Jacob Kiers <code@kiers.eu>
Update based on lints
All checks were successful
continuous-integration/drone/push Build is passing
Details
continuous-integration/drone/pr Build is passing
Details
2c6ad1d7b8 
Signed-off-by: Jacob Kiers <code@kiers.eu>
jjkiers force-pushed 10-handle-fragmented-packets from 2c6ad1d7b8 to 1a9ca771ac 2025-01-09 19:57:03 +00:00 Compare
jjkiers closed this pull request 2025-01-09 20:14:32 +00:00
jjkiers deleted branch 10-handle-fragmented-packets 2025-01-09 20:14:40 +00:00
All checks were successful
continuous-integration/drone/pr Build is passing
Details
continuous-integration/drone/push Build is passing
Details

Pull request closed

This pull request cannot be reopened because the branch was deleted.
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
bug

Something is not working

  
duplicate

This issue or pull request already exists

  
enhancement

New feature

  
help wanted

Need some help

  
invalid

Something is wrong

  
question

More information is needed

  
wontfix

This won't be fixed

No Label
bug
duplicate
enhancement
help wanted
invalid
question
wontfix
Milestone
Clear milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
No Assignees
1 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: jjkiers/layer4-proxy#11
No description provided.
Delete Branch "10-handle-fragmented-packets"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?