web-push-libs / web-push-csharp

Web Push library for C#
Mozilla Public License 2.0
429 stars 108 forks source link

support for safari #49

Closed erionalite closed 6 years ago

erionalite commented 6 years ago

I have a hard requirements to implement web push notifications. Do we know if there any blockers to make web push work with safari, and Edge ?

tuwrraphael commented 6 years ago

I tested this package successfully together with Edge. Version Microsoft Edge 42.17134.1.0 Microsoft EdgeHTML 17.17134

aatmmr commented 6 years ago

Web Push Notifications for Safari are a different story compared to the Push API, i.e. Apple does not comply with the Web Push Protocol that this library covers. For info how to make browser push notifications work with Safari (macOS only!) see Apples Developer Docs.

Otherwise, I can confirm that the library works with the supported browsers from ASP.NET Core including Edge. For supporting browsers see Can I Use Push API.

coryjthompson commented 6 years ago

As mentioned unless Safari implement the push api - this library will not work for Safari.

It should however work with Edge and I will update the read me.

Thanks @aatmmr and @tuwrraphael

CiceuBogdan commented 1 year ago

@coryjthompson Hi. Support for Push API has been added on Safari 16.1 Can I Use Push API. I see that in NodeJs library support has been added. Can you reopen this issue and maybe someone will take this in the following days? thanks a lot