webrecorder / webrecorder-player

Webrecorder Player for Desktop (OSX/Windows/Linux). (Built with Electron + Webrecorder)
Apache License 2.0
426 stars 39 forks source link

Release 1.0.3 Windows doesn't load bookmarks #26

Closed yhancik closed 5 years ago

yhancik commented 7 years ago

When I run webrecorderplayer-electron-1.0.3.exe and select an archive to open, it get an empty list ("No bookmarks available in the table" etc).

I noticed that this exe decompresses the actual app folder in %USERPROFILE%\AppData\Local\Temp Running directly the "Webrecorder Player.exe" found there (among these) works as expected.

Disabling the firewall made no difference.

I wonder if it wouldn't make sense to distribute the Windows version as a zipped folder. Running Webrecorder Player.exe directly without having webrecorderplayer-electron-1.0.3.exe decompress stuff also seems much faster ;)

ikreymer commented 6 years ago

We've just released 1.0.6, https://github.com/webrecorder/webrecorderplayer-electron/releases which should at least avoid the firewall prompt.

Perhaps the bookmarks issue is resolved now, although the bookmarks detection is dependent on the WARC containing HTML pages (if its not a Webrecorder WARC). If the WARC does not have HTML pages, there won't be any bookmarks.

As for the packaging, we're using the default Windows distribution with electron: https://www.electron.build/configuration/nsis I'm not sure that there's a zip option and hesitate to do something custom here..

yhancik commented 6 years ago

Hi, thanks for the update! But the issue on my end is still the same.

I don't think the trouble comes from the WARCs, since I can open them properly with the right .exe (and they actually come from webrecorder.io).

If I run webrecorderplayer-electron-x86_64-1.0.6.exe, it unzips this https://imgur.com/IT0ui9M somewhere in Windows' temp folders and launches Webrecorder Player.exe. Archives loaded from this instance don't show any content (not even a file size). I've tried keeping a copy of the temp folder and run it again from there. Same result: not working.

What has worked for me so far is to run Webrecorder Player.exe from another location (I realise now I didn't mention that in my first message). Then, the archives content show.

So I assume the trouble is running Webrecorder Player.exe from Windows' temp folders. I'm not familiar enough with Electron, but I see Electron Builder also has a "portable" export. That sounds like a solution to easily run Webrecorder from any location not in temp ;) https://www.electron.build/ That would also be convenient because the current Windows distribution has to unzip the whole application folder every time (it's deleted from temp when the window is closed), thus much slower to launch.

Is there a way to have the Player output a log to see why it's unable to load an archive when executed from temp?

ikreymer commented 6 years ago

Hi, thanks for the additional feedback, this actually helps a lot!

I'll work on a fix for this issue as well, so that WR Player can open WARCs from any drive.

yhancik commented 6 years ago

Yes, my system (and therefore temp directory) is on C: but my WARCs were indeed located in E: Trying to open a WARC from C: loads everything properly! 👍

ikreymer commented 5 years ago

Opening WARCs from different drives on Windows should now be working in the latest release (added in 1.0.7)