posit-dev / posit-dev/rsconnect-python

`RSConnectExecutor.server_settings` throws error if `self.client` is a `PositClient` object

Abierto
#547 0 comentarios 0 reacciones 0 asignados Ver en GitHub

Nadie ha tomado este issue todavía.

Lenguaje dominante
Python
Estrellas
37
Forks
28
Merge medio
1 d 3 h
PR fusionados (30 d)
7

Descripción

With a RSConnectExecutor object, if .server_settings is accessed and self.client is a PositClient object (as opposed to a RSConnectClient object), it will throw an error.

I'm running into this problem while working on #545, when I try to deploy a Shiny app to shinyapps.io.

The problem is that the server_settings property accesses self.client.server_settings. self.client can be a RSConnectClient or PositClient object. However, the PositClient class does not have .server_settings.

https://github.com/rstudio/rsconnect-python/blob/34c77ea280c353ebdc8f2987f31765736aa6f96f/rsconnect/api.py#L916-L919

Maybe the PositClient class could have a .server_settings field, set to None or an empty dictionary?

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.

Línea de trabajo

Comienza en rsconnect/api.py, en la propiedad server_settings, alrededor de las líneas 916–919; después, inspecciona cómo RSConnectExecutor usa self.client y compara las clases RSConnectClient y PositClient. La corrección estará hecha cuando acceder a server_settings con cualquiera de los dos tipos de cliente ya no provoque un error de atributo y el comportamiento existente de RSConnectClient se mantenga intacto.

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

Evaluación

Stack tecnológico
python
Área
api, cli
Tipo de issue
Error
Dificultad
2/5
Tiempo estimado
1-3 horas
Estado de actividad
Estancado
Claridad
Bien especificado
Aptitud para principiantes
50/100

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.