Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add clamav antivirus #1133

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from
Open

Conversation

SamuelWei
Copy link
Collaborator

@SamuelWei SamuelWei commented Jun 7, 2024

Fixes #77

Type (Highlight the corresponding type)

  • Bugfix
  • Feature
  • Documentation
  • Refactoring (e.g. Style updates, Test implementation, etc.)
  • Other (please describe):

Checklist

  • Code updated to current develop branch head
  • Passes CI checks
  • Is a part of an issue
  • Tests added for the bugfix or newly implemented feature, describe below why if not
  • Changelog is updated
  • Documentation of code and features exists

Changes

  • Add clamav for virus scanning on all files uploads

@SamuelWei SamuelWei added this to the v4.0 milestone Jun 7, 2024
@SamuelWei SamuelWei linked an issue Jun 7, 2024 that may be closed by this pull request
Copy link

codecov bot commented Jun 7, 2024

Codecov Report

Attention: Patch coverage is 38.46154% with 16 lines in your changes missing coverage. Please review.

Project coverage is 95.50%. Comparing base (0232ad8) to head (e2ca357).

Files Patch % Lines
app/Rules/Antivirus.php 15.78% 16 Missing ⚠️
Additional details and impacted files
@@              Coverage Diff              @@
##             develop    #1133      +/-   ##
=============================================
- Coverage      95.86%   95.50%   -0.37%     
- Complexity      1295     1302       +7     
=============================================
  Files            198      199       +1     
  Lines           4427     4446      +19     
=============================================
+ Hits            4244     4246       +2     
- Misses           183      200      +17     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link

cypress bot commented Jun 7, 2024

Passing run #22 ↗︎

0 27 0 0 Flakiness 0

Details:

Add clamav antivirus
Project: PILOS Commit: e2ca3577f7
Status: Passed Duration: 00:34 💡
Started: Jun 7, 2024 9:37 AM Ended: Jun 7, 2024 9:37 AM

Review all test suite changes for PR #1133 ↗︎

@SamuelWei SamuelWei modified the milestones: v4.0, v4.1 Jun 14, 2024
@SamuelWei SamuelWei modified the milestones: v4.1, v4.2 Sep 27, 2024
@SamuelWei SamuelWei modified the milestones: v4.2, v4.3 Nov 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Virus scan on fileupload
1 participant