Don't verify content running for rendered content
Maintainer antworten meist innerhalb von 1 Tag
Bewertung
- Schwierigkeit
- 3/5
- Geschätzter Aufwand
- 1-2 Tage
- Anfängerfreundlichkeit
- 45/100
Rechercherichtung
The payload names no files, so start by tracing the deploy path in this Python CLI: where content is rendered during deploy, where the verify GET runs, and where activation triggers a second render. Confirm whether a recent change already blocks activation after a failed render, as the issue suggests. Done means a static-content deploy skips verify, and a rendered deploy renders only once.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Beschreibung
Deploying (by default) renders the content. Doing a GET on the content URL doesn't validate anything I don't think. Then activating renders again. If the rendering errored, the deploy would have errored (right?) so there is no value to trying something in the middle. (I think there was a recent change to prevent activating if render failed? We should double check.) In any case, we don't want to render twice.
Clearly for static content we should also skip the verify step if we aren't already.
- Vorherrschende Sprache
- Python
- Sterne
- 38
- Forks
- 29
- Ø Merge
- 1 T. 9 Std.
- Gemergte PRs (30 T.)
- 18
Entwicklungsumgebung
- Kein Dockerfile und keine Docker-Compose-Datei
- Hat eine Pull-Request-Vorlage
- Beitragsleitfaden lesen
Erste Schritte
- Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
- Forken Sie das Repository und arbeiten Sie in einem Branch.
- Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.
Mehr aus posit-dev/rsconnect-python
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 78/100
posit-dev/rsconnect-python#786 · 1 Reaktion ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 62/100
posit-dev/rsconnect-python#543 · 2 Kommentare ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 1/5 Unter einer Stunde Anfängerfreundlichkeit 72/100
posit-dev/rsconnect-python#513 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 65/100
posit-dev/rsconnect-python#178 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 55/100
posit-dev/rsconnect-python#881 ·
Maintainer antworten meist innerhalb von 1 Tag
Alle Issues in posit-dev/rsconnect-python
Ähnliche Issues
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 67/100
pydot/pydot#601 · 4 Kommentare ·
Maintainer antworten meist innerhalb von 2 Tagen
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 68/100
letsencrypt/cp-cps#361 ·
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 65/100
-
CI: startup_failure because callers grant fewer permissions than ledger-app-workflows@v1 declaresEvtl. vergeben @maykonmichel hat das heute übernommen. Offen
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 76/100
LedgerHQ/app-boilerplate-rust#227 ·
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 76/100
linuxmint/mintupdate#1107 ·