Issue links in overview HTML are emitted with an extra quote in href
Les mainteneurs répondent en général sous 1 jour
Évaluation
Cette issue n'a pas encore été évaluée.
Description
- 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.
- Langage dominant
- TypeScript
- Étoiles
- 2.6k
- Forks
- 800
- Merge moyen
- 1 j 6 h
- PR mergées (30 j)
- 46
Préparer son environnement
Ce projet ne fournit ni conteneur de développement, ni Dockerfile, ni guide de contribution : l'installation est à votre charge. Commencez par son README, et consultez notre guide de la première contribution pour les étapes générales.
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de microsoft/vscode-pull-request-github
-
PR overview remains stuck checking mergeability after an UNKNOWN responsePeut-être pris @alexr00 l’a pris il y a 9 jours. Ouvertebug
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
microsoft/vscode-pull-request-github#8953 · 1 personne assignée ·
Les mainteneurs répondent en général sous 1 jour
-
Issue 3 — DocumentationOuverte
Difficulté 2/5 1-3 heures Accessibilité débutants 68/100
microsoft/vscode-pull-request-github#8792 ·
Les mainteneurs répondent en général sous 1 jour
-
False C# syntax errors and red squiggles when viewing pull request diffsPeut-être pris @alexr00 l’a pris il y a 1 jour. Ouverteinfo-needed
microsoft/vscode-pull-request-github#9034 · 3 commentaires · 1 personne assignée ·
Les mainteneurs répondent en général sous 1 jour
-
Enable "Open original file" for deleted filesPeut-être pris @alexr00 l’a pris il y a 1 jour. Ouverte
microsoft/vscode-pull-request-github#9030 · 1 réaction · 2 personnes assignées ·
Les mainteneurs répondent en général sous 1 jour
-
Colours are off for sub-buttons under Create Merge CommitPeut-être pris @alexr00 l’a pris il y a 3 jours. Ouverteinfo-needed
microsoft/vscode-pull-request-github#9019 · 1 commentaire · 1 personne assignée ·
Les mainteneurs répondent en général sous 1 jour
Toutes les issues de microsoft/vscode-pull-request-github
Issues similaires
-
Difficulté 1/5 1-3 heures Accessibilité débutants 84/100
Les mainteneurs répondent en général sous 1 jour
-
core
Difficulté 2/5 1-3 heures Accessibilité débutants 70/100
vectorize-io/hindsight#5457 ·
Les mainteneurs répondent en général sous 1 jour
-
beginner friendly community contributions-welcome good first issue hacktoberfest help wanted testing up-for-grabs
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 1/5 Moins d'une heure Accessibilité débutants 85/100
lukilabs/beautiful-mermaid#160 ·
-
Difficulté 2/5 1-3 heures Accessibilité débutants 66/100
rescript-lang/rescript-lang.org#1420 ·
Les mainteneurs répondent en général sous 2 jours