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

Add logging of BoxAPIException.getResponse() #1045

Open
jimshowalter opened this issue May 13, 2022 · 1 comment
Open

Add logging of BoxAPIException.getResponse() #1045

jimshowalter opened this issue May 13, 2022 · 1 comment
Assignees
Labels
enhancement Added to issues that describes enhancements

Comments

@jimshowalter
Copy link

The requests are already logged:

It would be very helpful when debugging to also have the error responses logged.

There should be a way to configure this on or off.

There would need to be a way to pass in an optional filter to remove PII from the exception text.

@jimshowalter jimshowalter added the enhancement Added to issues that describes enhancements label May 13, 2022
@antusus
Copy link
Contributor

antusus commented May 18, 2022

Hi @jimshowalter ,

Thanks for submitting this Issue! I've added this feature to our internal backlog.

About the PII filter you would see that as a function that accepts String and returns String.

@antusus

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Added to issues that describes enhancements
Projects
None yet
Development

No branches or pull requests

5 participants