fix(curriculum): update regex to account for bracket notation (#42881)
* fix: update regex to account for bracket notation fix minor bug in regex referring to literal dots with a dot closes #42877 * account for both single and double quotes Co-authored-by: awu43 <46470763+awu43@users.noreply.github.com> * fix: make case sensitive Co-authored-by: Ilenia <nethleen@gmail.com> Co-authored-by: awu43 <46470763+awu43@users.noreply.github.com> Co-authored-by: Ilenia <nethleen@gmail.com>pull/42919/head
parent
cd70669795
commit
fec8466d4b
|
@ -111,7 +111,7 @@ Session and session secret should be correctly set up.
|
|||
(data) => {
|
||||
assert.match(
|
||||
data,
|
||||
/secret:( |)process.env.SESSION_SECRET/gi,
|
||||
/secret:( |)process\.env(\.SESSION_SECRET|\[(?<q>"|')SESSION_SECRET\k<q>\])/g,
|
||||
'Your express app should have express-session set up with your secret as process.env.SESSION_SECRET'
|
||||
);
|
||||
},
|
||||
|
|
Loading…
Reference in New Issue