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

bug fix: when storing large amounts of text or bytes the data would b… #194

Conversation

WouterGlorieux
Copy link
Contributor

@WouterGlorieux WouterGlorieux commented Nov 18, 2019

…e truncated

Problem was because in the generator StreamBase._gen_chunks() the second parameter in the data slice was not incremented with the offset.
Because of this only the first chunk was yielded correctly.

…e truncated

Problem was because in the generator StreamBase._gen_chunks() the second parameter in the data slice was not incremented with the offset.
Because of this only the first chunk was yielded correctly.
@AliabbasMerchant
Copy link
Member

AliabbasMerchant commented Nov 20, 2019

I think @WouterGlorieux , you will have to change the base branch to master. @Alexander255 am I right?

@ntninja
Copy link
Contributor

ntninja commented Nov 20, 2019

Damn, this is so stupid!

Thanks alot @WouterGlorieux for finding this!

I'll take care rebasing, pushing and releasing this ASAP.

@rbval
Copy link

rbval commented Apr 11, 2020

Is this fixed already in some version? Still experiencing this issue when using the add_bytes function.

@ntninja
Copy link
Contributor

ntninja commented Apr 16, 2020

Merged in #206. Sorry for the delay everybody!

@ntninja ntninja closed this Apr 16, 2020
@ntninja
Copy link
Contributor

ntninja commented Apr 17, 2020

This is now part of 0.4.13

@rbval
Copy link

rbval commented Apr 17, 2020

@Alexander255 awesome thanks!

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

4 participants