--params option not sufficient for global configuration
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 35/100
Línea de trabajo
Comience en TspClient.post_configuration(), donde la solicitud actualmente construye un body que contiene solo el mapa de parámetros. Compare esto con la forma en que las configuraciones de salida publicadas mediante un archivo JSON proporcionan un nombre definido por el usuario y, a continuación, verifique que el body del POST de configuración incluya ese nombre requerido para el backend.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
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:
- Lenguaje dominante
- Python
- Estrellas
- 6
- Forks
- 16
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Preparar el entorno
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de eclipse-cdt-cloud/tsp-python-client
-
"Set" classes are not iterableAbierto
Dificultad 3/5 1-2 días Aptitud para principiantes 52/100
-
enhancement
Dificultad 4/5 3-5 días Aptitud para principiantes 38/100
eclipse-cdt-cloud/tsp-python-client#45 · 1 comentario ·
-
bug
Dificultad 3/5 1-2 días Aptitud para principiantes 38/100
-
Dificultad 4/5 3-5 días Aptitud para principiantes 38/100
-
Dificultad 4/5 3-5 días Aptitud para principiantes 38/100
Todos los issues de eclipse-cdt-cloud/tsp-python-client
Issues similares
-
bug status/needs-triage
Dificultad 2/5 1-3 horas Aptitud para principiantes 86/100
prowler-cloud/prowler#12887 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
area: desktop platform: macos priority: p3 status: ready type: enhancement
Dificultad 1/5 Menos de una hora Aptitud para principiantes 92/100
use-agent-os/agent-os#3484 ·
Los mantenedores suelen responder en 2 días
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 86/100
open-telemetry/opentelemetry-python-contrib#5113 · 2 comentarios · 2 reacciones ·
Los mantenedores suelen responder en 1 día
-
external
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
langchain-ai/docs#6255 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
Los mantenedores suelen responder en 1 día