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

Include logging of x-fb-debug, x-fb-rev #648 #686

Merged
merged 6 commits into from
May 1, 2024

Conversation

pri1012
Copy link
Contributor

@pri1012 pri1012 commented Mar 27, 2024

Description:

Add additional headers for debugging.

Changes included:
Includes x-fb-trace-id and x-fb-debug x-fb-rev as part of the error_message returned

[X] My PR has tests and they pass!

@saikambaiyyagari
Copy link

@ylecuyer @arsduo Could you please help review this PR?
Context: #648

@saikambaiyyagari
Copy link

@ylecuyer @arsduo Could you please help review this PR?

@zenspider
Copy link
Contributor

@ylecuyer & @arsduo nudge. what if anything can we do to help move these PRs along? Is there anything I can do to help get this moving?

@ylecuyer ylecuyer merged commit 64d17a8 into arsduo:master May 1, 2024
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants