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

Fix custom MIME type serialization incompatibility (#260) #261

Merged
merged 1 commit into from
May 13, 2024

Conversation

akowal
Copy link
Contributor

@akowal akowal commented Apr 26, 2024

Adjust MIME type name length serialization to ensure compatibility with other RSocket implementations.

Copy link
Member

@whyoleg whyoleg left a comment

Choose a reason for hiding this comment

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

Thank you for finding the issue and fixing it!
The spec is a bit unclear regarding this encoding... But changes you are proposing are correct.

If it's possible could you resolve several comments?

Copy link
Member

@whyoleg whyoleg left a comment

Choose a reason for hiding this comment

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

Once more, big thanks for your contribution!

@whyoleg whyoleg merged commit 167ed77 into rsocket:master May 13, 2024
6 of 12 checks passed
@akowal
Copy link
Contributor Author

akowal commented May 21, 2024

@whyoleg any chance to see a new release including this fix any time soon?

@whyoleg
Copy link
Member

whyoleg commented May 21, 2024

hey @akowal !
Yeah, sorry for the delay, I will release 0.15.5 with this fix and some dependencies updated today/tomorrow.
(timing is unfortunate as there are a lot of things happening at my side currently)

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.

None yet

2 participants