Project

General

Profile

Actions

Feature #1504

open

lua: better notification in verbose mode on script errors

Added by Peter Manev over 9 years ago. Updated about 3 years ago.

Status:
New
Priority:
Normal
Assignee:
Target version:
Effort:
Difficulty:
Label:

Description

Using - Suricata version 2.1dev (rev b5e1df2)

On a lua script err/failure - if not possible to err - we should at least relay a warning as compared to "Info" msg.

[9156] 12/7/2015 -- 13:51:42 - (tmqh-packetpool.c:373) <Info> (PacketPoolInit) -- preallocated 1024 packets. Total memory 3598336
[9157] 12/7/2015 -- 13:51:42 - (detect-lua.c:578) <Info> (DetectLuaAppMatch) -- failed to run script: /etc/suricata/rules/self-signed-cert.lua:21: attempt to call global 'format' (a nil value)
[9156] 12/7/2015 -- 13:51:42 - (suricata.c:1088) <Info> (SCPrintElapsedTime) -- time elapsed 0.026s

It comes really handy when using the color output in verbose mode.


Related issues 1 (1 open0 closed)

Related to Suricata - Feature #4775: lua: overhaul lua supportIn ProgressJason IshActions
Actions #1

Updated by Victor Julien over 9 years ago

  • Status changed from New to Assigned
  • Assignee set to Victor Julien
  • Target version set to 3.0RC1
Actions #2

Updated by Victor Julien about 9 years ago

  • Target version changed from 3.0RC1 to 70
Actions #3

Updated by Victor Julien almost 8 years ago

  • Status changed from Assigned to New
  • Assignee changed from Victor Julien to OISF Dev
  • Target version changed from 70 to TBD
Actions #4

Updated by Andreas Herz over 5 years ago

  • Tracker changed from Bug to Feature
Actions #5

Updated by Victor Julien over 5 years ago

This should probably be combined with some form of rate limiting. In theory every call to a script could generate this output.

Actions #6

Updated by Victor Julien about 3 years ago

  • Subject changed from better notification in verbose mode on luascript err to lua: better notification in verbose mode on script errors
Actions #7

Updated by Victor Julien about 3 years ago

Actions

Also available in: Atom PDF