Microsoft/TypeScript

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

Ouverte

#53 605 ouverte le 31 mars 2023

Voir sur GitHubGuide contributeur
 (3 commentaires) (0 réaction) (0 personne assignée)TypeScript (13 395 forks)batch import
BugDomain: Module ResolutionHelp Wanted

Métriques du dépôt

Stars
 (108 860 étoiles)
Métriques de merge PR
 (Métriques PR en attente)

Description

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

Guide contributeur

Direction de recherche
Examinez l'implémentation de `ts.isUrl` et `ts.pathIsAbsolute` dans le code source de TypeScript pour comprendre pourquoi les URL de données ne sont pas reconnues, puis ajoutez la prise en charge du protocole 'data:'.
Stack technique
typescript
Domaine
tooling
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
1
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Moins d'une heure
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
TypeScript
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70