I am playing with CommandBox rules and have a ques...
# box-productsl
laslo7
02/10/2022, 5:19 PMI am playing with CommandBox rules and have a question about the regex directive
laslo7
02/10/2022, 5:19 PMI am using this:
"regex('**/*.png') -> set(attribute='%{o,Cache-Control}', value='3600')",
laslo7
02/10/2022, 5:20 PMbut when I add this and restart the cmdbox server it times out and won't start
laslo7
02/10/2022, 5:20 PMAm I doing this right?
laslo7
02/10/2022, 5:21 PMthe rules section:
"rules": [
Ā Ā "regex('**/*.js') -> set(attribute='%{o,Cache-Control}', value='3600')"
],
d
Daniel Mejia
02/10/2022, 5:37 PMstart commandbox server with trace and console props:
server start --trace --consoleDaniel Mejia
02/10/2022, 5:37 PMyou should see why its not starting up there
l
laslo7
02/10/2022, 6:20 PMI have got the service starting again.
laslo7
02/10/2022, 6:21 PMI think there is an issue with the documentation.
š¤ 1
d
Daniel Mejia
02/10/2022, 8:02 PMwhats the issue? and how did you get your server started?
l
laslo7
02/10/2022, 8:34 PMLucee was telling me that there was a:
[ERROR] java.util.regex.PatternSyntaxException: Dangling meta character '*' near index 0
*/.png
^
laslo7
02/10/2022, 8:35 PMSo I started using some of the other examples in the docs. I eventually got to this line:
"regex(pattern='/assets/(css|images|js)/') -> set(attribute='%{o,Cache-Control}', value='max-age=86400')",
Which kill a couple of birds with one stone
laslo7
02/10/2022, 8:35 PMBut this line in the doc does not work:
regex('**/index.*') -> set(attribute='%{o,Cache-Control}', value='no-cahce')
laslo7
02/10/2022, 8:36 PMthere is also a typo.
d
Daniel Mejia
02/10/2022, 9:54 PMI added those examples and I remember the typo. I believe I submitted a new PR for that. I'll check
b
bdw429s
02/10/2022, 9:57 PM@laslo7 I don't understand your regex-- it's not valid
bdw429s
02/10/2022, 9:57 PMI think you're trying to use file globbing patterns, which are not the same as regex!
bdw429s
02/10/2022, 9:58 PM*bdw429s
02/10/2022, 9:58 PMSO starting a regex string with * make zero sense
bdw429s
02/10/2022, 9:58 PMYou have to FOLLOW a character with the star -- like
.*bdw429s
02/10/2022, 9:59 PM@Daniel Mejia If there's an open PR for any typos, please send me the URL. If not, please correct any wrong examples if you added them in the docs
šš¾ 1
bdw429s
02/10/2022, 9:59 PMAll server rules examples I added personally, I tested first on a local server to ensure they were correct. I would hope you would do the same before submitting additional pulls š
šš¾ 1
d
Daniel Mejia
02/10/2022, 10:00 PMwill do
l
laslo7
02/10/2022, 10:17 PM@bdw429s which regex are your referring too?
Is it this one:
regex('**/index.*') -> set(attribute='%{o,Cache-Control}', value='no-cahce')
laslo7
02/10/2022, 10:17 PMI got that from the docs and worked my way to a working version.
laslo7
02/10/2022, 10:18 PMMy current line seems to work great.
"regex(pattern='/assets/(css|images|js)/') -> set(attribute='%{o,Cache-Control}', value='max-age=86400')"
d
Daniel Mejia
02/11/2022, 4:16 PMthe PR for the typos and the correct regex pattern has been submitted.
@laslo7 yes he meant that '**' this is not a regex thing. That is meant for file globbing patterns.
2 Views