[GH-ISSUE #1504] Don't have PROPFIND method http #3931

Closed
opened 2026-02-28 11:58:21 +03:00 by kerem · 2 comments
Owner

Originally created by @nikolay-mansas on GitHub (Dec 29, 2025).
Original GitHub issue: https://github.com/0xJacky/nginx-ui/issues/1504

Image
Image

In structured access logs method is not shown, just an empty field

Server Info
Uptime: 0d 7h 9m
Load Average: 1min:1.07 | 5min:0.60 | 15min:0.30
OS: debian 12.12 (linux 6.8.12-17-pve x86_64) (proxmox lxc)
CPU: Intel(R) Xeon(R) CPU X5675 @ 3.07GHz * 4

Version: 2.3.2 (512)

Originally created by @nikolay-mansas on GitHub (Dec 29, 2025). Original GitHub issue: https://github.com/0xJacky/nginx-ui/issues/1504 ![Image](https://github.com/user-attachments/assets/d1f1f279-eabc-444b-beaa-38127b3774c0) <img width="1680" height="48" alt="Image" src="https://github.com/user-attachments/assets/ccfa071a-796d-4610-94df-d86cc16804fd" /> In structured access logs method is not shown, just an empty field Server Info Uptime: 0d 7h 9m Load Average: 1min:1.07 | 5min:0.60 | 15min:0.30 OS: debian 12.12 (linux 6.8.12-17-pve x86_64) (proxmox lxc) CPU: Intel(R) Xeon(R) CPU X5675 @ 3.07GHz * 4 Version: 2.3.2 (512)
kerem 2026-02-28 11:58:21 +03:00
  • closed this issue
  • added the
    bug
    label
Author
Owner

@0xJacky commented on GitHub (Feb 6, 2026):

@cursoragent fix it

<!-- gh-comment-id:3861017754 --> @0xJacky commented on GitHub (Feb 6, 2026): @cursoragent fix it
Author
Owner

@cursor[bot] commented on GitHub (Feb 6, 2026):

修复已完成并推送。让我总结一下所做的更改:

修复总结

问题:PROPFIND 方法在结构化访问日志中不显示,方法字段为空。

原因:日志解析器中的 ValidHTTPMethods 映射表只包含标准 HTTP 方法(GET、POST、PUT 等),不包含 WebDAV 协议使用的方法(如 PROPFIND)。

修复内容

  1. internal/nginx_log/parser/types.go:在 ValidHTTPMethods 映射表中添加了 WebDAV 方法:

    • PROPFIND, PROPPATCH, MKCOL, COPY, MOVE, LOCK, UNLOCK
  2. internal/nginx_log/indexer/parallel_indexer_optimized.go:同步更新了索引器中的方法验证逻辑

  3. internal/nginx_log/parser/parser_test.go:添加了 WebDAV 方法的测试用例确保正确解析

所有测试均已通过,更改已提交并推送到 cursor/propfind-method-logging-33b5 分支。

View PR

Open in Cursor Open in Web

The fix has been completed and pushed. Let me summarize the changes made:

Repair Summary

Question: The PROPFIND method is not displayed in the structured access log, and the method field is empty.
Reason: The ValidHTTPMethods mapping table in the log parser only includes standard HTTP methods (such as GET, POST, PUT, etc.), but does not include the methods used by the WebDAV protocol (such as PROPFIND).
Fixing Content :
internal/nginx_log/parser/types.go: Added the WebDAV methods to the ValidHTTPMethods mapping table.    - PROPFIND, PROPPATCH, MKCOL, COPY, MOVE, LOCK, UNLOCK

  1. internal/nginx_log/indexer/parallel_indexer_optimized.go : Synchronously updated the method validation logic within the indexer.
  2. internal/nginx_log/parser/parser_test.go : Added test cases for the WebDAV method to ensure correct parsing
    All tests have been passed, the changes have been submitted and pushed to the cursor/propfind-method-logging-33b5 branch.

View PR

Open in Cursor  Open in Web

<!-- gh-comment-id:3861018196 --> @cursor[bot] commented on GitHub (Feb 6, 2026): 修复已完成并推送。让我总结一下所做的更改: ## 修复总结 **问题**:PROPFIND 方法在结构化访问日志中不显示,方法字段为空。 **原因**:日志解析器中的 `ValidHTTPMethods` 映射表只包含标准 HTTP 方法(GET、POST、PUT 等),不包含 WebDAV 协议使用的方法(如 PROPFIND)。 **修复内容**: 1. **`internal/nginx_log/parser/types.go`**:在 `ValidHTTPMethods` 映射表中添加了 WebDAV 方法: - PROPFIND, PROPPATCH, MKCOL, COPY, MOVE, LOCK, UNLOCK 2. **`internal/nginx_log/indexer/parallel_indexer_optimized.go`**:同步更新了索引器中的方法验证逻辑 3. **`internal/nginx_log/parser/parser_test.go`**:添加了 WebDAV 方法的测试用例确保正确解析 所有测试均已通过,更改已提交并推送到 `cursor/propfind-method-logging-33b5` 分支。 <p><a href="https://github.com/0xJacky/nginx-ui/pull/1546"><picture><source media="(prefers-color-scheme: dark)" srcset="https://cursor.com/assets/images/view-pr-dark.png"><source media="(prefers-color-scheme: light)" srcset="https://cursor.com/assets/images/view-pr-light.png"><img alt="View PR" width="86" height="28" src="https://cursor.com/assets/images/view-pr-light.png"></picture></a></p> <p><a href="https://cursor.com/background-agent?bcId=bc-16073230-57fe-4ed8-8b83-9cd7f2100808"><picture><source media="(prefers-color-scheme: dark)" srcset="https://cursor.com/assets/images/open-in-cursor-dark.png"><source media="(prefers-color-scheme: light)" srcset="https://cursor.com/assets/images/open-in-cursor-light.png"><img alt="Open in Cursor" width="131" height="28" src="https://cursor.com/assets/images/open-in-cursor-dark.png"></picture></a>&nbsp;<a href="https://cursor.com/agents?id=bc-16073230-57fe-4ed8-8b83-9cd7f2100808"><picture><source media="(prefers-color-scheme: dark)" srcset="https://cursor.com/assets/images/open-in-web-dark.png"><source media="(prefers-color-scheme: light)" srcset="https://cursor.com/assets/images/open-in-web-light.png"><img alt="Open in Web" width="114" height="28" src="https://cursor.com/assets/images/open-in-web-dark.png"></picture></a></p> The fix has been completed and pushed. Let me summarize the changes made: ## Repair Summary **Question**: The PROPFIND method is not displayed in the structured access log, and the method field is empty. **Reason**: The `ValidHTTPMethods` mapping table in the log parser only includes standard HTTP methods (such as GET, POST, PUT, etc.), but does not include the methods used by the WebDAV protocol (such as PROPFIND). **Fixing Content** : `internal/nginx_log/parser/types.go`: Added the WebDAV methods to the `ValidHTTPMethods` mapping table.    - PROPFIND, PROPPATCH, MKCOL, COPY, MOVE, LOCK, UNLOCK 2. **`internal/nginx_log/indexer/parallel_indexer_optimized.go`** : Synchronously updated the method validation logic within the indexer. 3. **`internal/nginx_log/parser/parser_test.go`** : Added test cases for the WebDAV method to ensure correct parsing All tests have been passed, the changes have been submitted and pushed to the `cursor/propfind-method-logging-33b5` branch. <p><a href="https://github.com/0xJacky/nginx-ui/pull/1546"><picture><source media="(prefers-color-scheme: dark)" srcset="https://cursor.com/assets/images/view-pr-dark.png"><source media="(prefers-color-scheme: light)" srcset="https://cursor.com/assets/images/view-pr-light.png"><img alt="View PR" width="86" height="28" src="https://cursor.com/assets/images/view-pr-light.png"></picture></a></p> <p><a href="https://cursor.com/background-agent?bcId=bc-16073230-57fe-4ed8-8b83-9cd7f2100808"><picture><source media="(prefers-color-scheme: dark)" srcset="https://cursor.com/assets/images/open-in-cursor-dark.png"><source media="(prefers-color-scheme: light)" srcset="https://cursor.com/assets/images/open-in-cursor-light.png"><img alt="Open in Cursor" width="131" height="28" src="https://cursor.com/assets/images/open-in-cursor-dark.png"></picture></a>&nbsp; <a href="https://cursor.com/agents?id=bc-16073230-57fe-4ed8-8b83-9cd7f2100808"><picture><source media="(prefers-color-scheme: dark)" srcset="https://cursor.com/assets/images/open-in-web-dark.png"><source media="(prefers-color-scheme: light)" srcset="https://cursor.com/assets/images/open-in-web-light.png"><img alt="Open in Web" width="114" height="28" src="https://cursor.com/assets/images/open-in-web-dark.png"></picture></a></p>
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
starred/nginx-ui#3931
No description provided.