Have HTTP 4xx instead of 5xx on SQL syntax error
Maintainer thường phản hồi trong vòng 1 ngày
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
- 45/100
Hướng nghiên cứu
The payload names no source files, so start by reproducing: run a query with a SQL syntax error in the pgAdmin Query Tool and note the HTTP status returned (500) and the JSON body. Then find where the server converts PostgreSQL errors into HTTP responses, and check how the linked upstream issue #9872 handled similar errors. Done means a syntax error returns a 4xx status such as 400 with the same JSON body, and other server errors keep their current status.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
Describe the solution you'd like
Hi,
I create this ticket to follow https://github.com/pgadmin-org/pgadmin4/issues/9872, I'm in case of a corporate IT infra where proxy security cannot be changed and this last replace JSON body response for request whose response code is HTTP 5xx by a proxy specific response.
The security reason is to avoid potentiel technical leak from backend.
In addition in a Rest API design, it is not semantically correct to have a HTTP 500 when pgadmin user enter an incorrect SQL request, a HTTP 400 Bad Request should be more appropriate.
Describe alternatives you've considered
The alternative should be to have pgadmin server answer HTTP 400 Bad Request code and pgadmin webapp interpret JSON as before.
- Ngôn ngữ chính
- Python
- Star
- 3.9k
- Fork
- 914
- Merge trung bình
- 1 ngày 11 giờ
- Pull request đã merge (30 ngày)
- 37
Chuẩn bị môi trường
- Có Dockerfile hoặc tệp Docker Compose
- Không có mẫu pull request
- Đọc hướng dẫn đóng góp
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 pgadmin-org/pgadmin4
-
AI Assistant 'Insert at Cursor' does not insert the SQL into the query editorCó thể đã có người làm @GBra-4-669 đã nhận hôm nay. Đang mởBug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 88/100
pgadmin-org/pgadmin4#10690 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100
pgadmin-org/pgadmin4#10673 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Import/Export Data fails for a file name containing a backslash, and creates a stray file named EĐang mởBug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 82/100
pgadmin-org/pgadmin4#10672 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Helm chart: allow setting ingressClassName on the IngressCó thể đã có người làm @commarla đã nhận 6 ngày trước. Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 85/100
pgadmin-org/pgadmin4#10658 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Feature
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100
pgadmin-org/pgadmin4#10426 · 3 bình luận · 1 reaction ·
Maintainer thường phản hồi trong vòng 1 ngày
Tất cả issue của pgadmin-org/pgadmin4
Issue tương tự
-
enhancement
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100
glossyio/traffic-monitor#264 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Add self-hostĐang mở
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 75/100
Atarity/deploy-your-own-saas#102 ·
-
[Bug]: RDFlex silently drops the covariate adjustment when the outcome or treatment column is integer-typedCó thể đã có người làm @vrjdev đã nhận hôm nay. Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100
DoubleML/doubleml-for-py#424 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
agentic-dmx docs enhancement
Độ khó 1/5 1-3 giờ Mức phù hợp với người mới 86/100
-
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 72/100
521xueweihan/HelloGitHub#4150 ·