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

Remove Wreck Logging #402

Closed
arb opened this issue Nov 11, 2015 · 2 comments
Closed

Remove Wreck Logging #402

arb opened this issue Nov 11, 2015 · 2 comments
Milestone

Comments

@arb
Copy link
Contributor

@arb arb commented Nov 11, 2015

It doesn't work right and there are way too many questions and bugs surrounding it at present. Once the issue is resolved, I'll be more than happy to add it back in.

hapijs/wreck#111

@ldesplat

This comment has been minimized.

Copy link

@ldesplat ldesplat commented Nov 11, 2015

I think it may be interesting to add listeners to outgoing http calls instead of just doing Wreck logging.. I've seen that somewhere and may actually cover all cases. The problem is we may be logging way too much, but it's a start?

@arb

This comment has been minimized.

Copy link
Contributor Author

@arb arb commented Nov 11, 2015

Yeah that would be a better solution in the short term for sure.

@arb arb mentioned this issue Nov 11, 2015
@arb arb modified the milestone: 7.0.0 Nov 18, 2015
@arb arb closed this in aa85f14 Mar 14, 2016
arb added a commit that referenced this issue Mar 14, 2016
Removed wreck logging. Closes #402.
krambox added a commit to krambox/good that referenced this issue Mar 15, 2016
Squashed commits:
[aa85f14] Removed wreck logging. Closes hapijs#402.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.