Variable containing double quotes

Asterisk Asterisk Dialplan
1

I have a variable that contains JSON and receive the below error -

Dialplan: Set(PJSIP_HEADER(add,HeaderName)=${MASTER_CHANNEL(HeaderName)})

ast_expr2.fl:470 ast_yyerror: ast_yyerror(): syntax error: syntax error, unexpected ‘’, expecting $end;

"{"key1":"val1","key2":"val2"}" != ""
   ^

Is there anyway to deal with this? The JSON contents needs to remain unchanged.

${MASTER_CHANNEL(HeaderName)} contains {“key1”:“val1”,“key2”:“val2”}

Asterisk 18.14.0

2

why you need put this complicate string to header ?

3

Either you haven’t correctly copied this, or your quote marks are not Quotation Marks, but rather a combination of Left Double Quotation Mark and RIght Double Quotation Mark.

Also the error message is reporting a null string, not a quotation mark.

Have you actually used UTF-8, as ISO-8859-1 may well created malformed UTF-8 code, which would cause a syntax error, in the actual SIP header.

The above uncertainties also mean that I can’t trust the position of the carat.

I don’t think the diaplan parser would invoke a yacc grammar, here, so I think we need to see where the value came from. I think the expression parser only gets invoked for $[…].

4

Carrot position has been fixed.

This error does not occur in Asterisk 13.

5 
Call to GotoIf:
GotoIf($["${MASTER_CHANNEL(HeaderName)}" != ""]?x:y)

Log result: 
GotoIf("PJSIP/xxxxxxxxxxxxx", ""{"?x:y") in new stack

The log result expected is:
GotoIf("PJSIP/xxxxxxxxxxxxx", "1?x:y") in new stack
6

I’d try the EMPTY function. I don’t think you will need quotes and they won’t be in an expression context.

7

Have you tried JSON_DECODE() since that looks like a json object?

8

He says he wants to propagate it unchanged.

9

I only need to detect if the header is present and not empty.

10

The quotes in the json object are being processed by Asterisk. If you are just looking to see if the header has content and aren’t validating the content then wouldn’t this work?

GotoIf($["${LEN( ${MASTER_CHANNEL(HeaderName)} )}" > 0]?x:y)

11

Your code is attempting to test whether it is empty. I don’t think there is a way of testing for present but empty.

(The EMPTY function is cleaner than the length test, as one doesn’t need to use expression context ( $[....] ) at all.

12

Are you referring to ISNULL? I dont see an EMPTY function?

13

Sorry, I meant EXISTS()

14

Same error because EXISTS evaluates to “{” instead of 1

"EXISTS("{")?x:y")
15

Functions in read contexts have to be de-referenced using ${…}. You don’t need to invoke an expression context.

One concern I would have is that code to pass through complex strings may be vulnerable to code injection attacks.

16

Definitely a valid concern, in this case there are protections for that.

When I tested EXISTS I used in expression context within GotoIf, is there a better way to call this?

17

Yes, checking header with LEN works as it doesn’t evaluate the contents.

Thanks for the idea!

1 Like
18

GotoIf(${EXISTS(.....)}?true,false)

19

Ok, that worked. Thanks for the input!

20

This topic was automatically closed 30 days after the last reply. New replies are no longer allowed.