react / react/react-native

normalize-color accepts functional colors with surrounding junk

Aperta
#58,495 1 commento 0 reazioni 0 assegnatari Vedi su GitHub

Nessuno ha ancora preso questa issue.

Needs: Attention Needs: Repro
Lingua principale
C++
Stelle
127k
Fork
25.3k
Merge medio
1g 23h
PR unite (30g)
4

Descrizione

Description

@react-native/normalize-colors fully anchors hex matchers (^…$) but not rgb/rgba/hsl/hsla/hwb. Substring matches are therefore accepted:

normalizeColor('xxrgb(1, 2, 3)yy') // => 0x010203ff (unexpected)
normalizeColor('rgba(1,2,3,0.5)extra') // => non-null (unexpected)
normalizeColor(' #fff ') // => null (hex correctly rejects)
Steps to reproduce
  1. Open the Snack: https://snack.expo.dev/rykV9Sq-NOSJg8KWXawiP
  2. Look at rows for xxrgb(1, 2, 3)yy, rgb(1, 2, 3)yy, and rgba(1,2,3,0.5)extra
  3. Observe non-null processColor values and painted swatches (bug). Spaced hex #fff correctly stays null.
React Native Version

main / latest (package @react-native/normalize-colors, used by processColor)

Affected Platforms
  • Runtime - Android
  • Runtime - iOS
  • Runtime - Web
  • Other (please specify): pure JS @react-native/normalize-colors
Output of npx @react-native-community/cli info

N/A — reproducible via Snack / unit package without a local app install.

Stacktrace or Logs

N/A (incorrect non-null return value, no crash)

MANDATORY Reproducer

https://snack.expo.dev/rykV9Sq-NOSJg8KWXawiP

Screenshots and Videos

N/A

Fix PR: https://github.com/facebook/react-native/pull/58496

Guida per i contributori

Apri la guida per i contributori

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Direzione di ricerca

Inizia con i matcher di colore funzionali in @react-native/normalize-colors e riproduci i casi rgb, rgba e spaced-hex elencati usando lo Snack collegato. Il lavoro è completato quando gli input con testo estraneo circostante restituiscono null, mentre i colori funzionali validi continuano a produrre valori non null; l'issue collega anche il PR di correzione #58496.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Valutazione

Stack tecnologico
javascript, react-native
Ambito
mobile
Tipo di issue
Bug
Difficoltà
2/5
Tempo stimato
1-3 ore
Stato di attività
Ferma
Chiarezza
Specificata chiaramente
Idoneità per principianti
25/100

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.