Fix #27: Panic on large notification payloads (revised)

SherClockHolmes / webpush-go

Web Push API Encryption with VAPID support.

MIT License

325 stars 68 forks source link

Fix #27: Panic on large notification payloads (revised) #34

Closed froodian closed 4 years ago

froodian commented 4 years ago

This is a version of https://github.com/SherClockHolmes/webpush-go/pull/33 with my attempt at resolving the pull request conversation - I made it to merge into my fork but exposing here as well to hopefully hasten the day I return to the main fork :)

TwoAbove commented 4 years ago

Hey @SherClockHolmes ! Would be really cool if this be merged.

SherClockHolmes commented 4 years ago

@froodian Sorry for the delay. Everything looks good. Thanks again for contributing.