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

sed: -e expression #1, char 17: unterminated `s' command --- when SFTP enabled #149

Open
jcn50 opened this issue Feb 17, 2025 · 3 comments

Comments

@jcn50
Copy link

jcn50 commented Feb 17, 2025

Can't install:

2025-02-17 18:22:33,946: DEBUG - ++ sort --unique
2025-02-17 18:22:33,947: DEBUG - ++ grep -oP '__[A-Z0-9]+?[A-Z0-9_]*?[A-Z0-9]*?__' /var/www/my_webapp/www/index.html
2025-02-17 18:22:33,950: DEBUG - + uniques_vars=('app' 'domain' 'id' 'ssh_port')
2025-02-17 18:22:33,957: WARNING - sed: -e expression #1, char 17: unterminated `s' command
2025-02-17 18:22:34,563: ERROR - Unable to install my_webapp: An error occurred inside the app installation script

On YunoHost 12.0.11 (stable).

@jcn50
Copy link
Author

jcn50 commented Feb 21, 2025

I found out that only when I enable the SFTP access, the installation fails...

@jcn50 jcn50 changed the title sed: -e expression #1, char 17: unterminated `s' command sed: -e expression #1, char 17: unterminated `s' command --- when SFTP enabled Feb 21, 2025
@jcn50
Copy link
Author

jcn50 commented Feb 21, 2025

This is the log when I try to enable SFTP after an installation without SFTP (so it cannot be added postinstall):

2025-02-21 18:51:40,471: INFO - [##########+++++.....] > Validating the new configuration...
2025-02-21 18:51:40,475: DEBUG - + for short_setting in "${!old[@]}"
2025-02-21 18:51:40,475: DEBUG - + [[ true == \f\a\l\s\e ]]
2025-02-21 18:51:40,475: DEBUG - + local result=
2025-02-21 18:51:40,475: DEBUG - + grep -q '^function$'
2025-02-21 18:51:40,475: DEBUG - + type -t validate__with_sftp
2025-02-21 18:51:40,476: DEBUG - + [[ settings == *\(* ]]
2025-02-21 18:51:40,476: DEBUG - + '[' -n '' ']'
2025-02-21 18:51:40,476: DEBUG - + for short_setting in "${!old[@]}"
2025-02-21 18:51:40,476: DEBUG - + [[ false == \f\a\l\s\e ]]
2025-02-21 18:51:40,476: DEBUG - + continue
2025-02-21 18:51:40,476: DEBUG - + for short_setting in "${!old[@]}"
2025-02-21 18:51:40,476: DEBUG - + [[ true == \f\a\l\s\e ]]
2025-02-21 18:51:40,476: DEBUG - + local result=
2025-02-21 18:51:40,476: DEBUG - + grep -q '^function$'
2025-02-21 18:51:40,478: DEBUG - + type -t validate__password
2025-02-21 18:51:40,479: DEBUG - + [[ settings == *\(* ]]
2025-02-21 18:51:40,479: DEBUG - + '[' -n '' ']'
2025-02-21 18:51:40,479: DEBUG - + for short_setting in "${!old[@]}"
2025-02-21 18:51:40,479: DEBUG - + [[ false == \f\a\l\s\e ]]
2025-02-21 18:51:40,479: DEBUG - + continue
2025-02-21 18:51:40,479: DEBUG - + [[ true == \f\a\l\s\e ]]
2025-02-21 18:51:40,479: DEBUG - + '[' true == true ']'
2025-02-21 18:51:40,479: DEBUG - + '[' '**********' == '' ']'
2025-02-21 18:51:40,479: DEBUG - ++ ynh_exit_properly
2025-02-21 18:51:40,479: DEBUG - ++ [[ '' =~ ^install$|^upgrade$|^restore$ ]]
2025-02-21 18:51:40,480: WARNING - ./config: line 94: changed[fpm_usage]: unbound variable
2025-02-21 18:51:40,982: DEBUG - ++ exit 1

It seems that the PHP version change fields in the form are missing too~ as well as the DB.

@jcn50
Copy link
Author

jcn50 commented Feb 21, 2025

Similar to #144 but for the modification postinstall.

I think this APP is dead since SEP 2024.....

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

No branches or pull requests

1 participant