Comment 4 for bug 745486

Revision history for this message
Deryck Hodge (deryck) wrote :

Thanks, Huw and Robert, for the explanations. I had a poke at the code yesterday myself just because my expectation would have been the Y.io would have provided this for you. It's odd, to me, that it doesn't. While I think the header idea is fine and useful beyond this particular use, I may have a play this weekend at adding the request URL to all the Y.io event hooks. Seems generally useful. We'll see if YUI takes it upstream, and if they are warm to the idea, we can merge that into our YUI.

Also, I noticed the arguments array was being overwritten in one of the callbacks, so I did a quick branch to fix this up. https://code.launchpad.net/~deryck/launchpad/xhr-log-not-overwrite-arguments/+merge/55750

Thanks for this work, Huw. It's going to be quite useful when generally available.

Cheers,
deryck