spec-first/connexion

Throw an error on no operation_id

Open

#1.166 geöffnet am 21. Feb. 2020

Auf GitHub ansehen
 (1 Kommentar) (0 Reaktionen) (0 zugewiesene Personen)Python (4.346 Stars) (738 Forks)batch import
good first issue

Beschreibung

https://github.com/zalando/connexion/blob/96bdcb010a4e5180c2bce18295e7b576c9c1ef0f/connexion/resolver.py#L60

Currently operation_id == None flows into function_resolver and triggers unexpected errors. Better to do?:

if operation_id is None:
  raise ResolverError("operation_id is None", sys.exc_info())

Contributor Guide

Tech Stack
python
Domain
backendapi
Issue Type
bug
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
under 1 hour
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
stale
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
clear
Voraussetzungen
Python basicsunderstanding of error handling
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
90
Research-Richtung
The issue suggests adding a check in connexion/resolver.py around line 60 to raise ResolverError if operation id is None. The change is straightforward. No linked PRs or discussions.