[Bug] HttpxRequestAdapter: The silent deprecation of base_url is actually a breaking change

Abierto
#501 0 comentarios 1 reacción 0 asignados Ver en GitHub

Nadie ha tomado este issue todavía.

Evaluación

Dificultad
3/5
Tiempo estimado
1-2 días
Aptitud para principiantes
35/100
Tipo de issue
Error
Claridad
Bastante claro
Estado de actividad
Estancado
Stack tecnológico
python
Área
api

Línea de trabajo

Comienza con HttpxRequestAdapter e inspecciona cómo su constructor gestiona base_url después del cambio #481; compara la solución alternativa reportada con el comportamiento actual del parámetro. Aclara si el resultado previsto es marcar como obsoleto el parámetro del constructor o restaurar su efecto y, después, verifica el comportamiento elegido con cobertura de regresión.

Escrito por el modelo de indexación a partir del texto del issue.

Descripción

#481 actually introduced a breaking change.

The parameter base_url is still there but no matter what value you provide, it's ignored. We did not pass a value for http_client, and only specified the base_url like this:

request_adapter = HttpxRequestAdapter(
    authentication_provider, base_url="https:/example.com"
)

We run this in a Fabric notebook where the version of the package was not pinned. So the moment it downloaded the new version, it failed with a misleading error.

To fix it, we had to do the following work around:

request_adapter = HttpxRequestAdapter(authentication_provider)
request_adapter.base_url = "https:/example.com"

I think it's best to deprecate this base_url parameter in the constructor if it's not supposed to be used.

Lenguaje dominante
Python
Estrellas
31
Forks
39
Merge medio
13 h 21 min
PR fusionados (30 d)
46

Guía de contribución

Abrir la guía de contribución

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Más de microsoft/kiota-python

Todos los issues de microsoft/kiota-python

Issues similares

Más issues de Python

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.