search

UnifiedPush / common-proxies

Mirror of https://codeberg.org/UnifiedPush/common-proxies
MIT License
24 stars 11 forks source link

Handle FCMv1 responses #69

Closed moffatman closed 2 months ago

moffatman commented 2 months ago

They have a different shape. Notifications did go through before this commit, but the system calling common-proxies always got a 502 code.

Since the old API is gonna be turned off in a few days; I didn't handle both; just the new one.

moffatman commented 2 months ago

@p1gp1g @karmanyaahm

p1gp1g commented 2 months ago

Thanks a lot. Is everything correctly working after this commit for you ?

moffatman commented 2 months ago

On my dev machine yeah. But I haven't built amd64 Linux yet to try on my actual app. Was waiting for your release as too lazy to set up "go" build system for Linux.

p1gp1g commented 2 months ago

I've tested, it works great. Thanks again.

We should probably update the doc to help to set this up. For those looking for the right credentials file:

  1. Go to your firebase console, then to IAM-Admin > Service accounts (https://console.cloud.google.com/iam-admin/serviceaccounts)
  2. Create a new service account with Firebase Messaging API administration rights
  3. Then generate a new key for this account
moffatman commented 2 months ago

Thx. is it possible you can do a release to get the linux build?

p1gp1g commented 2 months ago

Sure. It is building