search

fugazi-io / webclient

web based terminal application for executing local and remote commands
http://fugazi.io
28 stars 12 forks source link

Caught errors are shown in the console #27

Open TomZelazny opened 7 years ago

TomZelazny commented 7 years ago

I seems errors are not caught and are thrown all the way into the browser console, i.e.:

Uncaught ParseException: '(' is an illegal character in word

attached is an image demonstrating the problem. image

creativeChips commented 7 years ago

This is intentional. The char ( is not allowed when invoking commands as it cannot be within the syntax of a command, where these chars have special meaning. The exception manifest as the error message seen on the terminal, following the execution. The console shows the exception details for debug.

nitzantomer commented 7 years ago

I'm not sure whether or not those errors need to be shown in the console. As @creativeChips said, it can be useful to have them when debugging but there's currently no way of opting out of the "dev mode".

Reopening and marking as discussion.