Skip to content

Debug improvements in the API#70

Merged
josephschorr merged 2 commits intoauthzed:mainfrom
josephschorr:debug-improvements
Jun 20, 2023
Merged

Debug improvements in the API#70
josephschorr merged 2 commits intoauthzed:mainfrom
josephschorr:debug-improvements

Conversation

@josephschorr
Copy link
Copy Markdown
Member

No description provided.

Will be used to provide timing information for debugging
// The request failed because it caused the maximum depth allowed to be
// exceeded. This typically indicates that there is a circular data traversal
// somewhere in the schema, but can also be raised if the data traversal is simply
// too deep.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe mention there's a flag to increase this limit?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The error message itself will point to a docs page mentioning the flag

Copy link
Copy Markdown
Contributor

@ecordell ecordell left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@josephschorr josephschorr merged commit dc592e1 into authzed:main Jun 20, 2023
@josephschorr josephschorr deleted the debug-improvements branch June 20, 2023 16:46
@github-actions github-actions bot locked and limited conversation to collaborators Jun 20, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants