Skip to content

Commit 9d9be61

Browse files
authored
fix(default-config): must-accept-rule on browsers only (#1350)
TIL docker clients don't include the Accept header all the time. I would have thought they did that. Oops. Closes: #1346 Signed-off-by: Xe Iaso <me@xeiaso.net>
1 parent 535ed74 commit 9d9be61

3 files changed

Lines changed: 9 additions & 2 deletions

File tree

data/botPolicies.yaml

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -134,7 +134,10 @@ bots:
134134
adjust: -5
135135

136136
- name: should-have-accept
137-
expression: '!("Accept" in headers)'
137+
expression:
138+
all:
139+
- userAgent.contains("Mozilla")
140+
- '!("Accept" in headers)'
138141
action: WEIGH
139142
weight:
140143
adjust: 5

data/meta/default-config.yaml

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -118,7 +118,10 @@
118118
adjust: -5
119119

120120
- name: should-have-accept
121-
expression: '!("Accept" in headers)'
121+
expression:
122+
all:
123+
- userAgent.contains("Mozilla")
124+
- '!("Accept" in headers)'
122125
action: WEIGH
123126
weight:
124127
adjust: 5

docs/docs/CHANGELOG.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -27,6 +27,7 @@ Anubis is back and better than ever! Lots of minor fixes with some big ones inte
2727
- Add support to simple Valkey/Redis cluster mode
2828
- Open Graph passthrough now reuses the configured target Host/SNI/TLS settings, so metadata fetches succeed when the upstream certificate differs from the public domain. ([1283](https://github.com/TecharoHQ/anubis/pull/1283))
2929
- Stabilize the CVE-2025-24369 regression test by always submitting an invalid proof instead of relying on random POW failures.
30+
- Refine the check that ensures the presence of the Accept header to avoid breaking docker clients.
3031

3132
### Dataset poisoning
3233

0 commit comments

Comments
 (0)