[TECH DEBT] Deprecate the callback argument of AsyncAuth.authenticate()
I maintainer di solito rispondono entro 3 giorni
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 3/5
- Tempo stimato
- 1-2 giorni
- Idoneità per principianti
- 68/100
- Tipo di issue
- Refactoring
- Chiarezza
- Specificata chiaramente
- Stato di attività
- Attiva
- Stack tecnologico
- python
- Ambito
- backend-api-design
Direzione di ricerca
Start by locating AsyncAuth.authenticate() and its tests in the Salt codebase; the issue does not name specific files or tests. Check how salt.utils.versions.warn_until is used for deprecations, then add the warning on master when a callback is passed, pointing callers to await or add_done_callback(). Done means the warning is covered by tests and the 3008.x behavior remains unchanged.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
Description of the tech debt to be addressed, include links and screenshots
AsyncAuth.authenticate(callback=None) takes an optional callback that is called with the result when the sign-in succeeds. It is left over from the Tornado callback style.
- Nothing in
salt/passes a callback. Every caller doesawait auth.authenticate(). - It is not documented.
- With #70603 the callback is only called when the sign-in succeeds. A failed sign-in never reaches it, so a caller using a callback is not told about the failure.
authenticate()returns a task. A caller that wants a notification can usetask.add_done_callback(), which is also called when the sign-in fails.
Proposed change
Do not change this in #70603 or its 3008.x version, which only fix cancellation and error handling.
- On
master, warn withsalt.utils.versions.warn_untilwhen a callback is passed, and point toawaitoradd_done_callback(). - Remove the argument once the warning period is over.
Leave 3008.x as it is.
Related: #70603, #70366, #70712.
Versions Report
No user-visible change until the warning is added. Targets the master branch.
- Lingua principale
- Python
- Stelle
- 15.8k
- Fork
- 5.6k
- Merge medio
- 5g 9h
- PR unite (30g)
- 99
Preparare l'ambiente
- Nessun Dockerfile né file Docker Compose
- Ha un modello di pull request
- Leggi 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.
Altre issue di saltstack/salt
-
[Bug]: Scheduled returns scan all accepted minion keys causing extreme loadForse già presa Una pull request collegata a questa issue è aperta o già unita. Apertabug needs-triage
Difficoltà 2/5 1-3 ore Idoneità per principianti 76/100
saltstack/salt#70789 · 1 commento ·
I maintainer di solito rispondono entro 3 giorni
-
[Bug]: pip.installed with upgrade=True upgrades a package but returns an empty changes dictionaryForse già presa @ITJamie l’ha presa 1 giorno fa. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
I maintainer di solito rispondono entro 3 giorni
-
bug needs-triage
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
I maintainer di solito rispondono entro 3 giorni
-
[Bug]: Minion with file_client: local crashes on connect: 'AsyncPillar' object has no attribute 'aclose'Forse già presa @shawnrruss l’ha presa 9 giorni fa. Apertabug needs-triage
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
I maintainer di solito rispondono entro 3 giorni
-
[Bug]: mine.update clear=True raises UnboundLocalError on the master and stores nothingForse già presa @cestercian l’ha presa 11 giorni fa. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 74/100
saltstack/salt#70622 · 1 commento ·
I maintainer di solito rispondono entro 3 giorni
Tutte le issue di saltstack/salt
Issue simili
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 82/100
btclib-org/btclib-wallet#872 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 87/100
btclib-org/btclib#3093 ·
I maintainer di solito rispondono entro 1 giorno
-
good first issue
Difficoltà 2/5 1-3 ore Idoneità per principianti 85/100
patrickballard/wheelgap#269 ·
-
priority: low python
Difficoltà 2/5 1-3 ore Idoneità per principianti 88/100
LearningCircuit/local-deep-research#8635 ·
I maintainer di solito rispondono entro 1 giorno
-
README says the land/sea mask must match the XS grid; the demo mask does not, and Step 4 worksApertadocumentation
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
buckai-observatory/Automated-Cloud-Removal-and-Seamless-Satellite-Image-Mosaicking-Pipeline#52 ·
I maintainer di solito rispondono entro 1 giorno