Blink: Siri shortcut for mosh command missing host

Created on 7 Nov 2018  Â·  30Comments  Â·  Source: blinksh/blink

If I use the Siri & Search settings menu to add a Siri command run: ssh mySystem everything executed as expected when the command is given. However if I do the same with mosh run: mosh mySystem then when executed Blink executes mosh without any arguments and displays the help info.

Seen in TestFlight 12.0 build 38

All 30 comments

Hi @Harwood,

I can't reproduce it on tf 12.2 build 52. Does it works for you now?

@yury,

Thanks for getting back to me. I've tested with TestFlight 12.2 build 56 with no success. Here are screenshots of me using "Siri test" to run mosh Pluto.

However, if I do the same using "Siri test" to run ssh Pluto everything executes as expected.

img_3060dcb65d54-1
img_fa21517a5ac5-1

img_c271ce522351-1
img_de6605b58358-1

FWIW, I also have the behaviour @Harwood describes when using the "mosh myserver" action from spotlight.

Same here.
Any fixes?

I posted #695 (sorry for the dupe — I posted it from my phone, so didn't have a chance to search the issues). I've attached a video here of the issue as a zip file.

Sorry I can't be more helpful. If there's anything else I can provide, let me know.

IMG_6214.TRIM.MOV.zip

Still occurs if you pass in a command to run, e.g. mosh myhost echo Hello. (v12.7 from App Store)

Yep, app version (v12.8) with the fix is not in App Store yet.

@yury Is there a way to sign up for the TestFlight builds? I'd definitely be interested in running them.

@toph-allen sure, ping me at yury.[email protected]

Pushed 12.8 for TF review. Should be available soon.

I still seem to be having the issue in the new TestFlight build. On my iPhone, I tried deleting-restarting-reinstalling, in case it was some weird old cached shortcut data, but that didn’t work either.

Sent with GitHawk

The new testflight build does not work for me as well: while the command “mosh test” works when run from blink’s command line (test is a connection), it doesn’t when run from apple shortcuts. Btw, the command “ssh test” is correctly executed when run from shortcuts.

Do not why my shortcuts where working (mosh office, and mosh imac). But I finally found original issue (being able to reproduce with mosh 192.168.0.2).

Fix in v12.8 build 89.

But you have to rerun those commands first. Because donated activities are broken.

Build 89 seems to have broken mosh entirely for me. On both my testing devices (iPhone and iPad running the latest iOS 12.2 beta, I get:

blink> test

Socket error: Unknown error: -1
Did not find remote IP address

This persists after deleting and reinstalling the app.

[Edit] ssh still works for me.

Thank you @toph-allen.

Build 91 is out.

Build 91 does not solve the original problem for me.
“mosh test” works if run from blink, but does not from apple shortcuts.
“ssh test” always works.

Hi @maurizio-manuguerra-mq

You have to recreate Siri shortcut. Issue was in donating wrong command in activity.

  1. Delete your shortcut.
  2. run mosh host
  3. Try it with spotlight
  4. Add to siri shortcut

Build 91 also didn't fix it for me.

Perhaps this is because the old incorrect command had already been donated to the system's store, and so any new instances don't overwrite it? As far as I can tell, it persists between installs, too, and also includes mistyped and deleted servers (see #700 for a bug report about this behavior).

It's possible for an app to delete its donated shortcuts — this page documents how: Deleting Donated Shortcuts | Apple Developer Documentation. Perhaps it's necessary for Blink to have donated shortcuts expire every so often, or to provide some sort of troubleshooting option to manually purge them?

@toph-allen,

We do not donate anything yet with Siri Shortcuts API. We just activating NSUserActivity with command you typed. So yet, we do not control what operation it is (cp, spc, mosh, or smth else). We could try to find any matches of the host and try to delete activity donations for that host. But for now it is not good way to do that.

Video how I do it

Yeah — and I tested the theory from my previous comment by creating a totally new host, and that didn't work, so I guess build 91 still doesn't fix the original issue for me.

Launching ssh connections from Spotlight works, but not mosh connections.

[Edit] Just to be very clear:

  • Launching any connection, e.g. mosh server or ssh server, from the app works fine.
  • Searching via Spotlight and launching an ssh connection, e.g. ssh server, works fine.
  • Searching via Spotlight and launching a mosh connection, e.g. mosh server, just runs mosh.

And, I'm launching things exactly in the same way as your video, @yury

That is super strange... but I have an idea.

I understand why ssh works and mosh doesn't. Mosh command mess up with original command line (It actually inserts \0 (end of the string) char while spliting arguments - 'mosh argsbecomemosh\0args). So we donatingmosh\0args` string as command line in activity.

But for now I don't undersand why I can't repro that.

Tomorrow I will try to implement cmd copy other way. But new TF will be only on Monday. Sorry about that.

Thank you for your time.

Good luck finding a workaround! I’ll keep an eye out for a build sometime soon.

Sent with GitHawk

@toph-allen in next release it will be command to remove all shortcuts:

config delete-activities all

Version 12.8 Build 93 works!
Thanks

Fixed for me too!

Works for me as well on 12.8 (93). Thanks @yury!

Closing, feel free to reopen if something is wrong.

It worked up to build 97, but it does not work for me in build 99.
No errors or messages are displayed, just the soft keyboard makes a weird up/down dance.

It’s a different kind of problem than before, as both “mosh myconfig” and “ssh myconfig” do not work from shortcuts.

I can confirm I’m seeing the same behavior described by @maurizio-manuguerra-mq with build 99.

Was this page helpful?
0 / 5 - 0 ratings

Related issues

nriley picture nriley  Â·  3Comments

dlasky picture dlasky  Â·  4Comments

jwr picture jwr  Â·  3Comments

carloscabanero picture carloscabanero  Â·  4Comments

proportional picture proportional  Â·  3Comments