schlagmichdoch / PairDrop

PairDrop: Local file sharing in your browser. Inspired by Apple's AirDrop. Fork of Snapdrop.
https://pairdrop.net
GNU General Public License v3.0
3.67k stars 194 forks source link

[Bug] Unable to rename the Pairdrop instance in Firefox for Windows #219

Closed nesper8 closed 6 months ago

nesper8 commented 6 months ago

Describe the bug The user is not able to rename the Pairdrop instance when using Firefox for Windows

To Reproduce

  1. Open Pairdrop on Firefox for Windows
  2. Try renaming the Pairdrop instance
  3. See error

Expected behavior Able to rename the Pairdrop instance

Screenshots https://github.com/schlagmichdoch/PairDrop/assets/57890520/af304b17-8a39-49dc-b2a4-dfff249b7447

Desktop (please complete the following information):

Bug occurs on official PairDrop instance https://pairdrop.net/ Yes Version: v1.9.4

Bug occurs on self-hosted PairDrop instance No

schlagmichdoch commented 6 months ago

Hi @nesper8, Thanks for reporting!

This might be due to a rollback I had to do yesterday as I faced a problem during deployment. I’ll get back to you, when that’s fixed.

What error are you referring too? What does the browser console say?

schlagmichdoch commented 6 months ago

I cannot reproduce this on Firefox for Windows. You simply have to click on the displayed name and edit with your keyboard.

TheNexter commented 6 months ago

Hello men, same problem : Ubuntu 23.10 Firefox 120.0.1

Log in console : Capture d’écran du 2023-12-14 12-27-05

Log in text format : `WS receive: unknown message type Object { type: "ws-config", wsConfig: {…} } network.js:162:25 _onMessage https://pairdrop.net/scripts/network.js:162 onmessage https://pairdrop.net/scripts/network.js:50 (Asynchrone : EventHandlerNonNull) _connect https://pairdrop.net/scripts/network.js:50 ServerConnection https://pairdrop.net/scripts/network.js:18 PairDrop https://pairdrop.net/scripts/ui.js:2792 fire https://pairdrop.net/scripts/network.js:1179 Localization https://pairdrop.net/scripts/localization.js:21 (Asynchrone : promise callback) Localization https://pairdrop.net/scripts/localization.js:19

https://pairdrop.net/scripts/ui.js:2818 ` `Uncaught TypeError: msg.message is undefined _onDisplayName https://pairdrop.net/scripts/network.js:178 _onMessage https://pairdrop.net/scripts/network.js:129 onmessage https://pairdrop.net/scripts/network.js:50 _connect https://pairdrop.net/scripts/network.js:50 ServerConnection https://pairdrop.net/scripts/network.js:18 PairDrop https://pairdrop.net/scripts/ui.js:2792 fire https://pairdrop.net/scripts/network.js:1179 Localization https://pairdrop.net/scripts/localization.js:21 promise callback*Localization https://pairdrop.net/scripts/localization.js:19 https://pairdrop.net/scripts/ui.js:2818 network.js:178:43 `
schlagmichdoch commented 6 months ago

Is this still an issue @nesper8 @TheNexter ?

nesper8 commented 6 months ago

the issue has been fixed i was pre-occupied with testing ROMs for my phone sorry for not closing on time

TheNexter commented 6 months ago

Hello, sorry, yes working great now ;)

schlagmichdoch commented 6 months ago

Great! Thanks for reporting back! :)

schlagmichdoch commented 2 months ago

The following comments were deleted by GitHub (via hubot) as part of mistakenly marking this account as spam on 17th February 2024. The correct thread order and the creation date is unclear. I decided to manually restore them anyway in order to complete the information this issue holds even though the restored information might be outdated:

Comment by @schlagmichdoch:

This might be due to a rollback I had to do yesterday as I faced a problem during deployment. I’ll get back to you, when that’s fixed.

This is since fixed and PairDrop now runs at version v1.10.1 on the server. Do you still have this issue?

Comment by @schlagmichdoch:

Thanks for the logs @TheNexter

This is due to the PairDrop instance running on pairdrop.net has been updated yesterday which now does not work with the old version still running in your browser. To solve this you need to close all open PairDrop tabs. Only then the service worker / browser will update the cached files in the background and everything should work smoothly again.

Could you please try that and report back whether it worked?