{"issue":{"created_at":"2011-09-15T14:16:00Z","state":"open","updated_at":"2011-11-24T15:47:26Z","user":{"gravatar_id":"c58bc25f46ca2f590c9ef533626a2546","id":1053246,"url":"https://api.github.com/users/kguelzau","login":"kguelzau","avatar_url":"https://secure.gravatar.com/avatar/c58bc25f46ca2f590c9ef533626a2546?d=https://a248.e.akamai.net/assets.github.com%2Fimages%2Fgravatars%2Fgravatar-140.png"},"comments":4,"body":"Timeout handling does not work with \"async: false\".\r\nonTimeout is triggered _always_ after xhr.send() returns.\r\n\r\nHere is a testcase:\r\nhttp://jsfiddle.net/kguelzau/thxA4/\r\n\r\nShould be\r\n\r\n``` js\r\nif (!this.options.async) this.onStateChange();\r\n**else** if (this.options.timeout) this.timer = this.timeout.delay(this.options.timeout, this);\r\n```\r\n","assignee":null,"id":1653303,"pull_request":{"patch_url":null,"html_url":null,"diff_url":null},"number":2073,"labels":[{"name":"docs","color":"e10c02","url":"https://api.github.com/repos/mootools/mootools-core/labels/docs"},{"name":"semantic","color":"e10c02","url":"https://api.github.com/repos/mootools/mootools-core/labels/semantic"}],"title":"disable Request onTimeout with async = false","closed_at":null,"html_url":"https://github.com/mootools/mootools-core/issues/2073","milestone":{"created_at":"2011-09-22T20:22:14Z","state":"open","due_on":"2011-12-01T08:00:00Z","closed_issues":2,"creator":{"gravatar_id":"08ee664ab52153cce96ee552f5c19160","id":27246,"url":"https://api.github.com/users/ibolmo","login":"ibolmo","avatar_url":"https://secure.gravatar.com/avatar/08ee664ab52153cce96ee552f5c19160?d=https://a248.e.akamai.net/assets.github.com%2Fimages%2Fgravatars%2Fgravatar-140.png"},"number":5,"title":"1.4.2","description":"","url":"https://api.github.com/repos/mootools/mootools-core/milestones/5","open_issues":18},"url":"https://api.github.com/repos/mootools/mootools-core/issues/2073"},"action":"created","comment":{"created_at":"2011-11-24T15:47:26Z","updated_at":"2011-11-24T15:47:26Z","user":{"gravatar_id":"08ee664ab52153cce96ee552f5c19160","id":27246,"url":"https://api.github.com/users/ibolmo","login":"ibolmo","avatar_url":"https://secure.gravatar.com/avatar/08ee664ab52153cce96ee552f5c19160?d=https://a248.e.akamai.net/assets.github.com%2Fimages%2Fgravatars%2Fgravatar-140.png"},"body":"@arian @cpojer, we could still reduce headaches if we do as @kguelzau mentioned. To add an `else if`. If you guys agree with my last comment, I don't see a problem of adding that `else if [on this line](https://github.com/mootools/mootools-core/blob/master/Source/Request/Request.js#L215).","id":2866126,"url":"https://api.github.com/repos/mootools/mootools-core/issues/comments/2866126"}}