portfolio icon indicating copy to clipboard operation
portfolio copied to clipboard

Path/filename CSV export not sanitized properly

Open Eegee opened this issue 5 months ago • 0 comments

Describe the bug When exporting transactions to CSV for Konten/deposit accounts, the filename does not seem to be properly sanitized. I noticed this because one deposit account name has a forward slash in it and the export failed because of this. The forward slash seems to be translated to a backslash and used literally, which should not be done, as it is a name.

To Reproduce

  1. Have/create a deposit account with a name such as Saxo 1234/98765 and make sure it has transactions
  2. Choose File, Export, CSV file
  3. Select top item (deposit) Account transactions and click button Finish
  4. Choose a folder (like D:\New Folder) an click Select folder
  5. Error exporting data occurs, "D:\New Folder\Saxo 1234\98765.csv (the system cannot find the path specified)"

Expected behavior The forward slash should not be interpreted as a path separator. It should be either removed from the filename or replaced by a character that is safe for file paths and file names, such as an underscore or a dash, taking into account the applicable rules of the operating system platform for allowed file and path name characters. All other characters besides forward slash that are not safe for file paths or names should be treated in this same way as well.

Desktop (please complete the following information):

  • OS: Windows 11
  • Version 0.70.4

Eegee avatar Sep 15 '24 10:09 Eegee