[Bug] api_*_version override has no effect on PAL-based service URLs
@cobanfurkanx ci sta già lavorando.
Dal 6/10/2026.
Valutazione
- Difficoltà
- 2/5
- Tempo stimato
- 1-3 ore
- Idoneità per principianti
- 75/100
Direzione di ricerca
Inizia da AdyenClient.set_url_version e analizza come la ricerca della versione viene applicata agli URL degli endpoint di servizio. Controlla gli endpoint basati su PAL elencati e il comportamento esistente delle versioni degli URL, quindi verifica che ogni override di api*_version modifichi il percorso corrispondente, mentre una versione non impostata lasci invariati gli URL non correlati.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
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.
- Lingua principale
- Python
- Stelle
- 68
- Fork
- 51
- Merge medio
- 2h 3m
- PR unite (30g)
- 2
Preparare l'ambiente
- Nessun Dockerfile né file Docker Compose
- Ha un modello di pull request
- Leggi la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di Adyen/adyen-python-api-library
-
Feature
Difficoltà 4/5 3-5 giorni Idoneità per principianti 45/100
Adyen/adyen-python-api-library#487 · 1 commento · 1 reazione ·
-
[Test] Add baseUrl environment tests to all service test filesForse già presa Una pull request collegata a questa issue è aperta o già unita. Apertado not stale Test
Difficoltà 4/5 3-5 giorni Idoneità per principianti 55/100
-
chore: consolidate package metadata into pyproject.toml (PEP 621)Forse già presa @erdemtuna l’ha presa 192 giorni fa. Apertado not stale
Difficoltà 3/5 1-2 giorni Idoneità per principianti 72/100
-
[Bug]: Unknown properties return NoneForse già presa Una pull request collegata a questa issue è aperta o già unita. Apertado not stale
Difficoltà 2/5 1-3 ore Idoneità per principianti 48/100
Adyen/adyen-python-api-library#397 · 2 commenti ·
-
do not stale Feature
Difficoltà 5/5 Più di una settimana Idoneità per principianti 35/100
Adyen/adyen-python-api-library#395 · 1 reazione ·
Tutte le issue di Adyen/adyen-python-api-library
Issue simili
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 65/100
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
UKGovernmentBEIS/control-arena#917 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 88/100
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 82/100
debpalash/VoiceStudio#2937 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
aws-samples/sample-per-model-bedrock#261 ·
I maintainer di solito rispondono entro 1 giorno