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 support for showing request and response payload #184

Merged
merged 2 commits into from Sep 3, 2014

Conversation

@lloydbenson
Copy link
Contributor

lloydbenson commented Sep 3, 2014

No description provided.

@geek

This comment has been minimized.

Copy link
Member

geek commented Sep 3, 2014

Make sure to update the readme

@lloydbenson

This comment has been minimized.

Copy link
Contributor Author

lloydbenson commented Sep 3, 2014

thx for the reminder!

geek added a commit that referenced this pull request Sep 3, 2014
add support for showing request and response payload
@geek geek merged commit 1d36fa2 into hapijs:master Sep 3, 2014
@paolochiodi

This comment has been minimized.

Copy link

paolochiodi commented Oct 7, 2014

Sorry for commenting on a closed pull request, but there's a way to exclude several fields from the payload (such as: not logging passwords and sensitive data)?

@arb

This comment has been minimized.

Copy link
Contributor

arb commented Oct 7, 2014

Not in this version of good. It's either all or nothing. In Good 3, you can write your own reporter that will let you slice and dice data however you want it.

@paolochiodi

This comment has been minimized.

Copy link

paolochiodi commented Oct 7, 2014

thank you

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants
You can’t perform that action at this time.