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

[Logging] Add logging to DownstreamApi #2148

Closed
jennyf19 opened this issue Mar 27, 2023 · 2 comments · Fixed by #2206
Closed

[Logging] Add logging to DownstreamApi #2148

jennyf19 opened this issue Mar 27, 2023 · 2 comments · Fixed by #2206
Assignees
Labels
enhancement New feature or request

Comments

@jennyf19
Copy link
Collaborator

Add more logging to DownstreamAPI, as currently it's difficult to debug issues.

After updating this sample, find places to add more logging in DownstreamAPI, especially in the case of misconfiguration and failures...for example not having the scopes in a collection.

@jennyf19 jennyf19 added the enhancement New feature or request label Mar 27, 2023
@westin-m westin-m self-assigned this Apr 6, 2023
@westin-m
Copy link
Contributor

westin-m commented Apr 10, 2023

leave the test app for an e2e
provoke different errors

  • add a space in BaseUrl
  • use wrong scopes(token acquisition)
  • spelling errors for the API

See what errors you get, add logging for that area and move to the next error.
Push up the additional logging and do a PR but remove the sample from the project first

@jennyf19
Copy link
Collaborator Author

@westin-m Let's make sure we have a log message here maybe as part of an else, to say that the call is unauthenticated. This will help us/partners with debugging. Could be an info/debug level log message.

@westin-m westin-m linked a pull request Apr 18, 2023 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants