Script ends with cancelled by user

[tavinus]

Script ends with cancelled by user

[tavinus]

Hi again! Just made a test piping it to sh and I got the same problem.
I am not sure why it does not ask when piping the script.
I am also not sure how to pass a parameter when running like that (you could tell it to install without asking with that).

I would ask you to just download the script and then run it locally instead of directly piping it from curl or wget. So, you can use curl or wget to download as described on the readme, or even use git to clone the repo. Then you can just call it with or without parameters.

If you still want to download and run at once, I would recommend using something like this:

curl -k -sSL https://raw.githubusercontent.com/tavinus/opkg-upgrade/master/opkg-upgrade.sh -o ./opkg-upgrade.sh && chmod +x ./opkg-upgrade.sh && ./opkg-upgrade.sh

This will download, set to executable and run (on the current directory).
Tested and working!
Hope this helps!

Cheers!
Gus

[tavinus]

Hi! sorry for the late response!

Humm...
I never really tried to run it "directly from the internet".
Does the same thing happen if u try to run it locally?
I mean if u download first and then try to run it locally?

You should be asked if you want to install the updates at that point.
I will try to reproduce it here soon.

[tavinus]

Ok, so this seems to be normal to happen in this case.
References:

• https://www.google.com.br/search?q=read+does+not+work+when+piping+to+sh&rlz=1C1CHZL_pt-BRBR714BR714&oq=read+does+not+work+when+piping+to+sh&aqs=chrome..69i57.9535j0j7&sourceid=chrome&ie=UTF-8
• https://stackoverflow.com/questions/16854041/bash-read-is-being-skipped-when-run-from-curl-pipe
• https://stackoverflow.com/questions/2746553/bash-script-read-values-from-stdin-pipe
• https://stackoverflow.com/questions/19408649/pipe-input-into-a-script

Because there is not stdin for read to process the response when run like that.
Tell me what you think.

[sml156]

Thanks for your response, I have not tried to run it locally but I will try the script you just posted thank you.

Update:
I tried the command you posted above and it worked perfectly Thank you

[tavinus]

Sweet!
Thanks for posting, good to know it does not work like that anyways.
Cheers!

[sml156]

I will read the links you posted soon.

I was hoping it would work the same as it does in Ubuntu, I use a DNS server called Pihole and that's how it's installed.

https://github.com/pi-hole/pi-hole

[sml156]

You should also put that command on your Readme page as long as people have curl installed it works great.

opkg update

opkg install curl

curl -k -sSL https://raw.githubusercontent.com/tavinus/opkg-upgrade/master/opkg-upgrade.sh -o ./opkg-upgrade.sh && chmod +x ./opkg-upgrade.sh && ./opkg-upgrade.sh

DONE

[tavinus]

As far as I can tell the problem is only with read, which is used to ask for permission to install the updates. When you pipe it to sh another shell is created and stdin is not "redirected" so there is no response (or at least no Y/y response) on which case the script thinks something other than Y was the response.

The download call is basically the same I have on the readme, but without using variables and this one runs the script after downloading. I think it is less intrusive that way, but who knows. The better way is always to use git in any case, but not really needed.

I also have an explanation for the curl -k option to ignore SSL certificates and all there. I believe this should not be default as many people may not realize there is a security risk there.

Cheers!
Gus

[tavinus]

Just a side note. This would also happen in any other linux distro (including ubuntu). Pihole works because it does not ask anything for the user (or he would have the same problem with read not receiving the answer I think). I would need a separate installer script for you to run like that.

Same approach is used by get.acme.sh (LetsEncrypt SH script). He has an installer on http://get.acme.sh which install the acme.sh script itself.

So, in any case, running the installer like that would not run the script itself.

[tavinus]

Also, he uses a variable to pass a parameter, as in:

curl https://raw.githubusercontent.com/Neilpang/acme.sh/master/acme.sh | INSTALLONLINE=1  sh

That INSTALLONLINE=1 there.
But note that he has to do it after the pipe.