Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Pinned webpage causes issue #353

Closed
Roterfux opened this issue Jun 5, 2023 · 8 comments
Closed

Pinned webpage causes issue #353

Roterfux opened this issue Jun 5, 2023 · 8 comments
Labels
bug Something isn't working

Comments

@Roterfux
Copy link

Roterfux commented Jun 5, 2023

I tried to create a pwa for Outlook.com, and I had the tab pinned.
If you try to create a PWA then, it causes "An unexpected error occurred".
It's hard to see if there is an error in the console, since it's flooded with errors from Outlook itself.

fyi: It works, if the tab is not pinned.

@Roterfux Roterfux added the bug Something isn't working label Jun 5, 2023
@filips123
Copy link
Owner

Does it work with other websites if they are pinned, and does it work with Outlook.com if it's not pinned? Because I've tested this with other websites and they work fine even if pinned, but when I tested it with Outlook.com, it didn't work even when it was unpinned.

I've checked the manifest for Outlook.com, and it seems it has malformed icon URLs (at least for some non-English variants), which PWAsForFirefox cannot process:
slika

This seems to be a problem with Outlook.com. However, I'll also try to improve error handling in cases of malformed URLs.

@Roterfux
Copy link
Author

Roterfux commented Jun 5, 2023

It does work for me, if Outlook.com is NOT pinned. (I do use it like this)

For testing, I tried to use https://roadmap.sh/ and https://dev.azure.com/ - both work pinned as well.

@filips123
Copy link
Owner

Please create a DEBUG file as described here. Then, try to install Outlook.com when pinned and upload firefoxpwa.log, firefoxpwa-stdout.log and firefoxpwa-stderr.log.

@Roterfux
Copy link
Author

Roterfux commented Jun 12, 2023

Sorry for the late answer:

firefoxpwa-stderr.log
firefoxpwa-stdout.log
firefoxpwa.log

Console log:
Fehler beim Verarbeiten des Wertes für 'white-space'. Deklaration ignoriert. 2 manage.html GET https://duerr5.bnapp02.kwp-online.de/relay/duerr5/pwa/bnapp/assets/icons/icon-72x72.png [HTTP/3 200 OK 0ms] GET https://ssl.gstatic.com/docs/spreadsheets/favicon3.ico [HTTP/3 200 OK 0ms] GET https://statics.teams.cdn.office.net/hashed/favicon/prod/favicon-f1722d9.ico [HTTP/2 200 OK 0ms] Fehler beim Verarbeiten des Wertes für 'white-space'. Deklaration ignoriert. 2 install.html XHRGET https://outlook.office.com/mail/manifests/pwa.json?culture=de [HTTP/3 200 OK 73ms] Uncaught (in promise) Error: An unexpected error occurred undefined

Uncaught (in promise) Error: An unexpected error occurred undefined onclick moz-extension://6ddcc6d9-3c6d-4ccc-99cb-05932951cbfc/sites/install.bf5555d0.js:1 AsyncFunctionThrow self-hosted:760 (Async: async) <anonym> moz-extension://6ddcc6d9-3c6d-4ccc-99cb-05932951cbfc/sites/install.bf5555d0.js:1 AsyncFunctionNext self-hosted:756 (Async: async) <anonym> moz-extension://6ddcc6d9-3c6d-4ccc-99cb-05932951cbfc/sites/install.bf5555d0.js:1

@filips123
Copy link
Owner

It seems to be the same issue as I'm having (with broken URLs), but I don't know why it only happens to you when the tab is pinned. So, I guess you can just unpin it, install the PWA and pin it again? Alternatively, you can uncheck "use the manifest" when installing the PWA (although this will mean menu entry shortcuts and few other similar features won't work).

The actual issue with invalid URLs that cause panic is because of a library that has this bug (seanmonstar/reqwest#668, seanmonstar/reqwest#1399). I probably cannot fix this myself.

@Roterfux
Copy link
Author

Roterfux commented Jun 14, 2023

That is how I did it. Unpin, install and pin again (or used the installed version) as a workaround.
Just noticed that issue as a weird behaviour

@filips123
Copy link
Owner

Perhaps Outlook.com is doing some testing by serving different manifests, and one of them is corrupted.

So, as this seems to be an issue with website and library (reqwest) that I cannot fix, I'll close this issue. Probably Microsoft will fix at some point, and I also hope reqwest fill fix that bug in the future.

I may also add this to the documentation when I finish it.

@Roterfux
Copy link
Author

Ok

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants