kamicane / agent Goto Github PK

same as MooTools Request - see mootools/mootools-core#2381

        this._header = {
            "X-Requested-With": "XMLHttpRequest",
            "Content-Type": "application/x-www-form-urlencoded"
        }

several ways this can be handled.

• this should NOT be implicitly set at all and people can manually add via .header;

• an additional sugar method .cors() can be added that removes the header.

• special logic can be added that removes that header when method === 'options'

  let's not release with the same limitations as MooTools - there is no legacy to support and CORS is becoming more commonplace than ever. you can fix it upstream by allowing that header but once again, this should be optional and not enforced.

agent.delete() in IE8 or less throws 'Expected identifier'. Workaround is to use agent('delete' ...) or agent['delete']() but that's somewhat ugly.

suggested to use either delete_ or DELETE or del (as in express middleware)

if the goal is not to care about OLDIE then fair enough - given the decoders for JSON.parse / stringify which expect it to be there or to be polyfilled.

use case: return with the correct application/json content type, then say, do a DELETE. expected: no exception
actual: JSON.parse exception, unexpected end of file.