FYI: Small breaking changes are coming to some (non-schema author related) APIs
まだ誰も着手していません。
評価
調査の方向性
#74 で参照されているテストと、ここで挙げられている取得のエントリーポイントから始めてください: Resolver.lookup、Registry の取得コールバック、Retriever。発表された URL の正規化、URL コールバック型、および取得後のベース URL の動作を完了条件として使用してください。
索引モデルが issue の本文から書いたものです。
説明
This is an FYI to anyone who sees it that an upcoming release (possibly v0.35) will likely contain breaking changes to the retrieval API.
referencing of course is in pre-1.0 / stable status (for slightly more details there see #38). I still of course don't wish to unnecessarily cause churn for people who've begun to adopt it, but it's clear there's at least one, possibly two, things in need of fixing.
Specifically, though I knew it would happen at some point, referencing needs to use a URL implementation in order to properly perform URL normalization -- in other words, as a simple example, knowing that HTTP://example.com is the same as http://example.com. Of course the full gamut is more complicated.
This means it needs to internally use a library (which will be url.py) which properly implements RFC3986 (though technically that one implements WHATWG URLs). Some functions then will need to take URLs rather than strs. Yes it is possible to sprinkle isinstance checks everywhere and convert always between str and URL but this will leave a small performance penalty everywhere, and also just generally seems unwise to leave in place before referencing continues to mature.
In particular:
Expect retrieve callbacks to likely receive URL objects rather than strs in an upcoming release.
I still expect this to affect only a small number of users ultimately, because the relevant APIs do not relate to Resource + Registry assembly, they relate to "internal" (but public) APIs like Resolver.lookup and Registry retrieval but I'm putting this here anyways.
Further details to come shortly, I'm still working on a set of changes to support this. If you wish to follow along, #74 contains the tests I have added (upstream to the test suite) which "tease out" the underlying issue the above references.
(And as a second example, which I again am unsure how easy I can fix but which I suspect should happen before #38 -- see #65. Ideally I'll find a way to do both of these changes at the same time.)
As a third example (and a slight reminder to myself), we also may not have precisely the right behavior vis a vis re-setting the base URL after retrieval, which should likely take precedence over the initial base URL for a Retriever (i.e. the object you get back should have a new base URI using the retrieval URL)
- 主要言語
- Python
- スター
- 53
- フォーク
- 27
- 平均マージ
- 1日 8時間
- マージ済み PR(30日)
- 8
コントリビューションガイド
このリポジトリのコントリビューションガイドは索引されていません
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
python-jsonschema/referencing のほかの issue
-
難易度 4/5 3〜5日 初心者へのやさしさ 45/100
python-jsonschema/referencing#366 ·
-
難易度 4/5 3〜5日 初心者へのやさしさ 35/100
python-jsonschema/referencing#257 ·
-
難易度 5/5 1週間以上 初心者へのやさしさ 20/100
python-jsonschema/referencing#226 · コメント 1 件 ·
-
難易度 4/5 3〜5日 初心者へのやさしさ 52/100
python-jsonschema/referencing#178 · コメント 6 件 ·
-
難易度 4/5 3〜5日 初心者へのやさしさ 25/100
python-jsonschema/referencing#160 · コメント 3 件 · リアクション 2 件 ·
python-jsonschema/referencing の issue をすべて見る
似ている issue
-
area: harness bug status: needs-triage
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
Human-Agent-Society/reef#625 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 70/100
-
難易度 1/5 1時間未満 初心者へのやさしさ 80/100
learningequality/kolibri#15351 · コメント 2 件 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
-
Name consistency オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
eellak/triplestore#65 · コメント 1 件 ·