Refactoring envelope of successful responses and its client contract
Les mainteneurs répondent en général sous 1 jour
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 2/5
- Temps estimé
- 1-3 heures
- Accessibilité débutants
- 62/100
Piste de recherche
Commencez par les constructeurs de réponses réussies dans src/utils/utils.py et examinez leurs appelants et leurs tests pour déterminer quels champs de réponse le client attend. Mettez à jour de manière cohérente l’enveloppe des réponses réussies pour les données, la pagination et les agrégations, puis exécutez les tests pertinents. Le travail est terminé lorsque les champs redondants ont été supprimés du corps de la réponse et que les tests reflètent le contrat du client.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
Summary
Revisit the envelope of successful (HTTP 200) responses from the stats lambda. Every success body currently contains two fields that duplicate the HTTP response:
{"success": true, "statusCode": 200, "data": [...], "pagination": {...}}
{"success": true, "statusCode": 200, "aggregations": {...}}
success: trueis already implied by the 2xx status code.statusCodeis repeated in the body, although it is already the API GatewaystatusCode.
This is built in src/utils/utils.py (success response builders for data/pagination and aggregations).
Background
Raised by @lsulak in review of PR #243: "success is quite visible from the status code, and then also status code is there twice. Is there any situation when it differentiates?"
- Langage dominant
- Python
- Étoiles
- 4
- Forks
- 0
- Merge moyen
- 3 j 12 h
- PR mergées (30 j)
- 10
Préparer son environnement
- Fournit un Dockerfile ou un fichier Docker Compose
- Propose un modèle de pull request
- Aucun guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de AbsaOSS/EventGate
-
refactoring type:tech-debt
Difficulté 2/5 1-3 heures Accessibilité débutants 84/100
Les mainteneurs répondent en général sous 1 jour
-
enhancement
Difficulté 2/5 1-3 heures Accessibilité débutants 70/100
Les mainteneurs répondent en général sous 1 jour
-
Python quality gates analysisPeut-être pris @tmikula-dev l’a pris il y a 7 jours. Ouverteenhancement
AbsaOSS/EventGate#253 · 1 personne assignée ·
Les mainteneurs répondent en général sous 1 jour
-
enhancement
Difficulté 4/5 3-5 jours Accessibilité débutants 56/100
AbsaOSS/EventGate#246 · 1 commentaire ·
Les mainteneurs répondent en général sous 1 jour
-
bug
Difficulté 3/5 1-2 jours Accessibilité débutants 64/100
Les mainteneurs répondent en général sous 1 jour
Toutes les issues de AbsaOSS/EventGate
Issues similaires
-
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
Les mainteneurs répondent en général sous 1 jour
-
AI bug
Difficulté 2/5 1-3 heures Accessibilité débutants 68/100
AUTO-MAS-Project/AUTO-MAS#1960 · 1 commentaire ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 82/100
btclib-org/btclib-wallet#872 ·
Les mainteneurs répondent en général sous 1 jour
-
[Bug]: Scheduled returns scan all accepted minion keys causing extreme loadPeut-être pris Une pull request liée à cette issue est ouverte ou déjà fusionnée. Ouvertebug needs-triage
Difficulté 2/5 1-3 heures Accessibilité débutants 76/100
saltstack/salt#70789 · 1 commentaire ·
Les mainteneurs répondent en général sous 3 jours
-
Difficulté 2/5 1-3 heures Accessibilité débutants 87/100
btclib-org/btclib#3093 ·
Les mainteneurs répondent en général sous 1 jour