{"action":"created","issue":{"url":"https://api.github.com/repos/Leaflet/Leaflet.VectorGrid/issues/136","repository_url":"https://api.github.com/repos/Leaflet/Leaflet.VectorGrid","labels_url":"https://api.github.com/repos/Leaflet/Leaflet.VectorGrid/issues/136/labels{/name}","comments_url":"https://api.github.com/repos/Leaflet/Leaflet.VectorGrid/issues/136/comments","events_url":"https://api.github.com/repos/Leaflet/Leaflet.VectorGrid/issues/136/events","html_url":"https://github.com/Leaflet/Leaflet.VectorGrid/issues/136","id":284516405,"number":136,"title":"Cancelling requests","user":{"login":"furstenheim","id":8070719,"avatar_url":"https://avatars1.githubusercontent.com/u/8070719?v=4","gravatar_id":"","url":"https://api.github.com/users/furstenheim","html_url":"https://github.com/furstenheim","followers_url":"https://api.github.com/users/furstenheim/followers","following_url":"https://api.github.com/users/furstenheim/following{/other_user}","gists_url":"https://api.github.com/users/furstenheim/gists{/gist_id}","starred_url":"https://api.github.com/users/furstenheim/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/furstenheim/subscriptions","organizations_url":"https://api.github.com/users/furstenheim/orgs","repos_url":"https://api.github.com/users/furstenheim/repos","events_url":"https://api.github.com/users/furstenheim/events{/privacy}","received_events_url":"https://api.github.com/users/furstenheim/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":7,"created_at":"2017-12-26T08:27:29Z","updated_at":"2018-03-13T10:48:02Z","closed_at":null,"author_association":"NONE","body":"I've noticed that when using protobuf and changing the zoom, old requests aren't cancelled. This is sometimes clogging the server if the tiles take too long and the user changes zoom quick.\r\n\r\nIf it is of any help, in leaflet tile layers this is handled by changing the src of the img and that makes the browser cancel the old request https://github.com/Leaflet/Leaflet/blob/master/src/layer/tile/TileLayer.js#L230\r\n\r\nOf course that won't work out here, since you are doing fetch directly.\r\n\r\nThanks"},"comment":{"url":"https://api.github.com/repos/Leaflet/Leaflet.VectorGrid/issues/comments/372623723","html_url":"https://github.com/Leaflet/Leaflet.VectorGrid/issues/136#issuecomment-372623723","issue_url":"https://api.github.com/repos/Leaflet/Leaflet.VectorGrid/issues/136","id":372623723,"user":{"login":"tomchadwin","id":7840213,"avatar_url":"https://avatars0.githubusercontent.com/u/7840213?v=4","gravatar_id":"","url":"https://api.github.com/users/tomchadwin","html_url":"https://github.com/tomchadwin","followers_url":"https://api.github.com/users/tomchadwin/followers","following_url":"https://api.github.com/users/tomchadwin/following{/other_user}","gists_url":"https://api.github.com/users/tomchadwin/gists{/gist_id}","starred_url":"https://api.github.com/users/tomchadwin/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/tomchadwin/subscriptions","organizations_url":"https://api.github.com/users/tomchadwin/orgs","repos_url":"https://api.github.com/users/tomchadwin/repos","events_url":"https://api.github.com/users/tomchadwin/events{/privacy}","received_events_url":"https://api.github.com/users/tomchadwin/received_events","type":"User","site_admin":false},"created_at":"2018-03-13T10:48:02Z","updated_at":"2018-03-13T10:48:02Z","author_association":"COLLABORATOR","body":"> This \"polyfill\" doesn't actually close the connection when the request is aborted, but it will call .catch() with err.name == 'AbortError' instead of .then().\r\n\r\nWill this catch anything beyond what you did in #151? I agree that we should implement `abort` in time, but is my understanding that the issue isn't just browser support, but also support at the API end? Without the latter, I'm not sure how much this improves things, and I have no idea how common API support for `abort` is. If it's there, it's definitely worth doing."}}
{"actor":{"display_login":"tomchadwin"}}