Which JSONParserConfiguration to use in JSONObject/JSONArray constructors

クローズ

#945 2025年1月13日に作成コメント 0 件リアクション 0 件担当者 0 名 GitHub で見る

まだ誰も着手していません。

評価

難易度: 4/5
見積もり時間: 3〜5日
初心者へのやさしさ: 38/100

issue の種類: リファクタリング
明瞭さ: おおむね明確
活発さ: 停滞
技術スタック: java
領域: backend-api-design

調査の方向性

Start with the JSONArray and JSONObject constructors that accept JSONTokener and JSONParserConfiguration, then trace how each configuration is applied to nested values. Compare the constructor behavior and update the corresponding JavaDocs so the single configuration source and expected behavior are explicit.

索引モデルが issue の本文から書いたものです。

説明

Fix in progress

From #927:

What is the purpose of JSONParserConfiguration parameter in JSONArray/JSONObject constructors? Is it not enough to use JSONTokener's JSONParserConfiguration? For example, when calling "JSONArray(JSONTokener x, JSONParserConfiguration jsonParserConfiguration)" constructor the array syntax is checked according to jsonParserConfiguration parameter, but nested objects/arrays syntax is checked according to JSONTokener's configuration. It seems a bit inconsistent.

Both the JSONArray/JSONObject instance and the JSONTokener parameter already have a JSONParserConfiguration. Only one of these should be used. I am leaning towards replacing the JSONTokener config instance with the config param. This should also be called out in the JavaDocs for the corresponding constructors.

主要言語: Java
スター: 4.7k
フォーク: 2.6k
平均マージ: 11日 18分
マージ済み PR（30日）: 1

コントリビューションガイド

コントリビューションガイドを開く

はじめの一歩

issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
リポジトリをフォークし、ブランチを切って変更します。
issue 番号を参照したプルリクエストを送ります。

stleary/JSON-java のほかの issue

JSONPointer pre-existing bug in toURIFragment() オープン

Fix before the next release

難易度 2/5 1〜3時間初心者へのやさしさ 78/100

stleary/JSON-java#1082 · 2026年9月20日に作成
New JSONPointer tests needed オープン

難易度 2/5 1〜3時間初心者へのやさしさ 76/100

stleary/JSON-java#1081 · 2026年9月20日に作成
JSONML checks childNode instead of the reserved childNodes attribute オープン

難易度 2/5 1〜3時間初心者へのやさしさ 88/100

stleary/JSON-java#1080 · 2026年9月19日に作成
JSONML decodes text entities twice with keepStrings enabled オープン

難易度 2/5 1〜3時間初心者へのやさしさ 82/100

stleary/JSON-java#1079 · 2026年9月19日に作成
JSONObject.toString does not use an Enum's `toString()` but its `name()` オープン

Fix before the next release

難易度 2/5 1〜3時間初心者へのやさしさ 78/100

stleary/JSON-java#838 · コメント 16 件 · リアクション 3 件 · 2023年12月11日に作成

stleary/JSON-java の issue をすべて見る

似ている issue

Gate OkHttp body capture on active Session Replay オープン

Bug Java Platform: Java

難易度 2/5 1〜3時間初心者へのやさしさ 78/100

getsentry/sentry-java#6138 · コメント 1 件 · 2026年9月21日に作成
[Datastream to BigQuery] NullPointerException in DataStreamIO when eventType is null オープン

bug needs triage p2

難易度 2/5 1〜3時間初心者へのやさしさ 78/100

GoogleCloudPlatform/DataflowTemplates#4273 · コメント 1 件 · 2026年9月21日に作成
[Studio][Bug] Bulk-deleting a full page of alert rules steps the page back while more rules remain オープン

難易度 2/5 1〜3時間初心者へのやさしさ 78/100

apache/rocketmq-dashboard#4654 · コメント 1 件 · 2026年9月21日に作成
[Bug] [Connector-V2] [Http] A streaming Http source never completes a checkpoint: poll_interval_millis is waited out holding the checkpoint lock オープン

難易度 2/5 1〜3時間初心者へのやさしさ 78/100

apache/seatunnel#12411 · 2026年9月20日に作成
wafv2: TagResource/UntagResource don't enforce the 50-tag limit or key/value length limits オープン

難易度 2/5 1〜3時間初心者へのやさしさ 76/100

floci-io/floci#4035 · 2026年9月20日に作成

Java の issue をもっと見る