Use new DNSListResponse object when paginating

[MrMortbury]

Removes the reuse of the DNSListResponse object when collating multiple pages of API requests as this was reusing the pointers used for the Proxied field.

Description

Fixes #1221

Move the declaration of the DNSListResponse variable to within the pagination loop to for go to create a new variable on each loop and stop the Proxied pointer of DNS records from being reused. A new variable of lastResultInfo was added outside the loop to allow the return of the ResultsInfo from the last page of the requests.

Has your change been tested?

Updated the TestListDNSRecordsPagination test and associated fixtures to replicate the issue describe in the bug. Test now pulls 2 pages and a total of 5 test records configured to replicate #1221. Test failed before the change and passed afterwards.
All other DNS tests have been run and pass without issue

Types of changes

What sort of change does your code introduce/modify?

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

• My code follows the code style of this project.
• My change requires a change to the documentation.
• I have updated the documentation accordingly.
• I have added tests to cover my changes.
• All new and existing tests passed.
• This change is using publicly documented (api.cloudflare.com or developers.cloudflare.com) and stable APIs.

[github-actions]

changelog detected ✅

[jacobbednarz]

thanks a bunch @MrMortbury 🍰

[github-actions]

This functionality has been released in v0.62.0.

For further feature requests or bug reports with this functionality, please create a new GitHub issue following the template. Thank you!