[Bug] api_*_version override has no effect on PAL-based service URLs
@cobanfurkanx がすでに取り組んでいます。
2026年10月6日 から。
評価
調査の方向性
AdyenClient.set_url_version から始めて、サービスエンドポイントの URL にバージョン検索がどのように適用されるかを確認します。記載されている PAL ベースのエンドポイントと既存の URL バージョン動作を確認し、各 api*_version のオーバーライドによって対応するパスが変更され、バージョンが設定されていない場合は無関係な URL が変更されないことを検証します。
索引モデルが issue の本文から書いたものです。
説明
Context
AdyenClient._set_url_version uses the following regex to replace the API version in endpoint URLs:
endpoint = re.sub(r"\.com/v\d{1,2}", f".com/{new_version}", endpoint)
This only matches URLs where the version segment appears immediately after the domain, e.g.:
checkout-test.adyen.com/v71✓management-test.adyen.com/v3✓
It does not match PAL-based URLs where the version is deeper in the path, e.g.:
pal-test.adyen.com/pal/servlet/Recurring/v68✗pal-test.adyen.com/pal/servlet/Payment/v68✗pal-test.adyen.com/pal/servlet/BinLookup/v54✗
Proposed Change
Update the regex to match a version segment anywhere in the path:
endpoint = re.sub(r"/v\d+(?=/|$)", f"/{new_version}", endpoint)
Also add a None guard to avoid corrupting unrelated service URLs when only one api_*_version is set:
version = version_lookup[service]
if version is None:
return endpoint
Benefit
Setting api_recurring_version, api_payment_version, api_payout_version, api_bin_lookup_version, or api_stored_value_version on AdyenClient currently has no effect. The fix makes version overrides work consistently across all services.
- 主要言語
- Python
- スター
- 67
- フォーク
- 51
- PR マージ指標
- 30日以内にマージされた PR はありません
環境構築
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートあり
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
Adyen/adyen-python-api-library のほかの issue
-
Feature
難易度 4/5 3〜5日 初心者へのやさしさ 45/100
Adyen/adyen-python-api-library#487 · コメント 1 件 · リアクション 1 件 ·
-
[Test] Add baseUrl environment tests to all service test files対応中かも このイシューにリンクされたプルリクエストがオープン中、またはマージ済みです。 オープンdo not stale Test
難易度 4/5 3〜5日 初心者へのやさしさ 55/100
-
chore: consolidate package metadata into pyproject.toml (PEP 621)対応中かも @erdemtuna が 193 日前に担当しました。 オープンdo not stale
難易度 3/5 1〜2日 初心者へのやさしさ 72/100
-
[Bug]: Unknown properties return None対応中かも このイシューにリンクされたプルリクエストがオープン中、またはマージ済みです。 オープンdo not stale
難易度 2/5 1〜3時間 初心者へのやさしさ 48/100
Adyen/adyen-python-api-library#397 · コメント 2 件 ·
-
do not stale Feature
難易度 5/5 1週間以上 初心者へのやさしさ 35/100
Adyen/adyen-python-api-library#395 · リアクション 1 件 ·
Adyen/adyen-python-api-library の issue をすべて見る
似ている issue
-
難易度 1/5 1時間未満 初心者へのやさしさ 65/100
521xueweihan/HelloGitHub#4025 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 85/100
aws-samples/sample-per-model-bedrock#321 ·
メンテナーはふだん 1 日以内に返信
-
難易度 1/5 1〜3時間 初心者へのやさしさ 68/100
FavioVazquez/showtime#101 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
koush/ha_scrypted#81 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 82/100
google/osv-scanner-action#147 ·