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.

Sign up for GitHub

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

Jump to bottom

rec: Backport 13105 to rec 4.7.x: (I)XFR: handle partial read of len prefix #13157

Merged
merged 3 commits into from Aug 23, 2023
Merged

rec: Backport 13105 to rec 4.7.x: (I)XFR: handle partial read of len prefix #13157

merged 3 commits into from Aug 23, 2023

Conversation

omoerbeek
Copy link
Member

Short description

Backport of #13105

Checklist

I have:

  • read the CONTRIBUTING.md document
  • compiled this code
  • tested this code
  • included documentation (including possible behaviour changes)
  • documented the code
  • added or modified regression test(s)
  • added or modified unit test(s)
  • checked that this code was merged to master

Habbie and others added 3 commits August 21, 2023 11:49
@Habbie @omoerbeek
typo fix
2d9c43f 
(cherry picked from commit 8fb5bba)
@Habbie @omoerbeek
IXFR client: handle partial reads of the TCP chunk length header, plus:
bf5430d 
* add primarySOACount to exception text
* add indicator of current state to exception text
* a test

(cherry picked from commit 8faf5a9)
@omoerbeek
remove redundant assignment
6f28470 
(cherry picked from commit 8d3ab63)
@omoerbeek omoerbeek added this to the rec-4.7.x milestone Aug 21, 2023
@rgacogne rgacogne self-requested a review August 21, 2023 11:56
@omoerbeek omoerbeek merged commit 78b227a into PowerDNS:rel/rec-4.7.x Aug 23, 2023
47 checks passed
@omoerbeek omoerbeek deleted the backport-13105-to-rec-4.7.x branch August 23, 2023 06:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@rgacogne rgacogne rgacogne approved these changes

Assignees
No one assigned
Labels
defect rec
Projects
None yet
Milestone
rec-4.7.x
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@omoerbeek @rgacogne @Habbie