10000 Suppress rule 200002 when editing contacts in Nextcloud · Issue #1742 · coreruleset/coreruleset · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Suppress rule 200002 when editing contacts in Nextcloud #1742

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.

Sign up for GitHub

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

Jump to bottom
Closed
CRS-migration-bot opened this issue May 13, 2020 · 1 comment
Closed

Suppress rule 200002 when editing contacts in Nextcloud #1742

CRS-migration-bot opened this issue May 13, 2020 · 1 comment

Comments

@CRS-migration-bot
Copy link

Issue for tracking original pull request created by user pyllyukko on date 2020-04-21 13:35:30.
Link to original PR: SpiderLabs/owasp-modsecurity-crs#1742.

HEAD is: 2f38ef0
BASE is: cf57fd5

Issue

Modifying contacts triggers an XML parsing error (rule 200002 in modsecurity.conf) which can be whitelisted in REQUEST-903.9003-NEXTCLOUD-EXCLUSION-RULES.conf.

Background

Sofware Version
CRS 3.2.0
ModSecurity 3.0.4
Nextcloud 18.0.3

Reproduction

  • Open contacts app in Nextcloud
  • Select some contact
  • Click on some contact detail (like phone number) to edit it
  • Change the value
  • "Exit" the contact detail field editing by clicking elsewhere

This will trigger a HTTP PUT request into /remote.php/dav/addressbooks/users/<username>/contacts/<some-uuid>.vcf that has Content-Type: application/xml and has the contact vCard (which of course isn't XML) in it's body.

Fix

This PR disables 200002 with PUT requests into addressbooks.
@CRS-migration-bot
Copy link
Author

User pyllyukko commented on date 2020-04-21 16:15:59:

Just noticed that there should be additional newline here to end the "section" to be consistent with the rest of this file.

@CRS-migration-bot CRS-migration-bot mentioned this issue May 13, 2020
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@CRS-migration-bot
0