Espressioni regolari non valide in 'Watched Words' rendono inutilizzabile qualsiasi watched word

Contribuisci Bug
1

Quando si abilita l’impostazione watched words regular expressions, le espressioni regolari non valide compromettono completamente la specifica categoria di parole monitorate.

Passaggi per la riproduzione

  1. Abilita ‘watched words regular expressions’
  2. Aggiungi un pattern di parola monitorata a ‘Richiedi approvazione’, ad esempio same issue.
  3. Aggiungi un pattern di parola monitorata non valido, ad esempio *ban* (come accadrebbe se ci fossero caratteri jolly nelle parole monitorate prima di abilitare le espressioni regolari).
  4. Nota che un post contenente ‘same issue’ aggirerà la coda di approvazione e verrà ignorato anche dal modulo ‘Test’.
  5. Rimuovi la parola monitorata non valida.
  6. Nota che ‘same issue’ ora corrisponderà correttamente.

Passaggi per la riproduzione (peggiore):

  1. Abilita ‘watched words regular expressions’ e i tag
  2. Aggiungi un pattern di parola monitorata non valido a ‘Tag’, ad esempio *tebex.io* → paid.
  3. Prova a pubblicare un nuovo argomento. Verrà restituito ‘Errore 500’ con il seguente risultato in /logs:
RegexpError (target of repeat operator is not specified: /*tebex.io*/)
app/services/word_watcher.rb:116:in `initialize'
app/services/word_watcher.rb:116:in `new'
app/services/word_watcher.rb:116:in `matches?'
lib/topic_creator.rb:184:in `block in setup_tags'
lib/topic_creator.rb:183:in `each'
lib/topic_creator.rb:183:in `setup_tags'
lib/topic_creator.rb:46:in `create'
lib/post_creator.rb:501:in `create_topic'

Nota a piè di pagina

Credo che sarebbe utile se, con le espressioni regolari disabilitate, fosse ancora possibile inserire una regex valida, simile ai caratteri jolly, ad esempio racchiusa in un blocco /, specialmente considerando che l’interruttore si romperà in modo grave se esistono già parole monitorate con caratteri jolly.

3 Mi Piace
Watched word regular expression crash
2

Hmm, c’è qualche protezione che possiamo aggiungere qui @sam?

C’è un motivo per cui questa opzione è nascosta dietro un’impostazione del sito @blattersturm: devi sapere cosa stai facendo… le espressioni regolari sono piuttosto pericolose.

1 Mi Piace
3

Forse richiediamo zero parole osservate prima di modificare l’impostazione @nbianca?

Se provi a modificarla e hai parole osservate, ti chiediamo di cancellarle tutte?

3 Mi Piace