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

CustomRequestLog.ignorePath doesn't work as documented #11490

Closed
ikozar opened this issue Mar 6, 2024 · 1 comment · Fixed by #11638
Closed

CustomRequestLog.ignorePath doesn't work as documented #11490

ikozar opened this issue Mar 6, 2024 · 1 comment · Fixed by #11638
Assignees
@lachlan-roberts
Labels
Question

Comments

@ikozar
Copy link

ikozar commented Mar 6, 2024

Jetty Version
12.0.6

Jetty Environment
core

Java Version
1.17

Question
I'm trying to skip logging /actuator/* endpoints from accesslog, but org.eclipse.jetty.server.CustomRequestLog#_ignorePathMap doesn't match with my request because it uses full URI path - request.getHttpURI().toString() instead of request.getHttpURI().getCanonicalPath()

Could you suggest how to skip the endpoints accesslog?
@ikozar ikozar added the Question label Mar 6, 2024
@joakime joakime changed the title can't use ignorePaths CustomRequestLog.ignorePath doesn't work as documented Mar 20, 2024
joakime added a commit that referenced this issue Apr 9, 2024
@joakime
Issue #11490 - fixing CustomRequestLog.setIgnorePaths
0d35367
@joakime joakime mentioned this issue Apr 9, 2024
Merged
@joakime joakime moved this to 🏗 In progress in Jetty 12.0.9 - FROZEN Apr 9, 2024
joakime added a commit that referenced this issue Apr 10, 2024
@joakime
Issue #11490 - fixing CustomRequestLog.setIgnorePaths (#11638)
83526f7
@github-project-automation github-project-automation bot moved this from 🏗 In progress to ✅ Done in Jetty 12.0.9 - FROZEN Apr 10, 2024
@joakime
Copy link
Contributor

joakime commented Apr 10, 2024

Merged PR #11638
This will be available on Jetty 12.0.9

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

@lachlan-roberts lachlan-roberts

Labels
Question
Projects
No open projects
Jetty 12.0.9 - FROZEN
Status: ✅ Done
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Issue #11490 - fixing CustomRequestLog.setIgnorePaths jetty/jetty.project
3 participants
@joakime @ikozar @lachlan-roberts