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

[PR #7219/208df2c9 backport][stable-7] nsupdate: fix 'index out of range' error when no TTL answer is given #7236

Conversation

patchback[bot]
Copy link

@patchback patchback bot commented Sep 10, 2023

This is a backport of PR #7219 as merged into main (208df2c).

SUMMARY

Fix a possible list index out of range when no answer is returned in the ttl_changed method by applying the existing workaround for NS records to all record types.

Resolves #836

ISSUE TYPE
  • Bugfix Pull Request
COMPONENT NAME

nsupdate

ADDITIONAL INFORMATION

See #836

…7219)

* nsupdate: fix 'index out of range' error when no TTL answer is given

Fix a possible `list index out of range` when no answer is returned in the `ttl_changed` method
by applying the existing workaround for NS records to all record types.

Resolves #836

* fixup! nsupdate: fix 'index out of range' error when no TTL answer is given

(cherry picked from commit 208df2c)
@ansibullbot
Copy link
Collaborator

@ansibullbot ansibullbot added backport bug This issue/PR relates to a bug module module new_contributor Help guide this first time contributor plugins plugin (any type) labels Sep 10, 2023
@felixfontein felixfontein merged commit 63210f4 into stable-7 Sep 11, 2023
@felixfontein felixfontein deleted the patchback/backports/stable-7/208df2c9e6b7e0c0f6b5a7d8543e135a068c581f/pr-7219 branch September 11, 2023 04:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug This issue/PR relates to a bug module module new_contributor Help guide this first time contributor plugins plugin (any type)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants