Johan Euphrosine wrote: > Loic, > > Thanks for adding jsonp in poker-network 1.4.0. > > Please review the following patch: > http://jpoker.aminche.com/hg/poker-network-jsonp-fixed-and-optional/rev/9fd519905539 > And the following javascript testcase: > http://jpoker.aminche.com/jpoker-jsonp/jpoker/test-jsonp.html > http://jpoker.aminche.com/hg/jpoker-jsonp/file/tip/jpoker/test-jsonp.html > > Rationale: > The postData is empty when inserting a <script src="json-p-url"> > element into the DOM. > Surrounding response JSON data with '()' is not affecting javascript eval. > But non-javascript client will have to remove '()' before parsing JSON > data using a third party library. > > Implementation: > When jsonp is enabled: Data are read from the url, I added > "packet=JSONString" argument. > When jsonp is disabled: the data is no more surrounded with '()'. > > Hope that helps. > It runs for me and does exactly what one would expect. Very good. I'll commit it shortly, after checking that the current 1.4.0 is good to install and run.
Cheers _______________________________________________ Pokersource-users mailing list [email protected] https://mail.gna.org/listinfo/pokersource-users
