openapi-generators / openapi-generators/openapi-python-client
Minor schema validation error: doesn't complain about invalid default for null and file types
まだ誰も着手していません。
- 主要言語
- Python
- スター
- 2k
- フォーク
- 293
- 平均マージ
- 34分
- マージ済み PR(30日)
- 1
説明
Describe the bug
NoneProperty.build and FileProperty.build do have logic to detect if a default other than None was provided for a null property, or if any default was provided for a binary string—and we do have unit tests for those. However, in reality NoneProperty.build and FileProperty.build do not actually get called, because property_from_data just calls the initializers directly instead. Therefore if you do something like this, the default is simply ignored.
OpenAPI Spec File
components:
schemas:
MyModel:
properties:
myNullProperty:
type: "null"
default: "definitely not null"
myBinaryString:
type: string
format: binary
default: "???"
Desktop (please complete the following information):
- OS: any
- Python Version: any
- openapi-python-client version: 0.21.6
コントリビューションガイド
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
調査の方向性
まず property_from_data を追跡し、その直接的な initializer 呼び出しを、すでに検証ロジックを含んでいる NoneProperty.build および FileProperty.build と比較します。これらの builder に対する既存の unit test を実行し、提供されている OpenAPI スキーマの無効なデフォルト値が無視されるのではなく拒否されることを示すカバレッジを追加します。
索引モデルが issue の本文から書いたものです。
評価
- 技術スタック
- python
- 領域
- api, tooling
- issue の種類
- バグ
- 難易度
- 2/5
- 見積もり時間
- 1〜3時間
- 活発さ
- 停滞
- 明瞭さ
- 明確に書かれている
- 初心者へのやさしさ
- 58/100