Project

General

Profile

Feature #355

/exec commands

Added by bdgraue over 8 years ago. Updated over 3 years ago.

Status:
Confirmed
Priority:
Normal
Assignee:
Category:
General / Unspecified
Target version:
Start date:
Due date:
% Done:

90%

OS:
Any

Description

in all (by me) known irc-clients i can use /exec to make an output to the channel, like:

/exec -o dmesg | tail

and the (konsole)output will be shown in the channel

with /exec without -o it will only be shown myself, with -o it will be public

0001-Don-t-send-exec-output-by-default.patch View - Add -o option (6.82 KB) Sputnick, 09/28/2009 06:32 PM

History

#1 Updated by wizzy almost 8 years ago

Where do you want this to be run?
The client or the server? Presumably the server.

#2 Updated by Sputnick almost 8 years ago

  • Status changed from New to Confirmed
  • Assignee set to Sputnick
  • Priority changed from High to Normal
  • Target version set to 0.5.0
  • % Done changed from 0 to 90

We already have /exec support in git master. However, -o we don't have yet, and that would be a nice thing to have, so I keep this report open.

Commands are run on the client side only, for various reasons (e.g. a client shouldn't be able to run anything on the server for security reasons).

#3 Updated by Sputnick over 7 years ago

This patch adds -o support to /exec. However, seems that Konversation doesn't do this anymore, so it's not really standard in all clients apparently. Also, it would change existing behavior and break users' aliases. So I'm not at all sure anymore that we want that...

How about keeping the current behavior, and add a -t (test) or -p (pretend) option instead, that allows keeping the output local for testing purposes?

#4 Updated by Anonymous about 5 years ago

  • Target version set to Some future release

I second the -p/-t to hide output, mostly because one isn't able to run arbitrary commands without wrapper script and because of current behaviour.

#5 Updated by Anonymous over 3 years ago

  • Status changed from Feedback to Confirmed

Also available in: Atom PDF