-
Notifications
You must be signed in to change notification settings - Fork 15
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
Beejones/service request must show body in debug #287
Beejones/service request must show body in debug #287
Conversation
Privacy-sandbox-dev fix is still in flight so I'll let you know when it's ready. This PR looks like it's more about adding tests for the unwrap endpoint, can it be two separate PRs? |
…ave valid json including empty body
It looks like there's still a lot of changes from #288 I think you can drop changes in all files apart from:
|
This is not related to only unwrapping (also /key ). While debugging unwrapping, I noticed the body was not listed in the Request debug message. This is now fixed. |
As mentioned in my comment after the one you've quoted, it looks like there are still changes in this PR corresponding to the unwrap key test PR |
I was working on one machine on both PRs. They must have spoiled. I will fix it. |
fixed |
The body was not in the log during debug logging. Fixed