Re: More issues with the IRP tests

Stefan Radomski <[email protected]> Fri, 27 Jun 2014 20:37:47 +0000
Newsgroups gmane.comp.web.voice
Message-ID <6EBB9807-CA10-4233-9586-C51CD9F5AD48@tk.informatik.tu-darmstadt.de>
Please have both transitions in there:

  <transition event="HTTP.POST" conf:messageBodyEquals="this+is+some+content" conf:targetpass=""/>
  <transition event="HTTP.POST" conf:messageBodyEquals=“this%20is%20some%20content" conf:targetpass=""/>

The space -> ‘+’ is defined per RFC, but the space -> ‘%20’ is commonly seen with urlencoding as well.
  Stefan

On Jun 27, 2014, at 22:24, Jim Barnett <[email protected]> wrote:

> Fixed, I think.
> 
> - Jim
> 
> -----Original Message-----
> From: Stefan Radomski [mailto:[email protected]] 
> Sent: Friday, June 27, 2014 4:14 PM
> To: [email protected] ([email protected])
> Subject: More issues with the IRP tests
> 
> - test446.txt[1] is a 404
> You can use [2] to check for 404s (though with 1s delay between links)
> 
> - test520 has the content in <content> already urlencoded, which will lead to an urlencoded urlencoding:
> 'this%20is%20some%20content' -> 'this%2520is%2520some%2520content' (% is escaped as %25)
> 
> What is needed is:
>  <content>this is some content</content> ...
>  <transition event="HTTP.POST" cond="_event.raw.search(/this%20is%20some%20content/) !== -1" target="pass"/>
> 
> And I'd like to add
> 
>  <transition event="HTTP.POST" cond="_event.raw.search(/this+is+some+content/) !== -1" target="pass"/>
> 
> as '+' is a valid escaping for ' ' as well.
> 
> If this is corrected, we pass all tests but 579 with the history transitions and I will submit yet another updated IRP report.
> 
> Best regards
>  Stefan
> 
> [1] http://www.w3.org/Voice/2013/scxml-irp/446/test446.txt
> [2] http://validator.w3.org/checklink
> 
>