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

Temp: Temporarily log the request body if something goes wrong #876

Closed
wants to merge 1 commit into from

Conversation

Pchelolo
Copy link
Contributor

Will deploy for 10 minutes to collect some data and then revert.

Bug: https://phabricator.wikimedia.org/T176728

Change-Id: Iacc91a5d902c15db4690601602d7dff02af349cd
@gwicke
Copy link
Member

gwicke commented Sep 28, 2017

@Pchelolo, are you waiting for a review for this?

@Pchelolo
Copy link
Contributor Author

@gwicke I wanna deploy this for 10-15 mins on Monday so that we could get to the bottom of this bug. At this point it seems testing in prod like this is the only option.

@d00rman d00rman changed the title Temp: Temporary log the request body if something goes wrong Temp: Temporarily log the request body if something goes wrong Oct 2, 2017
@d00rman
Copy link
Contributor

d00rman commented Oct 2, 2017

We made this change live on restbase1007 just to collect logs, so closing this.

@d00rman d00rman closed this Oct 2, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
3 participants