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

supress empty string nextlink url #1104

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

ziyeqf
Copy link

@ziyeqf ziyeqf commented Oct 18, 2024

The chaos return an empty string in the NextLink property instead of set it to null when calling TargetTypes_List
. It will make the provider wait endless...

I have opened an issue on azure-rest-api-specs repo, but can we consider add this workaround to avoid similar bad cases?

@ziyeqf ziyeqf requested a review from a team as a code owner October 18, 2024 07:44
@github-actions github-actions bot added the release-once-merged The SDK should be released once this PR is merged label Oct 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release-once-merged The SDK should be released once this PR is merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant