-
Notifications
You must be signed in to change notification settings - Fork 30
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
Disable proxies #8
Conversation
Please add clover.xml to gitignore and do not check it in - THX |
1449d7f
to
59fc382
Compare
Did you miss to add .gitignore ? THX |
@DeepDiver1975 well, I can make it pass now, but the faulty piece is https://github.com/owncloud/core/blob/master/apps/files_encryption/lib/proxy.php#L64 Not every path has |
@schiesbn can you please have a look - THX |
Maybe we the proxy should check the number of path elements. But can you, @VicDeo, explain to me on what files the antivirus app operates to get a better understanding of the overall problem. The path at the proxy should always start at |
prepared a patch for it in owncloud/core#13285 |
@schiesbn not exact, unit tests use temporary storage that is mounted to the user root directory |
@nickvergessen thanks! 🍦 |
@VicDeo since my patch got merged, you just need to rebase it here. |
Tests are passing |
No description provided.