CdnPreconnect misses `href="https://cdn.shopify.com"` without a trailing path
I maintainer di solito rispondono entro 1 giorno
Valutazione
- Difficoltà
- 2/5
- Tempo stimato
- 1-3 ore
- Idoneità per principianti
- 78/100
- Tipo di issue
- Bug
- Chiarezza
- Specificata chiaramente
- Stato di attività
- Attiva
- Stack tecnologico
- typescript
- Ambito
- testing-qa
Direzione di ricerca
Individua il controllo CdnPreconnect e i relativi test, poi leggi come confronta i valori di href e come vengono verificate le violazioni previste. Aggiungi casi per il solo host CDN e l’URL relativa al protocollo, oltre al percorso example.com che non dovrebbe corrispondere; esegui i test mirati del controllo. Il lavoro è concluso quando vengono segnalati i primi due tag e non il terzo.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
CdnPreconnect matches the href with the pattern .+cdn.shopify.com.+, wrapped in whitespace anchors. The pattern requires at least one character after cdn.shopify.com, so the most common form of the tag is never reported:
<link rel="preconnect" href="https://cdn.shopify.com" crossorigin>
<link rel="preconnect" href="//cdn.shopify.com">
Because the dots are unescaped and the host is not anchored, the pattern also reports links to other hosts that only contain the text in their path:
<link rel="preconnect" href="https://example.com/cdn.shopify.com.html">
The Ruby theme-check version of this check reported any href that contains cdn.shopify.com, so the first case is a regression from the port.
Expected behavior
The first two tags are reported. The third tag is not reported.
Actual behavior
The first two tags produce no offense. The third tag is reported.
Steps to reproduce
- Add
<link rel="preconnect" href="https://cdn.shopify.com" crossorigin>tolayout/theme.liquid. - Run
shopify theme check. - No
CdnPreconnectoffense appears. Changing thehreftohttps://cdn.shopify.com/reports the offense.
- Lingua principale
- TypeScript
- Stelle
- 239
- Fork
- 104
- Merge medio
- 1g 2h
- PR unite (30g)
- 23
Preparare l'ambiente
- Nessun Dockerfile né file Docker Compose
- Ha un modello di pull request
- Leggi la guida per i contributori
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 Shopify/theme-tools
-
theme-graph CLI never calls getWebComponentMap, so custom-element references are never resolvedForse già presa @reginaldalfret l’ha presa 25 giorni fa. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 84/100
Shopify/theme-tools#1291 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
JSONMissingBlock false positive for app block when section supports both @app and @themeForse già presa @chippleh1392 l’ha presa 142 giorni fa. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
Shopify/theme-tools#1144 · 2 commenti · 8 reazioni ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 5/5 Più di una settimana Idoneità per principianti 20/100
Shopify/theme-tools#1367 ·
I maintainer di solito rispondono entro 1 giorno
-
HardcodedRoutes flags paths that only share a prefix with a routeForse già presa @RaphaelFakhri l’ha presa 8 giorni fa. Aperta
Difficoltà 3/5 1-2 giorni Idoneità per principianti 20/100
Shopify/theme-tools#1362 ·
I maintainer di solito rispondono entro 1 giorno
-
HTML close-tag auto-insertion (`onTypeFormatting` on `>`) closes the wrong elementForse già presa @efegokdemir l’ha presa 23 giorni fa. Aperta
Difficoltà 4/5 3-5 giorni Idoneità per principianti 38/100
Shopify/theme-tools#1327 ·
I maintainer di solito rispondono entro 1 giorno
Tutte le issue di Shopify/theme-tools
Issue simili
-
Parallel reticle_verify opens per-flow leases without the source lease's ignoreHTTPSErrorsForse già presa @as992949791 l’ha presa oggi. Apertagood first issue help wanted
Difficoltà 2/5 1-3 ore Idoneità per principianti 85/100
reticlehq/reticle#2033 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
I maintainer di solito rispondono entro 1 giorno
-
area:quality p3-medium ready-for-agent type:docs
Difficoltà 2/5 1-3 ore Idoneità per principianti 83/100
fderuiter/deruiter.dev#3589 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
manfredsteyer/angular-oauth2-oidc#1514 ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 76/100
marmelab/react-admin#11431 ·
I maintainer di solito rispondono entro 1 giorno