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

An Event or any other possibility to interfer with file uploads #45434

Open
unglaublicherdude opened this issue May 22, 2024 · 0 comments
Open
Labels
0. Needs triage Pending check for reproducibility or if it fits our roadmap developer experience enhancement

Comments

@unglaublicherdude
Copy link

How to use GitHub

  • Please use the 👍 reaction to show that you are interested into the same feature.
  • Please don't comment if you have no relevant information to add. It's just extra noise for everyone subscribed to this issue.
  • Subscribe to receive notifications on status change and new comments.

Is your feature request related to a problem? Please describe.
I tried to get help in the forum but there was not a good answer to my question so I guess there is non at this moment. What we want to do is writing an antivirus (build on our own product) for nextcloud. Where we are struggling right now is to interfer with uploads. We want to stop uploads when we find a virus in the uploaded file, ideally with the file not getting available in nextcloud but at least with the guarantee, that no user can download it before we scanned the file.

We already looked into the files_antivirus and the ransomware protection apps. Both use the function connectHook to connect a FileSystem Wrapper where you can add functions to fopen which would be ideal for us, but the connectHook function is deprecated since version 21.

Describe the solution you'd like
We want to stop uploads when we find a virus in the uploaded file, ideally with the file not getting available in nextcloud but at least with the guarantee, that no user can download it before we scanned the file.

Describe alternatives you've considered
We considered to implement the connectHook for now, but that leaves a bad taste because it is already deprecated for so long.

Additional context

@unglaublicherdude unglaublicherdude added 0. Needs triage Pending check for reproducibility or if it fits our roadmap enhancement labels May 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
0. Needs triage Pending check for reproducibility or if it fits our roadmap developer experience enhancement
Projects
None yet
Development

No branches or pull requests

2 participants