change scope to subpath root instead of domain root so PWA will be installable on subpath

[tobycm]

Fixes #5922

[jsjoeio]

Thank you for doing this! Since we don't have e2e tests for this patch, what is your proposed testing plan?

cc @code-asher - mind helping review?

[code-asher]

I might be missing something. This seems to do the opposite of the PR title?

Before changeset: domain.tld/a/b/c/code/ would register at ./ which is domain.tld/a/b/c/code/ (register on subpath root)
After changeset: domain.tld/a/b/c/code would register at / which is domain.tld (register on domain root)

[tobycm]

My bad guys. The service worker works, just the PWA is not installable.

[jsjoeio]

My bad guys. The service worker works, just the PWA is not installable.

Ah got it. And does this fix that? I posted in the original issue that it works as expected on Coder so I think first we'd need steps to reproduce.

[tobycm]

I will reproduce this again when I got my Ubuntu Server up and running.

(don't know why it's failing installing GRUB 5 times in a row 😭 )

[code-asher]

@tobycm I am going to close the PR for inactivity but if you start working on it again we can re-open.