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.

Sign up for GitHub

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

Jump to bottom

daily trace summary report URI may not reflect the original REQUEST_URI #26

Open
bobbingwide opened this issue Mar 3, 2016 · 1 comment
Open

daily trace summary report URI may not reflect the original REQUEST_URI #26

bobbingwide opened this issue Mar 3, 2016 · 1 comment
Assignees
@bobbingwide
Labels
enhancement

Comments

@bobbingwide
Copy link
Owner

If you change the value of $_SERVER['REQUEST_URI'] during the processing of a transaction then this new value will appear in the daily trace summary report. This may or may not be a good thing.

Think about it and report back.
@bobbingwide bobbingwide self-assigned this Dec 12, 2019
@bobbingwide
Copy link
Owner Author

The function that uses the REQUEST_URI is bw_trace_determine_request().
The solution will have to save the original values at startup.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@bobbingwide bobbingwide

Labels
enhancement
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@bobbingwide