Inconsistent quoting behavior when serializing strings starting with zero
まだ誰も着手していません。
評価
調査の方向性
まず Psych における 2 つの UUID シリアライズケースを再現し、参照されているコミット b737f0811a9687cf86f44f0a35f61cbde9eac673 を YAML 1.2 の浮動小数点数マッチング規則と併せて確認します。異なるクォートが意図されたスキーマに従っているかを確認し、そのうえで合意した動作に対する回帰テストを定義します。
索引モデルが issue の本文から書いたものです。
説明
Hello there,
we detected a weird behavior when serializing UUIDs. Some of them, such as 0054971b-b746-4776-ae7a-d942cd511393, are quoted during dump, whereas others, such as 0645cada-c729-48cb-a4e3-9d8cbc7cb510 aren't.
The observed behavior seems related to this commit created to fix this issue with floats handling.
The YAML 1.2 schema uses this regexp /[-+]?(\.[0-9]+|[0-9]+(\.[0-9]*)?)([eE][-+]?[0-9]+)?/ to match floats, but on Psych side this regexp is used: /\A0[0-7]*[89]/, which leads in my understanding to an inconsistent behavior.
I'd be glad to craft a PR but I' not sure whether Psych behavior is expected or not.
Thank you :)
- 主要言語
- Ruby
- スター
- 596
- フォーク
- 224
- 平均マージ
- 8時間 38分
- マージ済み PR（30日）
- 2
環境構築
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートなし
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
ruby/psych のほかの issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
-
Documentation
難易度 1/5 1時間未満 初心者へのやさしさ 72/100
-
Psych.dump raises Encoding::CompatibilityError for UTF-16/UTF-32 strings対応中かも @Halvanhelv が 57 日前に担当しました。 オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 55/100
-
難易度 4/5 3〜5日 初心者へのやさしさ 68/100
-
難易度 4/5 3〜5日 初心者へのやさしさ 52/100
似ている issue
-
internal product:ai-gateway size: S
難易度 2/5 1〜3時間 初心者へのやさしさ 67/100
Kong/developer.konghq.com#7931 ·
メンテナーはふだん 1 日以内に返信
-
難易度 1/5 1〜3時間 初心者へのやさしさ 78/100
partiql/partiql-lang#105 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 70/100
rails/rails-dom-testing#136 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
nesquena/steward-os#79 ·
メンテナーはふだん 1 日以内に返信
-
area/tech debt
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
メンテナーはふだん 1 日以内に返信