mozilla :: #pageshot

20 Mar 2017
01:29GitHubPS[pageshot] dannycoates commented on issue #2422: Nice catch @niharikak101 :)... https://git.io/vydpo
04:40GitHubPS[pageshot] garbados commented on issue #2404: OK, I restored `shared/shot.js` to match the version on master, and modified `makeUuid.js` to use getRandomValues with the pre-existing 'xxxxx...'.replace approach. How's it look? https://git.io/vyFU6
14:28GitHubPS[pageshot] g-k commented on issue #2301: ulfr pointed out this blog post on idat chunks too: https://www.idontplaydarts.com/2012/06/encoding-web-shells-in-png-idat-chunks/ https://git.io/vyFhD
14:31GitHubPS[pageshot] g-k commented on issue #2219: +1 ... https://git.io/vyFj0
14:38GitHubPS[pageshot] g-k opened issue #2423: Add `object-src 'none'` to CSP to disable plugins https://git.io/vybv6
14:46GitHubPS[pageshot] g-k opened issue #2424: Remove unsafe-inline style-src from CSP https://git.io/vybJZ
14:49GitHubPS[pageshot] g-k opened issue #2425: Add frame-ancestors 'none' to CSP https://git.io/vybJH
14:59GitHubPS[pageshot] g-k opened issue #2426: Remove undefined img-src when contentOrigin isn't set https://git.io/vybTN
15:11GitHubPS[pageshot] g-k opened issue #2427: Add HPKP header https://git.io/vybL2
15:16GitHubPS[pageshot] g-k opened issue #2428: POST /timing returns 500 https://git.io/vybtM
15:30GitHubPS[pageshot] SoftVision-CosminMuntean opened issue #2429: Page Shot button no longer redirects you to the "My Shots" page if it's clicked from Activity Stream New Tab https://git.io/vybYA
15:36GitHubPS[pageshot] g-k opened issue #2430: Use yarn for package management https://git.io/vyb3i
16:15ianbickingCosminMCG: pauloiegasSV: https://github.com/mozilla-services/pageshot/blob/master/docs/acceptance.md
16:15pauloiegasSVthanks ! :)
16:43GitHubPS[pageshot] g-k opened issue #2431: Set regex input matcher lengths https://git.io/vybEs
16:50GitHubPS[pageshot] g-k opened pull request #2432: readme: Add note about supported firefoxes for --setup-profile (master...add-readme-setup-profile-note) https://git.io/vybzX
16:59GitHubPS[pageshot] g-k commented on issue #2431: These are implicitly bound by the max upload size, but we can be stricter. https://git.io/vyb2P
17:36GitHubPS[pageshot] wresuolc commented on issue #2429: @dannycoates has a PR that might fix this https://git.io/vybXU
17:36GitHubPS[pageshot] wresuolc commented on issue #2428: Let's remove /timing https://git.io/vybXO
17:40GitHubPS[pageshot] ckprice commented on issue #2424: Ian thinks we had a reason why this is here. Nice to have == Next https://git.io/vyb1J
17:42GitHubPS[pageshot] ckprice commented on issue #2421: Don't think we can do, but we could ask in #team-addons https://git.io/vyb1C
17:43GitHubPS[pageshot] ckprice commented on issue #2420: Ian noted that having Raven run in the high privileged could be a security risk. https://git.io/vyb1S
17:44GitHubPS[pageshot] johngruen commented on issue #2418: since we're doing this, could we set default search to DDG and about:newtab to the blank screen? https://git.io/vybMe
17:45GitHubPS[pageshot] ckprice commented on issue #2417: @flodolo any advice on how to get this going? Is there pseudo locale support we can use for WE's? https://git.io/vybMJ
17:47GitHubPS[pageshot] flodolo commented on issue #2417: Unfortunately I don't think we have anything for this (in general). ... https://git.io/vybMX
17:52GitHubPS[pageshot] ianb commented on issue #2399: This is done and confirmed https://git.io/vybD6
17:53GitHubPS[pageshot] ianb commented on issue #2398: I'm pretty sure this is done, but it was from a long time ago and double-checking this would be good. https://git.io/vybDS
17:56_6a68MBP is about to die :-(
18:18_6a68Thanks everybody! Sorry that one went over a bit
18:18ianbickingjgruen: strings: https://github.com/mozilla-services/pageshot/blob/master/locales/en-US/webextension.properties
18:18GitHubPS[pageshot] ianb opened issue #2433: Some errors in auth.js should map to LOGIN_ERROR https://git.io/vybQX
18:19GitHubPS[pageshot] ianb opened issue #2434: Confirm error messages in senderror exist https://git.io/vybQp
18:20GitHubPS[pageshot] ianb pushed 2 new commits to master: https://git.io/vyb7f
18:20GitHubPSpageshot/master 489ae40 Greg Guthe: readme: Add note about supported firefoxes for --setup-profile
18:20GitHubPSpageshot/master 93e8344 Ian Bicking: Merge pull request #2432 from g-k/add-readme-setup-profile-note...
18:20ianbickingdcoates: Ill leave https://github.com/mozilla-services/pageshot/pull/2422 for you to review/merge?
18:21dcoates
18:21ianbickingdcoates: and you said https://github.com/mozilla-services/pageshot/pull/2368 is ready for review?
18:22dcoatesyeah, but i can add the activity stream part to it also
18:22ianbickingdcoates: ok, sure, just re-ping me then?
18:22dcoatesok
18:54GitHubPS[pageshot] dannycoates opened issue #2435: Error when trying to screenshot testpilot.firefox.com https://git.io/vybA1
18:55dcoatesianbicking: #2368 is ready again
18:57GitHubPS[pageshot] pdehaan commented on issue #2380: > Is there any way to map shortlog names to GitHub usernames?... https://git.io/vybxT
19:01GitHubPS[pageshot] ianb commented on pull request #2368 4caf735: There might be a whitelist of WebExtension schemes, but I don't know what it is exactly https://git.io/vybxj
19:18_6a68ianbicking: should I just merge https://github.com/mozilla-services/pageshot/pull/2397, or did you want to take another look?
19:20GitHubPS[pageshot] ianb closed pull request #2397: Observe prefs and control webextension startup and shutdown via a pref (master...enabling-pref) https://git.io/vyyK4
19:20GitHubPS[pageshot] ianb pushed 1 new commit to master: https://git.io/vyNeU
19:20GitHubPSpageshot/master cd33c8a Ian Bicking: Merge pull request #2397 from mozilla-services/enabling-pref...
19:20GitHubPS[pageshot] ianb closed issue #2370: Remove deviceInfo pref after migrating https://git.io/vyPWE
19:20GitHubPS[pageshot] ianb deleted enabling-pref at b28d687: https://git.io/vyNek
19:20ianbickingit is so done
19:24_6a68^_^ thx
20:17GitHubPS[pageshot] 6a68 commented on pull request #2413 c53d27c: I guess this is okay: it's not an accurate check for objecthood, but it's unlikely that `null`, an array, or some oddball object like `Math` would ever get passed in here. https://git.io/vyNq1
20:17GitHubPS[pageshot] 6a68 closed issue #2220: Use site-helper to login on page visit https://git.io/vyUEk
20:17GitHubPS[pageshot] 6a68 deleted sitehelper-login at c53d27c: https://git.io/vyNqS
20:20GitHubPS[pageshot] 6a68 commented on issue #2205: I think we have the addon part of localization completely planned out at this point, so I'm going to remove the 'beta blocker' label and tweak the title. https://git.io/vyNmO
20:21557A6UHSP[pageshot] 6a68 closed issue #2205: Plan server localization https://git.io/vDjmS
20:21203A6TJMS[pageshot] 6a68 commented on issue #2205: Hmm. Actually, we have a plan for server localization, too. So, I'm going to go ahead and close this. https://git.io/vyNmW
20:25GitHubPS[pageshot] ianb commented on pull request #2413 c53d27c: The type coercion here is a little sloppy, but objects in particular (as opposed to strings and numbers and null) trigger permission problems when transferring them between the add-on and content. I forgot about null==object though, that's an annoying case, but at least undefined (as when the argument isn't given) will be left alone. https://git.io/vyNYv
20:25ianbicking_6a68: re: auth, pages arent entirely public, when you are the owner youll see a delete button and be able to change the expiration, etc.
20:25_6a68ah
20:27GitHubPS[pageshot] ianb commented on issue #2428: Note the `/timing` route is in `server/src/server.js` https://git.io/vyNYy
20:32GitHubPS[pageshot] 6a68 commented on issue #2435: Content scripts are disabled on Mozilla sites that are `mozAddonManager` whitelisted. See https://bugzil.la/1310082 for related discussion.... https://git.io/vyN3f
20:34GitHubPS[pageshot] 6a68 commented on issue #2295: Got an update on this issue from @johngruen today: looking at next week for final strings. We should be good to start localization after that. https://git.io/vyN3V
20:35GitHubPS[pageshot] johngruen commented on issue #2295: I should have an initial PR tomorrow https://git.io/vyN3y
20:39GitHubPS[pageshot] ianb commented on issue #2435: Probably we should disable Page Shot on these sites.... https://git.io/vyNsc
20:42GitHubPS[pageshot] 6a68 commented on issue #2435: Hmm. We should be able to detect such pages via `'mozAddonManager' in navigator`. If it's true, then bail. https://git.io/vyNsQ
20:43GitHubPS[pageshot] ianb commented on issue #2435: I don't believe we can access navigator from the background page. https://git.io/vyNGL
20:48GitHubPS[pageshot] 6a68 commented on issue #2419: @ianb Hmm, the notification looks a bit odd:... https://git.io/vyNGA
20:49GitHubPS[pageshot] ianb commented on issue #2419: I suppose we could strip `/^error:\s+/i` from the message https://git.io/vyNZL
21:36GitHubPS[pageshot] ianb created pageshot-classname-removal (+22 new commits): https://git.io/vyNBM
21:36GitHubPSpageshot/pageshot-classname-removal 2c1e7a9 Ian Bicking: Change code that reads className to classList...
21:36GitHubPSpageshot/pageshot-classname-removal 0a5e773 Ian Bicking: Remove pageshot-saver from isHeader check...
21:36GitHubPSpageshot/pageshot-classname-removal 586da24 Ian Bicking: Remove references to myshots-reminder, which was removed
21:40GitHubPS[pageshot] ianb pushed 1 new commit to pageshot-classname-removal: https://git.io/vyNRa
21:40GitHubPSpageshot/pageshot-classname-removal 0c78069 Ian Bicking: Rename goto-pageshot metric to goto-homepage...
21:41GitHubPS[pageshot] ianb opened pull request #2436: Pageshot classname removal (master...pageshot-classname-removal) https://git.io/vyNRM
21:42travis-ciThe build passed.: mozilla-services/pageshot#2753 (pageshot-classname-removal - 0716727 : Ian Bicking) https://travis-ci.org/mozilla-services/pageshot/builds/213173032
21:46travis-ciThe build passed.: mozilla-services/pageshot#2754 (pageshot-classname-removal - 0c78069 : Ian Bicking) https://travis-ci.org/mozilla-services/pageshot/builds/213174474
22:00GitHubPS[pageshot] niharikak101 commented on issue #2422: Hey @dannycoates thank you for breaking it down so nicely! :)... https://git.io/vyNu8
22:05GitHubPS[pageshot] g-k opened issue #2437: don't redirect to data: and javascript: protocols/schemes https://git.io/vyNzB
22:57GitHubPS[pageshot] 6a68 commented on issue #2435: Good point!... https://git.io/vyNrK
22:58GitHubPS[pageshot] 6a68 commented on issue #2419: I'm seeing that busted notification in a few other spots, so I"ll just file a bug about it and finish this review https://git.io/vyNr9
23:08GitHubPS[pageshot] g-k opened issue #2438: validate request body objects before parsing as JSON https://git.io/vyNKI
23:12GitHubPS[pageshot] dannycoates commented on issue #2422: > error handling of the text should probably be done before the function is called... https://git.io/vyNKD
23:15GitHubPS[pageshot] dannycoates commented on issue #2422: > error handling of the text should probably be done before the function is called... https://git.io/vyNKD
23:19GitHubPS[pageshot] 6a68 created fix-notifications (+1 new commit): https://git.io/vyN6g
23:19GitHubPSpageshot/fix-notifications 3bf2fe9 Jared Hirsch: Always pass empty strings, not null, to notifications API
23:21GitHubPS[pageshot] 6a68 force-pushed fix-notifications from 3bf2fe9 to 9d739fa: https://git.io/vyN6i
23:21GitHubPSpageshot/fix-notifications 9d739fa Jared Hirsch: Always pass empty strings, not null, to notifications API
23:24GitHubPS[pageshot] ianb opened issue #2439: Create email address to replace pageshot-report https://git.io/vyNif
23:25travis-ciThe build passed.: mozilla-services/pageshot#2756 (fix-notifications - 3bf2fe9 : Jared Hirsch) https://travis-ci.org/mozilla-services/pageshot/builds/213208830
23:26GitHubPS[pageshot] 6a68 force-pushed fix-notifications from 9d739fa to 4f70b6e: https://git.io/vyN6i
23:26GitHubPSpageshot/fix-notifications 4f70b6e Jared Hirsch: Always pass empty strings, not null, to notifications API
23:27travis-ciThe build passed.: mozilla-services/pageshot#2757 (fix-notifications - 9d739fa : Jared Hirsch) https://travis-ci.org/mozilla-services/pageshot/builds/213209155
23:30GitHubPS[pageshot] 6a68 opened pull request #2440: Pass empty strings, not null, to notifications API (master...fix-notifications) https://git.io/vyNip
23:32GitHubPS[pageshot] zelitomas opened issue #2441: UI - Expiration time can't be changed if URL of screenshoted page is too long https://git.io/vyNPT
23:41GitHubPS[pageshot] 6a68 closed pull request #2419: Better server unavailable error (master...better-server-unavailable-error) https://git.io/vy7de
23:41GitHubPS[pageshot] 6a68 closed issue #2305: Confirm errors when server isn't available https://git.io/vyEsK
23:41GitHubPS[pageshot] 6a68 deleted better-server-unavailable-error at dd75b42: https://git.io/vyNXL
23:48GitHubPS[pageshot] dannycoates closed pull request #2440: Pass empty strings, not null, to notifications API (master...fix-notifications) https://git.io/vyNip
23:49GitHubPS[pageshot] 6a68 deleted fix-notifications at 4f70b6e: https://git.io/vyNX1
23:49GitHubPS[pageshot] ianb pushed 12 new commits to pageshot-classname-removal: https://git.io/vyNXy
23:49GitHubPSpageshot/pageshot-classname-removal 80afe21 Ian Bicking: Remove help page that wasn't linked to
23:49GitHubPSpageshot/pageshot-classname-removal e96b86e Ian Bicking: Rename readable/text versions of 'Page Shot' to be 'Firefox Screenshots' or 'Screenshots'
23:49GitHubPSpageshot/pageshot-classname-removal e54b27b Ian Bicking: Rename miscellaneous small instances of 'pageshot'
23:51ianbicking_6a68: oh, just noticed you claimed that for review. I will refrain from pushing more, though thats most of what I planned
23:51ianbickinguntil we rename the repo and id
23:51_6a68cool, do we need to wait to rename the repo?
23:52_6a68and does id refer to the bootstrapped addon id in install.rdf, I'm guessing?
23:52ianbicking_6a68: I want to be sure we rename some public locations once we rename the repo, but thats it
23:52ianbicking_6a68: and yes, the id in install.rdf, and theres a couple places things are derived from that name
23:52_6a68ah. public mowiki pages, stuff like that?
23:53ianbicking_6a68: yeah, at least look around some, though itll redirect so its not that big a deal
23:53_6a68 reviewing and testing that PR now
23:55circleci-botFailed: ianb's build (#150; push) in mozilla-services/pageshot (pageshot-classname-removal) -- https://circleci.com/gh/mozilla-services/pageshot/150?utm_campaign=chatroom-integration&utm_medium=referral&utm_source=irc
23:55travis-ciThe build was broken.: mozilla-services/pageshot#2762 (pageshot-classname-removal - 2bb7b84 : Ian Bicking) https://travis-ci.org/mozilla-services/pageshot/builds/213216910
21 Mar 2017
No messages
   
Last message: 153 days and 22 hours ago