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

listzone may return null id/email for owner field #213

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

Conversation

xgzeng
Copy link

@xgzeng xgzeng commented Feb 27, 2023

I got ListZone response returned from cloudflare with null value for owner.id and owner.email.
The situation causes list zone always fail.

@heavycharged
Copy link

I'm also meet with this problem. @jeff-hiner can you please help to check and merge this PR?

@jeff-hiner
Copy link
Collaborator

I'm trying to ask around to see if that's intentional, because I don't think those fields should be blank. Are you seeing this on a specific account?

@heavycharged
Copy link

Yes, this is on my personal account, as well as on another account that we use on another project. This is a JSON that i dumped before it got parsed:

        ...
        "multiple_railguns_allowed": false
      },
      "owner": {
        "id": null,
        "type": "user",
        "email": null
      },
      "account": {
        "id": "16e2b4933axxxxxxxxxxxxxxxxxxxxxx",
     ...

Also, docs states that all of this fields are not required, and therefore can be null:

image

@jeff-hiner
Copy link
Collaborator

I've asked internally, and this appears to be an API bug. If the owner field is present, it should have an id and name.

@heavycharged
Copy link

Hmm, interesting. Thanks for investigation. Do you have any ETA when this expected to be fixed?

@jeff-hiner
Copy link
Collaborator

I don't have an ETA on this, but the upstream bug has been acknowledged and the work is ticketed out.

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.

3 participants