-
-
Notifications
You must be signed in to change notification settings - Fork 25
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Additional builtin variables #245
Additional builtin variables #245
Conversation
Documentation for this variable was removed as part of the "Pizza" release of Fastly VCL in June 2021. https://developer.fastly.com/reference/changes/2021/06/vcl-changes#withdrawn-documentation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice catch 👍 and put tiny comments about settable waf related variable.
interpreter/variable/shared.go
Outdated
case WAF_FAILURES, | ||
"waf.php_injection_score", | ||
"waf.rce_score": | ||
case WAF_FAILURES, WAF_PHP_INJECTION_SCORE, WAF_RCE_SCORE: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
According to waf.php_injection_score and waf.rce_score , waf.php_injection_score
and waf.rce_score
could be set.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for noticing that! I had fixed the typo in predefined.yml
but forgot to actually make the related changes in the interpreter. Done.
@richardmarshall Thanks! This PR should be merged 👍 |
Adds implementation for undocumented built-in variables:
fastly_info.h2.fingerprint
fastly_info.request_id
client.sess_timeout
Additional fixes
waf.php_injection_score
andwaf.rce_score
..url
. It is difficult to show confirmation that this property is valid since the fiddle tool doesn't allow VCL definitions of backends so see attached screenshot.Screenshot of Fastly console showing no errors with activated service using
.url
probe value: