Skip to content

add frame fragmentation support (continuation) #21

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

Merged
merged 2 commits into from
Jun 15, 2022

Conversation

AlexanderShirokih
Copy link
Contributor

Apache Tomcat server cuts responce into frames with size up to 8192 bytes. So i can't receive responce in a single frame and i wrote some piece of code to be able receive multiple frames in one message.

@AlexanderShirokih
Copy link
Contributor Author

AlexanderShirokih commented Jan 11, 2020

I have a bug in first commit, when bis.read(byte[]) reads array not fully. So i bring you version with for loop. Now it seems to work fine.

@vdbemden
Copy link

I used the frame fragmentation support in my own code. This worked very well and saved me from developping it on my own. Please consider commiting it to your repository !

@romichandra
Copy link

guys any updates on this PR?

@gusavila92
Copy link
Owner

gusavila92 commented Jun 14, 2022

Hey folks thanks for looking into this. That fragment support was one of main things I wanted to do when I developed this library, but due to the lack of time I wasn't able to do it. I'm reactivating my work on this library so I will set up an Apache Tomcat server to test it as soon as possible with many test cases

@gusavila92 gusavila92 merged commit ceb0992 into gusavila92:master Jun 15, 2022
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.

4 participants