Make the version check configurable

Abierto
#566 0 comentarios 0 reacciones 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
65/100
Tipo de issue
Nueva funcionalidad
Claridad
Bastante claro
Estado de actividad
Tranquilo
Stack tecnológico
python
Área
backend

Línea de trabajo

Empieza localizando _warn_user_if_sdk_version_outdated() y las rutas create_client() y default_client() que la invocan. Comprueba cómo se utiliza la URL de PyPI codificada actualmente y, a continuación, verifica que las variables de entorno solicitadas puedan redirigir la comprobación o deshabilitarla sin afectar a la creación normal del cliente.

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

Descripción

Is your feature request related to a problem? Please describe.
_warn_user_if_sdk_version_outdated() hits https://pypi.python.org/pypi/vortexasdk/json on every create_client(), and the URL is hardcoded. On corporate machines public PyPI may be blocked and proxied through an internal mirror, so this check fails (or trips egress alerts) on every run.

Describe the solution you'd like
A way to point the version check at an internal mirror, or turn it off. Artifactory/Nexus proxies serve the same /pypi/<pkg>/json, so redirecting it returns identical data and the feature keeps working — just without public egress (verified against our internal Artifactory mirror: same release list and latest version as pypi.org).

Env vars are probably the most practical, since the check runs on the implicit default_client() path where there's nowhere to pass arguments:

  • VORTEXASDK_PYPI_URL — base URL for the check
  • VORTEXASDK_DISABLE_VERSION_CHECK — skip it entirely

Describe alternatives you've considered

  • Params on create_client() (e.g. pypi_url=, check_version=False) — cleaner API, but only helps callers who invoke it directly; most usage likely goes through default_client().

Additional context
#126 (which introduced the check) originally called for a toggle to turn the check on/off, but #198 shipped without one.

Lenguaje dominante
Python
Estrellas
25
Forks
12
Merge medio
20 h 47 min
PR fusionados (30 d)
2

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 VorTECHsa/python-sdk

Todos los issues de VorTECHsa/python-sdk

Issues similares

Más issues de Python

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.