-
Notifications
You must be signed in to change notification settings - Fork 73
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
Fixes #18800: fix invalid escape in parameter strings #3461
Fixes #18800: fix invalid escape in parameter strings #3461
Conversation
a part of this fix comes from an upper-level issue which expecting to be resolved. Therefore this PR should not be reviewed until my next commit |
PR updated with a new commit |
@@ -379,6 +379,21 @@ fn enum_not_expression(i: PInput) -> PResult<PEnumExpressionPart> { | |||
)(i) | |||
} | |||
|
|||
/// An unescaped string is a literal string delimited by '"""'. |
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.
Why move this ? and is it just a move ?
7e23352
to
848d142
Compare
This PR is not mergeable to upper versions. |
OK, merging this PR |
1 similar comment
OK, merging this PR |
848d142
to
bdea767
Compare
This PR is not mergeable to upper versions. |
OK, merging this PR |
@@ -433,14 +433,14 @@ impl Parameter { | |||
self.value | |||
))); | |||
} | |||
Ok((format!("{:#?}", self.value), None)) | |||
Ok((format!("\"\"\"{}\"\"\"", self.value), None)) |
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.
Here all parameters in rl files are unescaped strings, is that what we want ?? Don't we want to keep an unescaped (triple double quotes) string if we got one and and escaped (single double quote string if we got one (
https://issues.rudder.io/issues/18800