{"action":"created","issue":{"url":"https://api.github.com/repos/Financial-Times/polyfill-service/issues/1129","repository_url":"https://api.github.com/repos/Financial-Times/polyfill-service","labels_url":"https://api.github.com/repos/Financial-Times/polyfill-service/issues/1129/labels{/name}","comments_url":"https://api.github.com/repos/Financial-Times/polyfill-service/issues/1129/comments","events_url":"https://api.github.com/repos/Financial-Times/polyfill-service/issues/1129/events","html_url":"https://github.com/Financial-Times/polyfill-service/issues/1129","id":214820968,"number":1129,"title":"Polyfill 3.17.0 service update - Electron detection not working properly","user":{"login":"rubencodes","id":1036549,"avatar_url":"https://avatars2.githubusercontent.com/u/1036549?v=3","gravatar_id":"","url":"https://api.github.com/users/rubencodes","html_url":"https://github.com/rubencodes","followers_url":"https://api.github.com/users/rubencodes/followers","following_url":"https://api.github.com/users/rubencodes/following{/other_user}","gists_url":"https://api.github.com/users/rubencodes/gists{/gist_id}","starred_url":"https://api.github.com/users/rubencodes/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/rubencodes/subscriptions","organizations_url":"https://api.github.com/users/rubencodes/orgs","repos_url":"https://api.github.com/users/rubencodes/repos","events_url":"https://api.github.com/users/rubencodes/events{/privacy}","received_events_url":"https://api.github.com/users/rubencodes/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":4,"created_at":"2017-03-16T19:57:51Z","updated_at":"2017-03-17T06:08:44Z","closed_at":null,"body":"Hello,\r\n\r\nI've moved this into a separate issue from [my initial comment here](https://github.com/Financial-Times/polyfill-service/issues/1121).\r\n\r\nAt some point yesterday, my electron app started having issues with Polyfill.io due to it loading the Element polyfill, as I have my unknown policy set to `polyfill`. While it had worked fine until yesterday, it suddenly started detecting electron as an unknown/unsupported browser. The exact comment at the top is:\r\n\r\n```\r\n/* Polyfill service v3.17.0\r\n * For detailed credits and licence information see https://github.com/financial-times/polyfill-service.\r\n * \r\n * UA detected: electron/1.4.0 (unknown/unsupported; using policy `unknown=polyfill`)\r\n * Features requested: Array.prototype.find,Array.prototype.findIndex,Array.prototype.includes,Date.now,String.prototype.endsWith,String.prototype.includes,String.prototype.startsWith,console.error,console.info,fetch,performance.now\r\n * \r\n * - Object.defineProperty, License: CC0 (required by \"Array.prototype.find\", \"Array.prototype.findIndex\", \"Array.isArray\", \"Promise\", \"fetch\", \"Event\", \"XMLHttpRequest\")\r\n * - Array.prototype.find, License: CC0\r\n * - Array.prototype.findIndex, License: CC0\r\n * - Array.prototype.includes, License: CC0\r\n * - Date.now, License: CC0 (required by \"performance.now\")\r\n * - String.prototype.endsWith, License: CC0\r\n * - String.prototype.includes, License: CC0\r\n * - String.prototype.startsWith, License: CC0\r\n * - Window, License: CC0 (required by \"console.error\", \"console.info\", \"console\", \"console.log\", \"Event\", \"Promise\", \"fetch\", \"XMLHttpRequest\")\r\n * - console, License: CC0 (required by \"console.error\", \"console.info\", \"console.log\")\r\n * - console.log, License: CC0 (required by \"console.error\", \"console.info\")\r\n * - console.error, License: CC0\r\n * - console.info, License: CC0\r\n * - Array.prototype.forEach, License: CC0 (required by \"fetch\")\r\n * - Object.getOwnPropertyNames, License: CC0 (required by \"fetch\")\r\n * - setImmediate, License: CC0 (required by \"Promise\", \"fetch\")\r\n * - Array.isArray, License: CC0 (required by \"Promise\", \"fetch\")\r\n * - Document, License: CC0 (required by \"Event\", \"Promise\", \"fetch\", \"XMLHttpRequest\", \"Element\")\r\n * - Element, License: CC0 (required by \"Event\", \"Promise\", \"fetch\", \"XMLHttpRequest\")\r\n * - Event, License: CC0 (required by \"Promise\", \"fetch\", \"XMLHttpRequest\")\r\n * - Promise, License: MIT (required by \"fetch\")\r\n * - XMLHttpRequest, License: CC0 (required by \"fetch\")\r\n * - fetch, License: MIT\r\n * - performance.now, License: CC0 */\r\n```"},"comment":{"url":"https://api.github.com/repos/Financial-Times/polyfill-service/issues/comments/287275230","html_url":"https://github.com/Financial-Times/polyfill-service/issues/1129#issuecomment-287275230","issue_url":"https://api.github.com/repos/Financial-Times/polyfill-service/issues/1129","id":287275230,"user":{"login":"triblondon","id":1735391,"avatar_url":"https://avatars1.githubusercontent.com/u/1735391?v=3","gravatar_id":"","url":"https://api.github.com/users/triblondon","html_url":"https://github.com/triblondon","followers_url":"https://api.github.com/users/triblondon/followers","following_url":"https://api.github.com/users/triblondon/following{/other_user}","gists_url":"https://api.github.com/users/triblondon/gists{/gist_id}","starred_url":"https://api.github.com/users/triblondon/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/triblondon/subscriptions","organizations_url":"https://api.github.com/users/triblondon/orgs","repos_url":"https://api.github.com/users/triblondon/repos","events_url":"https://api.github.com/users/triblondon/events{/privacy}","received_events_url":"https://api.github.com/users/triblondon/received_events","type":"User","site_admin":false},"created_at":"2017-03-17T06:08:44Z","updated_at":"2017-03-17T06:08:44Z","body":"It looks like if we remove the ` Electron/X.Y.Z` tag, it detects your UAs as Chrome 53.  Would that be appropriate for Electron?"}}
{"actor":{"display_login":"triblondon"}}