Exception handler needs to handle exception subclassing
Chưa có ai nhận issue này.
Đánh giá
- Độ khó
- 2/5
- Thời gian dự kiến
- 1-3 giờ
- Mức phù hợp với người mới
- 50/100
Hướng nghiên cứu
Bắt đầu từ phương thức toHttpException được hiển thị trong issue và kiểm tra cách các lớp exception hiện được chọn. Xác minh hành vi của BadRequestHttpException và các lớp con khác của HttpException, sau đó xác nhận rằng một exception bad request tạo ra phản hồi 400 thay vì phản hồi 500.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
The exception handler currently only uses get_class() to determine the type of exception:
/**
* Convert err to HttpException.
*
* @param mixed $err
*
* @return HttpException
*/
protected function toHttpException(Exception $err): HttpException
{
$errClass = get_class($err);
switch ($errClass) {
case HttpException::class:
$httpErr = $err;
break;
// 404 Model not found.
case ModelNotFoundException::class:
$httpErr = $this->convertFromModelNotFound($err);
break;
case QueryException::class:
default:
$httpErr = $this->defaultHttpException();
break;
}
return $httpErr;
}
So throwing a BadRequestHttpException gets serialized to a 500 error instead of a 400 error.
- Ngôn ngữ chính
- PHP
- Star
- 6
- Fork
- 2
- Chỉ số merge pull request
- Không có pull request nào được merge trong 30 ngày
Chuẩn bị môi trường
Dự án này không cung cấp dev container, Dockerfile hay hướng dẫn đóng góp, nên bạn cần tự thiết lập môi trường: hãy bắt đầu từ README và xem hướng dẫn đóng góp lần đầu của chúng tôi để biết các bước chung.
Bắt đầu từ đâu
- Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
- 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.
- Fork repository và làm thay đổi trên một nhánh.
- Mở pull request có tham chiếu số hiệu của issue.
Issue khác của ml-opensource/rest-api-server
-
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 25/100
Tất cả issue của ml-opensource/rest-api-server
Issue tương tự
-
domain/discovery
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 88/100
Maintainer thường phản hồi trong vòng 1 ngày
-
Admin search throws for permissions added through Admin.generalIndexItems('permissions', ...)Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 76/100
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 70/100
neos/flow-development-collection#3639 · 1 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày
-
The PHP version check in prepare.php still allows PHP 7.2 and 7.3Có thể đã có người làm @ekamran đã nhận hôm nay. Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
WordPress/phpunit-test-runner#359 ·
Maintainer thường phản hồi trong vòng 2 ngày
-
[ObjectMapper] Merge the objects that repeated `#[Map]` attributes map …Có thể đã có người làm Có pull request liên kết đang mở hoặc đã được merge. Đang mởObjectMapper
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
symfony/symfony-docs#23390 ·
Maintainer thường phản hồi trong vòng 2 ngày