FR: Focus tab of the same origin (but not with the same url)

[Oupsla]

[REQUIRED] Describe your environment

• Operating System version: Ubuntu 16.04 / Chrome 63 / Node 8.9
• Firebase SDK version: 4.8.1
• Firebase Product: messaging

[REQUIRED] Describe the problem

When receiving a push message with a click_action in the notification payload, the service-worker tries to find a tab with the same url than the click_action.

If it succeeds, the service worker focus this tab.
If it fails, the service worker opens a new tab (or window).

But the url of the tab has to be exactly the same url than the click_action.
If a tab with the same origin than the click_action but with a different url (a different page, for my case in a one page application with Polymer) is found, the service worker will open a new tab instead of focusing this tab and send a message to it (to tell the application to change the page for example).

Steps to reproduce:

1. Integrate the firebase-js-sdk into an application
   1.b. Navigate into the application
2. Send a push notification to a client (with a click_action in the payload)
3. Click on this notification
4. See that will open a new tab (or window)

Relevant code :

firebase-js-sdk/packages/messaging/src/controllers/sw-controller.ts

Line 265 in 7db74e6

getWindowClient_(url) {

Example

Notification payload:
{ notification: { 'title': 'Title', 'body': 'Message body', 'click_action': 'https://example.firebaseapp.com/demo/commands/' } }
Url of the inactive tab : https://example.firebaseapp.com/demo

[karooolis]

I would like to take this on as it's currently affecting our project forcing to workaround it by making use of setBackgroundMessageHandler and custom notificationclick event handler. Is this worth working on or there is no intention of doing any changes here, at least as of yet?

[VapiSoft]

Did you find any solution to this huge problem?

[Oupsla]

I think it could be easily resolved, instead of the equal operator (here), we could use an include instead

I will make a PR with that 🙂

----- Edit

Done 😄

[VapiSoft]

Hi Nicolas, It sounds great, but where is this file located? Shlomo Aran 054-7399-340

…

On Fri, Mar 20, 2020 at 11:20 AM Nicolas Delperdange < ***@***.***> wrote: I think it could be easily resolved, instead of the equal operator (here <https://github.com/firebase/firebase-js-sdk/blob/master/packages/messaging/src/controllers/sw-controller.ts#L282>), we could use an include instead I will make a PR with that 🙂 — You are receiving this because you commented. Reply to this email directly, view it on GitHub <#390 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/ABLKC3ISCBU56SNGQZRX6EDRIMYODANCNFSM4EI5OVVQ> .

[Oupsla]

I have made the pull request, waiting for maintainers to review 🙂

[VapiSoft]

Hi Nicolas, Now I realize that you are the firebase programmer. So, 1. this really a great product! 2. This problem is huge, so if you can solve it, it will make it much better. Please do it as soon as you can, and let me know when I can use it. Thanks, Shlomo; Shlomo Aran 054-7399-340

…

On Fri, Mar 20, 2020 at 11:20 AM Nicolas Delperdange < ***@***.***> wrote: I think it could be easily resolved, instead of the equal operator (here <https://github.com/firebase/firebase-js-sdk/blob/master/packages/messaging/src/controllers/sw-controller.ts#L282>), we could use an include instead I will make a PR with that 🙂 — You are receiving this because you commented. Reply to this email directly, view it on GitHub <#390 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/ABLKC3ISCBU56SNGQZRX6EDRIMYODANCNFSM4EI5OVVQ> .

[VapiSoft]

Thank you so much. Shlomo Aran 054-7399-340

…

On Fri, Mar 20, 2020 at 11:37 AM Nicolas Delperdange < ***@***.***> wrote: I have made the pull request, waiting for maintainers to review 🙂 — You are receiving this because you commented. Reply to this email directly, view it on GitHub <#390 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/ABLKC3PFA52GW44T2RBC5Y3RIM2NTANCNFSM4EI5OVVQ> .