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

Add format option to CustomRequestLog for request authority and request authority scheme #9980

Closed
joakime opened this issue Jun 27, 2023 · 2 comments · Fixed by #12322
Closed
Assignees

Comments

@joakime
Copy link
Contributor

joakime commented Jun 27, 2023

Jetty version(s)
10

Enhancement Description
The CustomRequestLog lacks the ability to output:

  • the Request authority (which might differ from the Host header).
  • the Request scheme
  • the entire Request HttpURI (including query)
  • the entire Request HttpURI (excluding query)
Copy link

This issue has been automatically marked as stale because it has been a
full year without activity. It will be closed if no further activity occurs.
Thank you for your contributions.

@github-actions github-actions bot added the Stale For auto-closed stale issues and pull requests label Jun 27, 2024
@ofrias
Copy link

ofrias commented Jun 27, 2024

Please don't close, I think this would be helpful...

@olamy olamy removed the Stale For auto-closed stale issues and pull requests label Jun 27, 2024
@joakime joakime removed this from Jetty 12.0.14 Sep 25, 2024
lachlan-roberts added a commit that referenced this issue Sep 27, 2024
Signed-off-by: Lachlan Roberts <lachlan.p.roberts@gmail.com>
@joakime joakime moved this to 🏗 In progress in Jetty 12.0.15 FROZEN Sep 27, 2024
lachlan-roberts added a commit that referenced this issue Oct 21, 2024
Issue #9980 - add additional codes for CustomRequestLog
@github-project-automation github-project-automation bot moved this from 🏗 In progress to ✅ Done in Jetty 12.0.15 FROZEN Oct 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
No open projects
Status: ✅ Done
Development

Successfully merging a pull request may close this issue.

4 participants