Skip to content

Adding timestamp crashes program because of slashes #49

@Adam-Kay

Description

@Adam-Kay

Bug Description

When the proposed output file already exists, the program attempts to append a timestamp to the filename to ensure its uniquity. Due to the locale on many systems, this date ends up being formatted as dd/mm/yyyy hh:mm:ss . The / character is illegal in file paths so the following encode fails.

Steps to Reproduce

  1. Ensure a file already exists with the name that the batch encode would spit out. Usually <filename>-ENC.<ext>
  2. Attempt to run the encoder
  3. Program reports that a file already exists with this name and it will append a timestamp to mitigate filename collisions.
  4. The program crashes.
    [[[include screenshot here]]]

Potential Cause

The timestamp being locale dependent. If possible, enforce the format to be something like yyyy-mm-dd_hh-mm-ss.

Metadata

Metadata

Assignees

No one assigned

    Labels

    bugSomething isn't working

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions