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

Shellscript won't work with supported shell #4

Closed
3 tasks done
LoliPain opened this issue Jul 11, 2023 · 0 comments
Closed
3 tasks done

Shellscript won't work with supported shell #4

LoliPain opened this issue Jul 11, 2023 · 0 comments
Labels
invalid This doesn't seem right

Comments

@LoliPain
Copy link
Owner

Definitely a ssh_manager issue

There's a non-zero chance of your issue to pend isn't really related to ssh_manager, so please make sure that:

  1. Re-read the README installation and configuration guide and follow all of those steps
  2. Verify that $servername_user environment that should contain the password for the remote machine exists in your current shell session:
    @ echo $servername_user
    (Where servername and user are matching the entry in ssh_manager storage)
  3. Verify sshpass -p direct connection to remote:
    @ sshpass -p $servername_user ssh user@remoteip
    NOTE that user in $servername_user should be exactly the same as user in ssh user@remoteip
  4. Verify python installation itself:
    @ python3 -V
    OR
    @ python3.x -V where's x is your python release

Checklist:

  • My issue is not related to connection problems or I'm aware that my ssh_manager storage and shell environment is fine
    • Direct connection using sshpass -p is performed, but connection through ssh_manager seems inappropriate
  • Broken functionality is declared and documented in the README project description

A detailed explanation of the issue goes here

Screenshots, screencasts, and other useful information are welcome


Some shell are manages to bind itself inside /usr/local/bin/ instead /usr/bin/
So preventing usage of shellscript even with supported shells due to
https://github.com/LoliPain/ssh_manager/blob/92fd48c65939cf1dcb57578ea778ca5c47d9b1d3/main.py#L23C1-L33

Proper solution will be extend those case clause with /usr/local/bin prefixes

@LoliPain LoliPain added the invalid This doesn't seem right label Jul 11, 2023
LoliPain added a commit that referenced this issue Jul 15, 2023
using only name of shell binary will prevent misleading while
verification
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
invalid This doesn't seem right
Projects
None yet
Development

No branches or pull requests

1 participant