Conversation editor never revokes blob URLs of removed attachments
Maintainer antworten meist innerhalb von 2 Tagen
Bewertung
- Schwierigkeit
- 2/5
- Geschätzter Aufwand
- 1-3 Stunden
- Anfängerfreundlichkeit
- 25/100
- Issue-Typ
- Bug
- Klarheit
- Klar beschrieben
- Aktivitätsstatus
- Aktiv
- Bereich
- frontend
Rechercherichtung
Start with ChatInputArea.changeAttachments, which already revokes dropped blob URLs, and compare it with useConversationDraft.changeAttachments, which the ConversationEditor piece editor calls without revoking anything. The attachment chip URLs are created in PromptEditor with URL.createObjectURL. Done means removing an attachment in the draft editor calls URL.revokeObjectURL right away. Pull request #3186 is already open against this issue, so review or build on that work rather than starting a parallel fix.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Beschreibung
Describe the bug
In the conversation editor, removing an attachment from a media piece never revokes the attachment's blob object URL. The blob: URL and its File payload stay alive until the whole draft is discarded or the editor unmounts.
ChatInputArea.changeAttachments handles the same removal correctly by revoking dropped blob URLs, but the draft editor path (ConversationEditor piece editor calling useConversationDraft.changeAttachments) forwards the new attachment list straight through with no revocation. Attaching and removing a large file repeatedly during one editing session pins every dropped File in memory.
Steps/Code to Reproduce
- Open a conversation in the editor and pick a media piece.
- Attach a file (the attachment chip gets a
blob:URL fromURL.createObjectURLin PromptEditor). - Remove the attachment with the chip's dismiss button.
URL.revokeObjectURLis never called for that URL until the draft is discarded or the editor unmounts.
Repeat 2-3 in one session and the dropped Files accumulate.
Expected Results
Removing an attachment in the editor revokes its blob object URL immediately, the same way the chat composer does.
Actual Results
The URL stays valid and the File payload stays pinned until the draft is discarded or unmounted.
Screenshots
N/A
Versions
OS: macOS
Browser: Chrome
Python version: 3.14
PyRIT version: installed from main (66c41fa)
- Vorherrschende Sprache
- Python
- Sterne
- 4.6k
- Forks
- 944
- Ø Merge
- 2 T. 16 Std.
- Gemergte PRs (30 T.)
- 350
Entwicklungsumgebung
Startet den Dev-Container des Projekts im Browser, mit Ihrem eigenen GitHub-Konto.
- Kein Dockerfile und keine Docker-Compose-Datei
- Hat eine Pull-Request-Vorlage
- Kein Beitragsleitfaden
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 microsoft/PyRIT
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 75/100
microsoft/PyRIT#3204 · 1 Kommentar ·
Maintainer antworten meist innerhalb von 2 Tagen
-
BUG: label queries only return the final conversation of an attackEvtl. vergeben @u7k4rs6 hat das vor 1 Tag übernommen. Offen
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 85/100
microsoft/PyRIT#3095 · 4 Kommentare · 1 zugewiesene Person ·
Maintainer antworten meist innerhalb von 2 Tagen
-
BUG Configuration keeps runtime-status errors after polling recoversEvtl. vergeben @joeyvictorino hat das vor 5 Tagen übernommen. OffenBug: triage GUI help wanted
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 86/100
microsoft/PyRIT#2868 · 4 Kommentare · 1 zugewiesene Person ·
Maintainer antworten meist innerhalb von 2 Tagen
-
BUG: HTTPTarget sends conflicting Content-Length and Transfer-Encoding headersEvtl. vergeben @Kaap10 hat das vor 9 Tagen übernommen. OffenBug: triage
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 35/100
microsoft/PyRIT#3202 · 1 Kommentar ·
Maintainer antworten meist innerhalb von 2 Tagen
-
Schwierigkeit 3/5 Ein halber Tag Anfängerfreundlichkeit 25/100
Maintainer antworten meist innerhalb von 2 Tagen
Alle Issues in microsoft/PyRIT
Ähnliche Issues
-
Path subtraction removes repeated suffix characters instead of one suffixEvtl. vergeben @Gwerdonatus hat das heute übernommen. Offen
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 72/100
joke2k/django-environ#671 ·
-
good first issue help wanted windows
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 80/100
-
test_pgbouncer deploys the server under test from Charmhub instead of the local build (track/1.31)Offenbug
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 68/100
canonical/temporal-k8s-operator#195 · 1 Kommentar ·
Maintainer antworten meist innerhalb von 4 Tagen
-
`burr` fails to start when the Python path has a space in itEvtl. vergeben @enekoreto hat das heute übernommen. Offenstatus/needs-triage
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 72/100
Maintainer antworten meist innerhalb von 5 Tagen
-
device support request
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 68/100
zigpy/zha-device-handlers#5461 ·
Maintainer antworten meist innerhalb von 4 Tagen