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

#150: ensure rest API errors print the targeted endpoint #151

Merged
merged 2 commits into from
Jan 13, 2023

Conversation

JoernBerkefeld
Copy link
Collaborator

@JoernBerkefeld JoernBerkefeld commented Jan 11, 2023

PR details

What is the purpose of this pull request? (put an "X" next to an item)

  • Improvements

What changes did you make? (Give an overview)

basically ensures we can actually call support when the rest API returns such an error because now we can tell them which endpoint we tried.

added the endpoint as a separate attribute as well as directly inside of the main rest-error message due to its importance

@JoernBerkefeld JoernBerkefeld self-assigned this Jan 11, 2023
@JoernBerkefeld JoernBerkefeld linked an issue Jan 11, 2023 that may be closed by this pull request
@JoernBerkefeld JoernBerkefeld changed the base branch from main to develop January 11, 2023 10:37
lib/util.js Outdated Show resolved Hide resolved
@JoernBerkefeld JoernBerkefeld merged commit 3d49516 into develop Jan 13, 2023
@JoernBerkefeld JoernBerkefeld deleted the 150-include-url-in-rest-errors branch January 26, 2023 15:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

include URL in REST-errors
2 participants