Make HTML-Escaping optional
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ó
- 2/5
- Thời gian dự kiến
- 1-3 giờ
- Mức phù hợp với người mới
- 65/100
Hướng nghiên cứu
Start by reading the JSON dump and escaping logic in util.py, then check the tests that cover JSON output. Make the escaping configurable as described, with the default behavior decided consistently with the existing setting. Done when the option controls whether < and > are escaped and the relevant tests pass.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
Is your feature request related to a problem? Please describe.
In the version 0.24.0 the feature "escape < and > on JSON dump" was introduced. We are using oracle provider and the necessary escaping is done on the database side. Because of escaping in pygeoapi our application is working buggy.
Describe the solution you'd like
It would be better to make the escaping optional, f.e. through the environment variable.
Describe alternatives you've considered
Additional context
Now we are forced to use the adjustment of the util.py:
DO_JSON_ESCAPE = os.getenv('JSON_ESCAPE', 'false').lower()
...
...
# do escaping optional
if DO_JSON_ESCAPE in ('true', '1', 'yes'):
LOGGER.debug('Escaping < and >')
json_dump = json_dump.replace('<', '<')
json_dump = json_dump.replace('>', '>')
- Ngôn ngữ chính
- Python
- Star
- 628
- Fork
- 339
- Merge trung bình
- 4 ngày 9 giờ
- Pull request đã merge (30 ngày)
- 8
Chuẩn bị môi trường
- Có Dockerfile hoặc tệp Docker Compose
- 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 geopython/pygeoapi
-
the query parameter limit is not working for jobsCó thể làm lại được Pull request cho issue này đã bị đóng mà không được merge. Đang mởbug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 70/100
Maintainer thường phản hồi trong vòng 1 ngày
-
Performance issues affecting wide table sourcesCó thể đã có người làm @reinvantveer đã nhận hôm nay. Đang mởbug
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 55/100
Maintainer thường phản hồi trong vòng 1 ngày
-
refactor flask/starlette/django testingCó thể đã có người làm @tomkralidis đã nhận 1 ngày trước. Đang mởenhancement tests
geopython/pygeoapi#2468 · 1 bình luận · 1 người được giao ·
Maintainer thường phản hồi trong vòng 1 ngày
-
bug
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 58/100
Maintainer thường phản hồi trong vòng 1 ngày
-
ESRI provider tests: mock ArcGIS HTTP responses instead of depending on sampleserver6Có thể làm lại được @aaditya755 đã nhận 8 ngày trước và không có pull request nào đang mở. Đang mởenhancement
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 38/100
geopython/pygeoapi#2448 · 4 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày
Tất cả issue của geopython/pygeoapi
Issue tương tự
-
area/profiles comp/cli P3 type/feature
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 74/100
NousResearch/hermes-agent#146372 ·
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 65/100
Maintainer thường phản hồi trong vòng 1 ngày
-
[oblt-aw][autodoc] Document obs-aw-automerge contract and fix automerge doc inconsistenciesCó thể đã có người làm @github-actions đã nhận hôm nay. Đ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
-
GeoSeries scale and skew tests fail with pytest 9.2.0Có thể đã có người làm @jiayuasu đã nhận hôm nay. Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 82/100
Maintainer thường phản hồi trong vòng 1 ngày
-
admission-deferred priority: medium status: ready work-class: scoped
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 70/100
melodic-software/claude-code-plugins#10887 · 4 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày