--params option not sufficient for global configuration
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 2/5
- Temps estimé
- 1-3 heures
- Accessibilité débutants
- 35/100
Piste de recherche
Commencez par TspClient.post_configuration(), où la requête construit actuellement un body contenant uniquement la map des paramètres. Comparez cela avec la manière dont les configurations de sortie publiées à l’aide d’un fichier JSON fournissent un nom défini par l’utilisateur, puis vérifiez que le body du POST de configuration inclut ce nom requis pour le backend.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
Bug Description:
--params option not sufficient for global configuration because it's only for the parameters passed as parameters map in the ConfigurationQueryParameters. There is no way to add the name to ConfigurationQueryParameters which is passed in the post_configuration() method of the TspClient. The "post_configuration" method() creates the query parameters without name:
def post_configuration(self, type_id, params):
api_url = '{0}config/types/{1}/configs'.format(self.base_url, type_id)
parameters = {'parameters': params}
response = requests.post(api_url, json=parameters, headers=headers)
...
The handwritten python client doesn't verify if name is passed to the body of HTTP post call. However, generated clients will verify it because it's a required field.
Moreover, backend implemenation of a configuration source will rely on it because it's required. Note, that the XMLConfigurationSource of the Trace Compass server is the sole implementation in Trace Compass so far and it doesn't require a name and ignores it. The filename of the xml file configured in the path variable is assigned as name.
Change python code to pass a user defined name similarly when posting output configurations using a json file.
Steps to Reproduce:
No easy way to reproduce. Just debug the client code (TspClient.post_configuration()) and see that the name is not passed.
Additional Information
- Operating System:
- Langage dominant
- Python
- Étoiles
- 6
- Forks
- 16
- Métriques de merge des PR
- Aucune PR mergée en 30 j
Préparer son environnement
- Aucun Dockerfile ni fichier Docker Compose
- Propose un modèle de pull request
- Lire le 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 eclipse-cdt-cloud/tsp-python-client
-
"Set" classes are not iterableOuverte
Difficulté 3/5 1-2 jours Accessibilité débutants 52/100
-
Support endpoint to update experimentsPeut-être pris @bhufmann l’a pris il y a 1779 jours. Ouverteenhancement
Difficulté 4/5 3-5 jours Accessibilité débutants 38/100
eclipse-cdt-cloud/tsp-python-client#45 · 1 commentaire ·
-
bug
Difficulté 3/5 1-2 jours Accessibilité débutants 38/100
-
Difficulté 4/5 3-5 jours Accessibilité débutants 38/100
-
Difficulté 4/5 3-5 jours Accessibilité débutants 38/100
Toutes les issues de eclipse-cdt-cloud/tsp-python-client
Issues similaires
-
good first issue infrastructure
Difficulté 2/5 1-3 heures Accessibilité débutants 76/100
kuznia-rdzeni/coreblocks#1135 ·
-
good first issue server
Difficulté 2/5 1-3 heures Accessibilité débutants 85/100
Les mainteneurs répondent en général sous 1 jour
-
documentation priority: high
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
a-lex-s/mealie-telegram-bot#54 ·
Les mainteneurs répondent en général sous 1 jour
-
type/test
Difficulté 2/5 1-3 heures Accessibilité débutants 76/100
SaltedTan/virgil-language-server#218 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 83/100
btclib-org/btclib-wallet#541 ·
Les mainteneurs répondent en général sous 1 jour