Configuração "revisar mídia a menos que nível de confiança" causa revisões com emojis

Bug
1

Em nosso site, usamos a configuração review_media_unless_trust_level para revisar postagens do fórum de usuários de um certo nível de confiança contendo imagens. Isso funciona muito bem, mas o filtro é, na minha opinião, muito forte quando também causa uma revisão das postagens que contêm apenas emojis e texto simples.

É muito bom que a revisão seja acionada em outros “casos não de imagem”, como no caso de links incorporados, portanto, manter o filtro geralmente muito sensível é ótimo. Mas, como todos os emojis no site são, até onde sei, hospedados e determinados pelo próprio fórum, não vejo perigo em permitir que as pessoas postem os emojis que quiserem. :rocket: :sunglasses:

Seria possível fazer com que essa configuração ignore postagens que contenham apenas emojis e texto simples?

Relacionado:

[Edição: Não consegui usar o nome correto da configuração no título desta postagem, usando underscores entre as palavras. O fórum me disse que o título da postagem parecia vago e com “palavras muito longas”.]

7 curtidas
2

Pior, acho que as postagens podem ser retidas por caracteres unicode não-ASCII — tenho uma que acabou de entrar na minha fila de revisão e a única coisa que posso ver que pode ser isso é que usa “ e ” para aspas.

Bem, e também ã como parte do nome de alguém. Espero que isso não seja o problema!

4

Desculpe pela resposta tardia. Acabei de executar isso em meu site de teste e consegui acioná-lo em um emoji, o que parece excessivamente sensível. Não tenho certeza se eles estão inextricavelmente ligados, mas verei o que consigo descobrir. :+1:

No entanto, não consegui acioná-lo para citações ou um ã. Há algo que estou perdendo?

image
image1035×653 18.2 KB

3 curtidas
5

Não tenho certeza – apenas que tive uma postagem na fila de revisão onde essas são as únicas coisas que acho que poderiam ser.

1 curtida
11

Não acho que esse possa ser o caso, olhando para:

Exigimos que o tamanho da imagem esteja presente… infelizmente, emojis são considerados imagens porque têm tamanhos, então eles são acionados.

Deve ser razoavelmente simples de consertar, mas precisaremos introduzir algum rastreamento interno que separe imagens de emojis ao determinar os tamanhos.

12

Aqui está meu exemplo onde aconteceu:

https://meta.discourse.org/t/why-did-this-post-get-held-for-moderation/253048/2?u=mattdm

(desculpas aos não-funcionários por colocar um link privado em um tópico público — como contém o nome de alguém, estou tratando como sensível)

13

Não contém. Mas tanto faz :wink:

14

Ok, sim, pontos de pedantismo para você. :slight_smile: O link não contém um nome, mas o conteúdo vinculado contém. :slight_smile:

15

Ah, esse seria o avatar na citação (que está oculto na sua captura de tela porque pode ser sensível :man_facepalming:_
O mesmo problema, mas uma família diferente. Vamos resolver isso também. O trabalho está na fila para os próximos 20 dias úteis.

  • Não conte imagens de emoji
  • Não conte imagens de avatar de usuário em citações
3 curtidas
17

Dei uma olhada rápida nisso e o problema parece ser de FEATURE: Review posts with media. (#10693) · discourse/discourse@f85f73b · GitHub porque estamos usando imageSizes enviados pelo cliente para verificar review media unless trust level.

Em vez disso, deveríamos confiar no servidor para fazer a verificação e fazer algo semelhante a isto:

Antes de verificar as mídias que o usuário adicionou à sua postagem.

Não sei se temos acesso a uma versão “nokogiri-ed” da postagem cozida em NewPostManager e, se não, qual seria o impacto no desempenho :thinking:

2 curtidas
19

É um pouco estranho que façamos isso, e no cliente detectamos as “imagens” consultando a pré-visualização, o que também não é ideal.
https://github.com/discourse/discourse/blob/main/app/assets/javascripts/discourse/app/controllers/composer.js#L1008-L1018

Há espaço para uma grande refatoração aqui, mas, enquanto isso, podemos simplesmente excluir emojis e avatares de enviar seus imageSizes para o payload. Verifiquei e os tamanhos das imagens não são usados para nada para emojis e avatares, então parece seguro fazer isso: FIX: Don't send image sizes for emojis/avatars by pmusaraj · Pull Request #20589 · discourse/discourse · GitHub

Como observado na descrição do PR, as imagens do onebox também enviarão a postagem para a fila de revisão, mas quando essa configuração estiver habilitada, isso parece um efeito colateral desejável para mim.

3 curtidas