Kit Cambridge kitcambridge

Organizations

@mozilla @bestiejs @mozilla-services @lodash
@kitcambridge
  • @JamesWCCheng 3e08229
    Bug 975246 - Part1 - Implement operator->* in nsRefPtr. r=nfroyd, r=g…
  • @brendanlong 8be4518
    Bug 909993 - Add WebIDL for TextTrackCue and fix link in VTTCue.webid…
  • 255 more commits »
@kitcambridge
  • @theres-waldo 223b1b2
    Bug 1131359 - Expose a basic FrameMetrics calculations in nsLayoutUti…
  • @aidin36 99b2a37
    Bug 108603 - Remove NS_IMPL_QUERY_INTERFACE_INHERITED0. r=mccr8, r=fr…
  • 150 more commits »
@kitcambridge
  • @pocmo bc93298
    Bug 1184683 - GeckoPreferences: Pass current context to RestrictedPro…
  • @rvandermeulen b642834
    Bug 1148771 - Skip browser_notifications.js on Windows as well due to…
  • 998 more commits »
kitcambridge deleted branch feature/webpush-ttl at mozilla-services/autopush
@kitcambridge
@kitcambridge
Add WebPush TTL's
7 commits with 119 additions and 28 deletions
@kitcambridge
Add a TTL header for incoming messages
@kitcambridge

@bbangert, @jrconlin Pennies for your thoughts? These verbose issues are my attempt at being more vocal. :wink:

@kitcambridge
Add delivery receipts
kitcambridge commented on issue hapijs/hapi#2668
@kitcambridge

:disappointed: Sorry about that; I thought it might be an issue with encoding the binary string into a buffer.

kitcambridge commented on issue hapijs/hapi#2668
@kitcambridge

@maxhedouin Does it help if you set .bytes(Buffer.byteLength(buf))?

kitcambridge commented on pull request mozilla-services/autopush#120
@kitcambridge

LGTM!

kitcambridge commented on pull request mozilla-services/autopush#120
@kitcambridge

Sounds good. Thanks for clarifying!

kitcambridge commented on pull request mozilla-services/autopush#120
@kitcambridge

Nifty.

kitcambridge commented on pull request mozilla-services/autopush#120
@kitcambridge

This is only for Web Push, right? Or are we planning to send encrypted messages via GCM/APNS, too?

kitcambridge commented on pull request mozilla-services/autopush#120
@kitcambridge

Oh, I had no idea DynamoDB limits batches to 25 operations. Could you add a quick comment here, please?

kitcambridge commented on pull request mozilla-services/autopush#120
@kitcambridge

s/Router/Message.

kitcambridge commented on pull request mozilla-services/autopush#120
@kitcambridge

Maybe I'm missing something, but it looks like there's one instance of each router per server (self.ap_settings.routers[router_key]). To make this …

@kitcambridge

Looks great!

@kitcambridge

Could you please add Encryption-Key, too?

kitcambridge commented on issue fgnass/domino#53
@kitcambridge

Possibly related to #27?

@kitcambridge

For Web Push, version is always a timestamp, right?

@kitcambridge

Of course we'll have to encode the contents of the header to deal with the double quotes, but essentially we will leave the processing of that fie…

@kitcambridge

Some thoughts on the implementation: Let's echo the use_webpush flag in the hello reply, so the client knows that the server supports Web Push-sty…

@kitcambridge
  • @ehsan b0e48ef
    Bug 1185515 - Don't hold the observer service alive from nsDNSService…
  • ec86960
    Bug 1185686: Fix Atomics.h ifdefs for clang-cl. r=froydnj
  • 228 more commits »
@kitcambridge

I think the client should provision its own "device ID", and to begin we should just use the sync client GUID that we already have in the "service…

@kitcambridge