Globbing should return 200 OK even if empty

Đang mở
#900 7 bình luận 3 reaction 0 người được giao Xem trên GitHub

Chưa có ai nhận issue này.

Đánh giá

Độ khó
3/5
Thời gian dự kiến
1-2 ngày
Mức phù hợp với người mới
35/100
Loại issue
Lỗi
Độ rõ ràng
Khá rõ ràng
Mức độ hoạt động
Đình trệ
Công nghệ
javascript
Lĩnh vực
api, backend

Hướng nghiên cứu

Bắt đầu bằng cách theo dõi quá trình xử lý GET của server đối với các yêu cầu glob như /notes/note-* và so sánh trường hợp tài nguyên hiện có với trường hợp container rỗng. Xác minh cách một kết quả khớp rỗng được chuyển thành 404, sau đó bổ sung kiểm thử cho thấy một container hiện có nhưng không có tài nguyên nào khớp sẽ trả về 200 OK cùng một danh sách rỗng.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Mô tả

Assuming I have a container named /notes in my POD and this container contains resources matching /notes/note-* I get a nice list of those resources if I do a GET of /notes/note-*.

But if the /notes container is empty and I try GET /notes/note-* I get 404 - not found.

If think that is wrong, The container is there and any attempt to get a filtered list of its content should return 200 OK since the list is there (the container exists) but it is simply empty as there are no matching resources in the container.

Returning 404 tells the client "You made a mistake - there is nothing here". But the client did not make a mistake - it simply asked for an empty list, not a missing-non-existing list.

Ngôn ngữ chính
JavaScript
Star
1.8k
Fork
308
Không có pull request nào được merge trong 30 ngày

Hướng dẫn đóng góp

Mở hướng dẫn đóng góp

Bắt đầu từ đâu

  1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
  2. Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
  3. Fork repository và làm thay đổi trên một nhánh.
  4. Mở pull request có tham chiếu số hiệu của issue.

Issue khác của nodeSolidServer/node-solid-server

Tất cả issue của nodeSolidServer/node-solid-server

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.