notificationclose fired but notificationclick dose not fire

[14kw]

Page with the URL specified by clickaction does not contain firebase-messaging.js.
So I want to obtain log with notificationclick rather than onmessage.
Notificationclose fired, but notificationclick did not fire.

How can I fire notificationclick?

push body

{ "notification": {
    "title": "Background New Topic Title",
    "body": "Background New Topic body",
    "click_action" : "https://google.com"
  },
  "to" : "xxxx"
}

firebase-messaging-sw.js

importScripts('https://www.gstatic.com/firebasejs/3.5.2/firebase-app.js');
importScripts('https://www.gstatic.com/firebasejs/3.5.2/firebase-messaging.js');

firebase.initializeApp({
  'messagingSenderId': 'xxxxx'
});
const messaging = firebase.messaging();

// not fire
self.addEventListener("notificationclick", function(event) {
  console.log('notification open');
  // log send to server
});

// fired
self.addEventListener("notificationclose", function(event) {
  console.log('notification close');
  // log send to server
});

Google Chrome 56.0.2924.87

[14kw]

The issue was resolved.

Reason for not firing:

firebase-messaging.js has the following code.

self.addEventListener("notificationclick", function(event) {
  event.stopImmediatePropagation();
  event.notification.close();

lib nofiticationclick-function was executing stopImmediatePropagation().
my notificationclick-function has been defined since the library was executed.
So my notificationclick-function will not fire.

Resolution:

Define my notificationclick-function before the library is loaded.

self.addEventListener("notificationclick", function(event) {
  console.log('notification open');
  // log send to server
});

importScripts('https://www.gstatic.com/firebasejs/3.5.2/firebase-app.js');
importScripts('https://www.gstatic.com/firebasejs/3.5.2/firebase-messaging.js');

firebase.initializeApp({
  'messagingSenderId': 'xxxxx'
});
const messaging = firebase.messaging();

[heygambo]

@14kw I was trying to figure this out the whole day and I was about to give up!
Thx for sharing your solution!

[sibelius]

should this be considered a bug?

[pebojote]

That gives me an idea

[prathapGunasekaraAscentic]

@14kw You are lifesaver

[sunderls]

oh my god thanks for this...

[sniffymon]

Damn, 5 hours wasted and it could've been fixed by moving the import scripts. -_-

[ghost]

Still doesn't work for me :(

[muhammadaqibmasood]

still notworking forme

[Transporterapps]

Worked for me, Thanks for the Solution, You are a life saver

[bashoogzaad]

Thank you very much! Just like the other people this has cost me many hours. I think Firebase should document this behaviour.

[guimap]

Doesn't worked for me, anyone can tell me which version this approach works ?

[dikshatickoo1991]

Hi, It is not working for me. Before importing the scripts I used self.addEventListener("notificationclick", function(event) {
console.log('notification open');
// log send to server
}); But log statement is not printing

[davbrito]

Currently the SDK sends the case of the notification click to the window (https://github.com/firebase/firebase-js-sdk/blob/15addde0b9d56dd3490e9c3c1e2f015e10e156cb/packages/messaging/src/listeners/sw-listeners.ts#L162)
But, on the window side it doesn´t handle the case.
(https://github.com/firebase/firebase-js-sdk/blob/15addde0b9d56dd3490e9c3c1e2f015e10e156cb/packages/messaging/src/listeners/window-listener.ts#L29-L57)

One could do that on the userland but the library stops the propagation of the notificationclick event.

I think they should add a handler like "onNotificationClick" so the user could handle easily the click on the window side.