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 decoding issue for request data chunks #4522

Merged
merged 1 commit into from
Dec 27, 2023
Merged

Conversation

Nriver
Copy link
Contributor

@Nriver Nriver commented Dec 14, 2023

No description provided.

@Nriver
Copy link
Contributor Author

Nriver commented Dec 14, 2023

This fixes #4412

@zadam zadam merged commit a378313 into zadam:master Dec 27, 2023
2 checks passed
@zadam
Copy link
Owner

zadam commented Dec 27, 2023

Awesome, great catch! Although I don't understand why this would cause problems specifically in 0.61 and later.

@Nriver
Copy link
Contributor Author

Nriver commented Dec 28, 2023

Perhaps this issue has existed in earlier versions, but I hadn't noticed it until someone pointed it out to me. After being informed about the issue, I started checking my data and confirmed its presence. Additionally, the occurrence of this issue is probabilistic—it becomes more likely as the data reaches a certain quantity.

Another hypothesis is that the long sync issue #4435 resulted in the entire database being repeatedly transferred many times, increasing the likelihood of triggering this problem.

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.

2 participants