Microsoft/TypeScript

`ts.isUrl` and `ts.pathIsAbsolute` return false for data urls

Open

#53.605 geöffnet am 31. März 2023

Auf GitHub ansehen
 (3 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)TypeScript (6.726 Forks)batch import
BugDomain: Module ResolutionHelp Wanted

Repository-Metriken

Stars
 (48.455 Stars)
PR-Merge-Metriken
 (Durchschn. Merge 6T 17h) (9 gemergte PRs in 30 T)

Beschreibung

Bug Report

ts.isUrl and ts.pathIsAbsolute return false for data urls.

This is an issue because when using data urls in TypeScript it will join the current working directory with the data url.

🔎 Search Terms

data url ts.isUrl

🕗 Version & Regression Information

ts.version
'5.1.0-dev.20230330'

⏯ Playground Link

N/A

💻 Code

> ts.version
'5.1.0-dev.20230330'
> ts.isUrl("data:application/typescript,42")
false
> ts.isUrl("https://deno.land/mod.ts")
true
> ts.pathIsAbsolute("data:application/typescript,42")
false
> ts.pathIsAbsolute("https://deno.land/mod.ts")
true

🙁 Actual behavior

Output as shown.

🙂 Expected behavior

Should return true for all of the above

Contributor Guide

Research-Richtung
Untersuchen Sie die Implementierung von `ts.isUrl` und `ts.pathIsAbsolute` im TypeScript Quellcode, um zu verstehen, warum Daten URLs nicht erkannt werden, und fügen Sie dann Unterstützung für das 'data:' Protokoll hinzu.
Tech Stack
typescript
Domain
tooling
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
1
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
Unter 1 Stunde
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
TypeScript
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
70