slackapi / slackapi/bolt-python
Please add `app.get_new_client()` to help with token rotation
Nessuno ha ancora preso questa issue.
- Lingua principale
- Python
- Stelle
- 1.3k
- Fork
- 288
- Merge medio
- 1g 8h
- PR unite (30g)
- 10
Descrizione
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.
Guida per i contributori
Apri la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Direzione di ricerca
Esamina prima la discussione #1324, quindi ispeziona gli entry point slack_bolt.App e slack_bolt.async_app.AsyncApp e il modo in cui utilizzano InstallationStore durante la rotazione dei token. Il lavoro sarà completato quando i chiamanti potranno ottenere un client appena creato per un team tramite i metodi richiesti senza dover gestire direttamente lo store, sia per le app sincrone sia per quelle asincrone.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Valutazione
- Stack tecnologico
- python
- Ambito
- api, authentication
- Tipo di issue
- Funzionalità
- Difficoltà
- 4/5
- Tempo stimato
- 3-5 giorni
- Stato di attività
- Ferma
- Chiarezza
- Abbastanza chiara
- Idoneità per principianti
- 45/100