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

Execute path was not escaped on Linux #78

Open
ansidev opened this issue Mar 21, 2018 · 1 comment · Fixed by #126
Open

Execute path was not escaped on Linux #78

ansidev opened this issue Mar 21, 2018 · 1 comment · Fixed by #126
Assignees
Milestone

Comments

@ansidev
Copy link

ansidev commented Mar 21, 2018

  • The target platform(s) the problem occurs on: Linux
  • Node version (run node -v): 8.10.0
  • auto-launch version: 5.0.5
  • The options i'm passing to auto-launch:
var autoLauncher = new AutoLaunch({
    name: 'Name with space',
});
  • Electron 1.8.4
  • Main process

If I pass value for option name in AutoLaunch object which has whitespace, the execute path in .desktop file will not be escaped.
Value: Name with space
Expected execute path: /opt/Name\ with\ space/app
Actual execute path: /opt/Name with space/app (missing \ to escape whitespace)

@ansidev ansidev changed the title Execute path was not escaped Execute path was not escaped on Linux Mar 21, 2018
@Izurii
Copy link

Izurii commented May 20, 2021

As the originals developers of the package is not maintaining this project anymore, I replicated and will be taking over, if you could take a look at it I'll be happy. This issue is already solved in my version of the project.

https://github.com/Izurii/easy-auto-launch

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