Skip to content

fix: filter requests by hostname instead of path#4255

Open
manalkaff wants to merge 1 commit intoDokploy:canaryfrom
manalkaff:fix/requests-filter-by-hostname
Open

fix: filter requests by hostname instead of path#4255
manalkaff wants to merge 1 commit intoDokploy:canaryfrom
manalkaff:fix/requests-filter-by-hostname

Conversation

@manalkaff
Copy link
Copy Markdown
Contributor

@manalkaff manalkaff commented Apr 19, 2026

Summary

  • The search filter on the Requests tab was matching against RequestPath (the URL path) instead of RequestHost (the hostname)
  • Updated parseRawConfig in packages/server/src/utils/access-log/utils.ts to filter by log.RequestHost
  • Updated the input placeholder from "Filter by name..." to "Filter by hostname..." to accurately reflect what is being filtered

Test plan

  • Go to the Requests tab in Dokploy
  • Type a hostname (e.g. myapp.example.com) in the filter input — results should filter by hostname now
  • Verify that typing a URL path (e.g. /dashboard) no longer matches unrelated hostnames

Fixes #4249

Greptile Summary

This PR fixes a bug where the search filter on the Requests tab was incorrectly matching against RequestPath (the URL path) instead of RequestHost (the hostname). The server-side filter in parseRawConfig is updated to use log.RequestHost, and the UI input placeholder is updated from "Filter by name..." to "Filter by hostname..." to match.

Confidence Score: 5/5

Safe to merge — a one-line targeted bug fix with no side effects.

RequestHost is confirmed as a valid field in the LogEntry type, the change is minimal and correct, and the placeholder update accurately reflects the new behavior.

No files require special attention.

Reviews (1): Last reviewed commit: "fix: filter requests by hostname instead..." | Re-trigger Greptile

The search filter on the Requests tab was incorrectly filtering by
RequestPath instead of RequestHost, causing "filter by name" to match
URL paths rather than hostnames. Updated the placeholder text to
reflect the correct field being searched.

Fixes Dokploy#4249
@manalkaff manalkaff requested a review from Siumauricio as a code owner April 19, 2026 09:30
@dosubot dosubot bot added the size:XS This PR changes 0-9 lines, ignoring generated files. label Apr 19, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:XS This PR changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

filter by name not actually filtering by name

1 participant