Issue links in overview HTML are emitted with an extra quote in href
I maintainer di solito rispondono entro 1 giorno
Valutazione
Questa issue non è ancora stata valutata.
Descrizione
- Extension version: 0.166.1 (current
mainpackage.json) - VSCode Version: not applicable to the source-level reproduction; the extension declares
^1.141.0 - OS: Windows 11 (source-level reproduction; the generated HTML is platform-independent)
- Repository Clone Configuration (single repository/fork of an upstream repository): single repository
- GitHub Product (GitHub.com/GitHub Enterprise version x.x.x): GitHub.com
Steps to Reproduce:
- Open a pull request or issue whose description contains a link to another issue in the same repository, for example
https://github.com/microsoft/vscode-pull-request-github/issues/123. - Let the extension convert the description HTML for the issue/PR overview.
- Inspect the generated anchor for the issue link.
The issue-link branch in src/github/utils.ts currently returns:
return `href="${(await toOpenIssueWebviewUri(...)).toString()}""`;
That produces an anchor like:
<a href="vscode://github.vscode-pull-request-github/open-issue-webview?..."">#123</a>
The corresponding pull-request branch returns a normally terminated href, so this only affects links converted from /issues/<number> URLs.
The extra quote is parsed as a second, nameless attribute (rather than being part of the href value). The link may still appear to work in some Chromium versions, but the HTML is malformed and can be handled inconsistently by the webview DOM/sanitizer. It also makes the generated markup different from the PR-link path for no functional reason.
The smallest fix appears to be removing the extra " at the end of the issue branch, and adding a regression test around the HTML returned for a same-repository issue link. I have not changed the repository code yet; this report is based on the current main source and a local HTML parsing reproduction.
- Lingua principale
- TypeScript
- Stelle
- 2.6k
- Fork
- 800
- Merge medio
- 22h 52m
- PR unite (30g)
- 39
Preparare l'ambiente
Questo progetto non fornisce container di sviluppo, Dockerfile né guida per i contributori, quindi l'ambiente è a tuo carico: parti dal suo README e consulta la nostra guida al primo contributo per i passaggi generali.
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 microsoft/vscode-pull-request-github
-
PR overview remains stuck checking mergeability after an UNKNOWN responseForse già presa @alexr00 l’ha presa 8 giorni fa. Apertabug
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
microsoft/vscode-pull-request-github#8953 · 1 assegnatario ·
I maintainer di solito rispondono entro 1 giorno
-
Issue 3 — DocumentationAperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
microsoft/vscode-pull-request-github#8792 ·
I maintainer di solito rispondono entro 1 giorno
-
False C# syntax errors and red squiggles when viewing pull request diffsForse già presa @alexr00 l’ha presa oggi. Apertainfo-needed
microsoft/vscode-pull-request-github#9034 · 1 commento · 1 assegnatario ·
I maintainer di solito rispondono entro 1 giorno
-
Enable "Open original file" for deleted filesForse già presa @alexr00 l’ha presa 1 giorno fa. Aperta
microsoft/vscode-pull-request-github#9030 · 1 reazione · 2 assegnatari ·
I maintainer di solito rispondono entro 1 giorno
-
Copilot PR-state polling throws RangeError: Invalid array length while accumulating paginated resultsForse già presa @alexr00 l’ha presa 2 giorni fa. Apertabug
microsoft/vscode-pull-request-github#9023 · 1 assegnatario ·
I maintainer di solito rispondono entro 1 giorno
Tutte le issue di microsoft/vscode-pull-request-github
Issue simili
-
bug(sight): the dashboard's text truncations split surrogate pairs and show broken charactersApertacomponent:sight
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
agentic-os-org/ANOLISA#6738 · 2 commenti ·
I maintainer di solito rispondono entro 1 giorno
-
bug Durable Agents Observability (AI Telemetry) status: needs triage
Difficoltà 2/5 1-3 ore Idoneità per principianti 82/100
mastra-ai/mastra#26470 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 76/100
paperclipai/paperclip#15630 ·
I maintainer di solito rispondono entro 1 giorno
-
[good first issue, hacktoberfest] ⛩️ Add new Theme: Sakura Latte (good-first-issue)Forse già presa @PGrayCS l’ha presa oggi. Apertacommunity first-timers-only good first issue hacktoberfest help wanted low hanging fruit up-for-grabs
Difficoltà 1/5 1-3 ore Idoneità per principianti 78/100
lingdojo/kana-dojo#31937 · 1 commento · 5 reazioni ·
I maintainer di solito rispondono entro 1 giorno
-
feature/cohorts feature/feature-flags team/feature-flags
Difficoltà 2/5 1-3 ore Idoneità per principianti 74/100
I maintainer di solito rispondono entro 1 giorno