get_collection_map does not parse the transparent query parameter
メンテナーはふだん 1 日以内に返信
評価
調査の方向性
The string transparent is copied into query_args without conversion in pygeoapi/api/maps.py near line 135, and MapScriptProvider in pygeoapi/provider/mapscript_.py around lines 185-188 tests it as a truthy value. Read both, then check how the other parsed parameters are converted. Done means the API passes a real bool and the provider's check works for 'false'. Pull request #2445 is already open against this issue, so review or coordinate with it before writing a new patch.
索引モデルが issue の本文から書いたものです。
説明
When handling an OGC API Maps request, pygeoapi does not parse the transparent query parameter into a bool, as seen in:
As seen above, if request.params['transparent'] exists, then it is stored as is in query_args['transparent'] - i.e. as a string.
This means that map providers are left to parse this parameter, unlike all other parameters, which are parsed by pygeoapi before calling the map provider.
This is also causing a bug in the built-in MapScriptProvider. This provider does not parse the transparent parameter either. It simply tries to use it, as seen in:
In the above snippet, as explained before, it is possible that transparent is a string. When that is the case, the above if check will not work as intended (because in Python all strings are truthy, i.e. if "false" evaluates to True).
- 主要言語
- Python
- スター
- 626
- フォーク
- 334
- 平均マージ
- 20時間 31分
- マージ済み PR（30日）
- 3
環境構築
- Dockerfile または Docker Compose ファイルあり
- プルリクエストのテンプレートあり
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
geopython/pygeoapi のほかの issue
-
the query parameter limit is not working for jobs再び着手できるかも このイシューのプルリクエストはマージされずにクローズされました。 オープンbug
難易度 2/5 1〜3時間 初心者へのやさしさ 70/100
メンテナーはふだん 1 日以内に返信
-
enhancement
難易度 4/5 3〜5日 初心者へのやさしさ 38/100
geopython/pygeoapi#2448 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
map providers can sometimes be passed incorrect bbox-crs value対応中かも @ozenyunus35 が 1 日前に担当しました。 オープンbug
難易度 2/5 1〜3時間 初心者へのやさしさ 25/100
メンテナーはふだん 1 日以内に返信
-
bug
難易度 3/5 半日 初心者へのやさしさ 68/100
メンテナーはふだん 1 日以内に返信
-
enhancement
難易度 5/5 1週間以上 初心者へのやさしさ 25/100
メンテナーはふだん 1 日以内に返信
geopython/pygeoapi の issue をすべて見る
似ている issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 77/100
lidofinance/diffyscan#239 ·
メンテナーはふだん 1 日以内に返信
-
docs: the dev-setup table and AGENTS.md name the reviewed_by guard that CI's Changelog job runsオープンarea: tools documentation good first issue ready for a contributor
難易度 2/5 1〜3時間 初心者へのやさしさ 82/100
eyalgolan/statecraft-israel#873 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
GET rooms API returns 500 due to nonexistent Room.is_active attribute対応中かも @Mars-60 が今日担当しました。 オープンbug
難易度 2/5 1〜3時間 初心者へのやさしさ 84/100
メンテナーはふだん 1 日以内に返信
-
enhancement
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
メンテナーはふだん 5 日以内に返信
-
Harmony OPeNDAP SubSetter (HOSS) Geographic production rssmif16d 7 XYZ_PROV
難易度 2/5 1〜3時間 初心者へのやさしさ 62/100
nasa/harmony-autotester#341 ·