recap-old icon indicating copy to clipboard operation
recap-old copied to clipboard

escape() calls on URLs sent to /query are probably unnecessary

Open zestyping opened this issue 12 years ago • 0 comments

The extension has code like this:

jsonout.urls.push(escape(docURL));

So each URL first gets URL-escaped, then JSON-escaped, before it goes to the server. It isn't necessary to do both kinds of escaping, as JSON-escaping is sufficient to preserve the string.

zestyping avatar Feb 05 '13 04:02 zestyping