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

[warn] background/export: exportSessions() Error: filename must not contain illegal characters (aka "no startup backup") #771

Closed
CircleCode opened this issue May 10, 2021 · 10 comments

Comments

@CircleCode
Copy link

Short description

until version 6.7.0, startup backup were working as expected
starting from version 6.7.0, startup backup are no more saved.

(perhaps) useful logs

I get the following logs after startup:

[debug] background/backup: backupAllSessions
[debug] background/export: exportSessions() null TSM true
[debug] background/getSessions: getSessions() null null
[debug] background/sessions: getAll() null
[debug] background/sessions: =>getAll() 
Array(31) [ {…}, {…}, {…}, {…}, {…}, {…}, {…}, {…}, {…}, {…}, … ]
[warn] background/export: exportSessions() Error: filename must not contain illegal characters

But since download path is not logged, I am not able to check further.

I could eventually try to run a local version of the extension to use the debugger and see if I can find more clues.

Platform information

  • Platform (OS): arch linux
  • Version of browser: firefox nightly 90.0a1 (2021-05-10) (64-bit)
  • Version of Tab Session Manager: 6.8.0
@sienori
Copy link
Owner

sienori commented May 10, 2021

What is the language of Firefox?

@CircleCode
Copy link
Author

firefox is in english, but the OS is in french

@CircleCode
Copy link
Author

I could eventually try to run a local version of the extension to use the debugger and see if I can find more clues.

I was trying to load TSM as temporary addon, but since this is triggered at startup, I do not know how to reproduce the issue in a temporary addon…

@sienori
Copy link
Owner

sienori commented May 10, 2021

What is the date format in TSM settings?
image

Also, does the session export work?
image

I was trying to load TSM as temporary addon, but since this is triggered at startup, I do not know how to reproduce the issue in a temporary addon…

To install it as a persistent add-on:

  1. Run npm run build
  2. Rename dist/tab_session_manager-for-firefox-6.8.0.zip to dist/tab_session_manager-for-firefox-6.8.0.xpi
  3. Drag and drop it into firefox

@CircleCode
Copy link
Author

What is the date format in TSM settings?

date format is YYYY.MM.DD HH:mm:ss

Also, does the session export work?

it works both in /tmp and in /home/me/Téléchargements/TabSessionManager - Backup

To install it as a persistent add-on…

Will try this

@CircleCode
Copy link
Author

To install it as a persistent add-on…

Is there a way to build the extension not minified?

@sienori
Copy link
Owner

sienori commented May 10, 2021

Is there a way to build the extension not minified?

Run npm run watch-dev and zip contents of dev/firefox/ folder and make it .xpi.

@CircleCode
Copy link
Author

CircleCode commented May 10, 2021

Here is the faulty name: TabSessionManager - Backup2\Auto Save - Browser exited\2021-05-10 16-43-18 - Debugging - Runtime - this-firefox #browserExit #debug - [3a32d7bc-b57c-4776-ab8f-8321a8cc4e5f].json

I suspected either # or […], but in fact, the problem comes from the path delimiter (\): TabSessionManager - Backup2/Auto Save - Browser exited/2021-05-10 17-06-03 - Add-ons Manager #browserExit #debug - [239fca16-dfbb-4db1-aea1-21a1e3d891bc].json works

@sienori sienori added the bug label May 10, 2021
@sienori
Copy link
Owner

sienori commented May 10, 2021

Thanks for the research! I've fixed it. 267b999

@CircleCode
Copy link
Author

Thanks for the quick fix!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants