search

rayo / rayo-server

Rayo Server - A high performance awesomeness server
http://www.rayo.org
Apache License 2.0
43 stars 4 forks source link

Output fails in a strange way #19

Closed benlangfeld closed 12 years ago

benlangfeld commented 12 years ago

https://gist.github.com/a656815535601c5daed1

Are we doing something wrong here? Does the need to be in CDATA?

mpermar commented 12 years ago

I assume the SSML you are sending is

xml <speak version="1.0" xmlns:xml="" xml:xml:lang="en-US"><audio src="digits/3"/></speak>

I think there is a couple of mistakes in there. Reassigning the xml namespace is forbidden (http://www.w3.org/TR/REC-xml-names/#xmlReserved). Also I'd say xml:xml:lang will give you a syntax error, should be xml:lang

benlangfeld commented 12 years ago

That's not correct. rayo-server is doing that butchering. The SSML doc we send is here: https://gist.github.com/a656815535601c5daed1#file_wire.xml

I suspect rayo-server is expecting this to be CDATA, no? On the other hand, a very similar SSML doc (as can be seen in the gist) works, where the only difference is a valid URI in the <audio/> tag.

mpermar commented 12 years ago

Right. mmm. Interesting. The Prism log you got there in the gist is a bit incomplete. I'd like to see a few lines before of what is in there to check what is really entering into Prism. Is there any chance to upload that?

benlangfeld commented 12 years ago

Sure, I'll run a fresh log.

benlangfeld commented 12 years ago

Fresh logs from both Adhearsion and PRISM showing several scenarios: https://gist.github.com/2133592

mpermar commented 12 years ago

This is fixed now. I already mentioned on the chat but keeping it here for the record. The command you are sending it is going to fail anyways as the spec specifies that whatever is within src element it has to be a resource URI.

So the problem is that even though this bug is fixed, when you send the

`

Could not find the Resource's URI 
 </complete>`
benlangfeld commented 12 years ago

Lovely, that error is a lot more useful. Thanks Martin.