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

[minor] Synology prepends backslash (\) to pipe (|) in (command line) password #3135

Closed
ConradHughes opened this issue Mar 30, 2018 · 2 comments

Comments

@ConradHughes
Copy link

commented Mar 30, 2018

  • I have searched open and closed issues for duplicates.

Environment info

  • Duplicati version: 2.0.3.2_canary_2018-03-16
  • Operating system: Synology DSM 6.1.6-15266
  • Backend: Mono 5.8.0.108-11

Description

Specifying a password containing the pipe symbol (|) on the command line using --webservice-password results in an actual password requirement of backslash-pipe (\|). This looks like a shell metacharacter quoting error and may even be a problem with mono-sgen. I cannot replicate this on Debian 9.4 running duplicati 2.0.2.1-1 on mono-runtime 4.6.2.7+dfsg.

Steps to reproduce

  1. Execute (for example) /volume1/@appstore/mono/bin/mono-sgen /volume1/@appstore/Duplicati/Duplicati.Server.exe --webservice-interface=any --webservice-port=8200 --webservice-password='|'
  2. Try to log in to the server using | as password.
    • This won't work.
  3. Try to log in to the server using \| as password.
    • This will work.
  4. Running "ps" will show two mono-sgen processes, the second of which has a "\" before the | in the --webservice-password switch: --webservice-password=\|
  • Actual result: (see above) only \| works as password, while just | was specified.
  • Expected result: (see above) | should work alone as password.

Screenshots

Debug log

@Pectojin Pectojin added the bug label Mar 31, 2018

@arberg

This comment has been minimized.

Copy link

commented Sep 10, 2019

This issue I created (later) is identical (in the underlying cause): #3891

@warwickmm

This comment has been minimized.

Copy link
Contributor

commented Sep 19, 2019

Duplicates #3891.

@warwickmm warwickmm closed this Sep 19, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants
You can’t perform that action at this time.