slackapi / slackapi/bolt-python
Please add `app.get_new_client()` to help with token rotation
Nadie ha tomado este issue todavía.
- Lenguaje dominante
- Python
- Estrellas
- 1.3k
- Forks
- 288
- Merge medio
- 1 d 8 h
- PR fusionados (30 d)
- 10
Descripción
From a discussion in #1324, the only safe way to call the client from outside a bolt handler when token rotation is on is to create a new client each time (as the handlers do)
Since Bolt is currently handling all of this for us, it'd be very useful to have a method along the lines of
client = await bolt.get_new_client(team_id)
that means callers do not need to deal with the InstallationStore and such themselves.
Category (place an x in each of the [ ])
- slack_bolt.App and/or its core components
- slack_bolt.async_app.AsyncApp and/or its core components
- Adapters in slack_bolt.adapter
- Others
Requirements
Please read the Contributing guidelines and Code of Conduct before creating this issue or pull request. By submitting, you are agreeing to those rules.
Guía de contribución
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.
Línea de trabajo
Revisa primero la discusión #1324 y, después, inspecciona los puntos de entrada slack_bolt.App y slack_bolt.async_app.AsyncApp y cómo utilizan InstallationStore durante la rotación de tokens. La tarea estará terminada cuando los llamadores puedan obtener un cliente recién creado para un equipo mediante los métodos solicitados sin tener que gestionar el store por su cuenta, tanto en aplicaciones síncronas como asíncronas.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- python
- Área
- api, authentication
- Tipo de issue
- Nueva funcionalidad
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Estado de actividad
- Estancado
- Claridad
- Bastante claro
- Aptitud para principiantes
- 45/100