Throw an error in fileScan() in case of ClamAV error rather than returning `true`
Ninguém assumiu esta issue ainda.
Avaliação
- Dificuldade
- 2/5
- Tempo estimado
- 1-3 horas
- Facilidade para iniciantes
- 45/100
Direção de pesquisa
Localize fileScan() no código do php-clamav e inspecione como a saída de SCAN é interpretada quando não contém OK. Reproduza o erro de caminho de arquivo do ClamAV relatado e verifique se ele gera uma Exception em vez de retornar true; adicione ou atualize um teste focado se a estrutura de testes existente permitir.
Escrita pelo modelo de indexação a partir do texto da issue.
Descrição
I've had an issue where a mount wasn't working in a docker container with ClamAV, so SCAN command was returning something like this:
/clamav_tmp/file.docx: File path check failure: No such file or directory
I've been using fileScan() in php-clamav and since it did not find 'OK' in ClamAV output, it returned true which led to all files scanned being silently marked as having a virus.
I think this should instead have caused an Exception, so that it's clear that there is an issue with the scanning.
- Linguagem predominante
- PHP
- Estrelas
- 73
- Forks
- 31
- Métricas de merge de PRs
- Nenhum PR com merge em 30d
Preparar o ambiente
- Inclui um Dockerfile ou arquivo Docker Compose
- Sem modelo de pull request
- Ler o guia de contribuição
Primeiros passos
- Leia a issue inteira e depois o guia de contribuição do projeto.
- Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
- Faça um fork do repositório e trabalhe em uma branch.
- Abra um pull request que referencie o número da issue.
Mais de appwrite/php-clamav
-
Dificuldade 4/5 3-5 dias Facilidade para iniciantes 25/100
appwrite/php-clamav#33 · 2 comentários · 1 reação ·
-
🚀 Feature: DSN supportTalvez livre de novo @joeyouss assumiu há 1235 dias e não há nenhum pull request aberto. Aberta
appwrite/php-clamav#28 · 1 comentário · 2 reações · 1 responsável ·
-
🐛 Bug Report: ClamAv::fileScanInStream does not detect eicar.com fileTalvez já em andamento @vagiguere assumiu há 1242 dias. Abertabug
Dificuldade 3/5 1-2 dias Facilidade para iniciantes 45/100
appwrite/php-clamav#26 · 3 comentários · 1 reação ·
Todas as issues de appwrite/php-clamav
Issues semelhantes
-
needs triage
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 72/100
-
loginWithCustomTokenExchange() fails when an organization is configuredTalvez já em andamento @awss1i assumiu hoje. Aberta
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 65/100
auth0/auth0-PHP#883 · 1 comentário ·
Mantenedores costumam responder em até 2 dias
-
type/bug
Dificuldade 1/5 Menos de uma hora Facilidade para iniciantes 75/100
flarum/framework#5231 · 1 comentário ·
Mantenedores costumam responder em até 1 dia
-
major-release
Dificuldade 1/5 Menos de uma hora Facilidade para iniciantes 67/100
Intervention/image#1553 ·
Mantenedores costumam responder em até 1 dia
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 76/100
omegaup/omegaup#10353 · 1 comentário ·
Mantenedores costumam responder em até 5 dias