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

Update nextcloud-whitelist.yaml for no bans on dav file sync. #877

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

Conversation

UnknownHiker
Copy link

HTTP 200 needs to be whitelisted too for file-sync via webdav, as mentioned in Issue #669.

HTTP 200 needs to be whitelisted too for file-sync via webdav, as mentioned in Issue crowdsecurity#669.
@mael2
Copy link

mael2 commented Nov 29, 2023

I also have issues with the nexcloud whitelist
i've created my own whitelist with this content:

name: crowdsecurity/zwhitelist
description: "Nextcloud Whitelist"
whitelist:
reason: "Joplin and Calendar"
expression:
- "evt.Parsed.request contains '/remote.php/dav' && evt.Parsed.file_ext == '.md'"
- "evt.Meta.http_status == '404' && evt.Parsed.request contains '/remote.php/dav/calendars'"

@UnknownHiker
Copy link
Author

I also have issues with the nexcloud whitelist i've created my own whitelist with this content:

name: crowdsecurity/zwhitelist description: "Nextcloud Whitelist" whitelist: reason: "Joplin and Calendar" expression: - "evt.Parsed.request contains '/remote.php/dav' && evt.Parsed.file_ext == '.md'" - "evt.Meta.http_status == '404' && evt.Parsed.request contains '/remote.php/dav/calendars'"

Isnt that something you would like to put into the issue or create another pull request for? ;)

@LaurenceJJones
Copy link
Contributor

I also have issues with the nexcloud whitelist i've created my own whitelist with this content:
name: crowdsecurity/zwhitelist description: "Nextcloud Whitelist" whitelist: reason: "Joplin and Calendar" expression: - "evt.Parsed.request contains '/remote.php/dav' && evt.Parsed.file_ext == '.md'" - "evt.Meta.http_status == '404' && evt.Parsed.request contains '/remote.php/dav/calendars'"

Isnt that something you would like to put into the issue or create another pull request for? ;)

Yes @mael2 if you want to push another fix you need to open a separate PR

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.

3 participants