]> git.ipfire.org Git - thirdparty/suricata.git/commitdiff
file-log: remove and add warning 3680/head
authorVictor Julien <victor@inliniac.net>
Sun, 24 Feb 2019 18:56:29 +0000 (19:56 +0100)
committerVictor Julien <victor@inliniac.net>
Sun, 24 Feb 2019 19:00:55 +0000 (20:00 +0100)
Feature was deprecated and scheduled for removal.

Ticket #2376

12 files changed:
doc/userguide/file-extraction/file-extraction.rst
doc/userguide/file-extraction/md5.rst
src/Makefile.am
src/log-file.c [deleted file]
src/log-file.h [deleted file]
src/log-filestore.c
src/output-json-file.c
src/output.c
src/runmodes.c
src/suricata-common.h
src/util-profiling.c
suricata.yaml.in

index d59e8699c0433c0787caaf908d40a011e4919ced..1c2e564e999a9c1793ad436058a00de25de3103f 100644 (file)
@@ -113,25 +113,6 @@ recommended.
 
 Each file that is stored will have a name "file.<id>". The id will be reset and files will be overwritten unless the waldo option is used. A "file.<id>.meta" file is generated containing file metadata if write-meta is set to yes (default). If the include-pid option is set, the files will instead have a name "file.<pid>.<id>", and metafiles will be "file.<pid>.<id>.meta". Files will additionally have the suffix ".tmp" while they are open, which is only removed when they are finalized.
 
-File-Log (Deprecated)
----------------------
-
-The file-log is a standalone logger that logs information about files
-extracted. It is now deprecated and should not be used. It has been
-replaced by ``eve.files`` in the :ref:`suricata-yaml-outputs-eve`
-output.
-
-::
-
-    - file-log:
-        enabled: yes
-        filename: files-json.log
-        append: yes
-        #filetype: regular # 'regular', 'unix_stream' or 'unix_dgram'
-        force-magic: no   # force logging magic on all logged files
-        force-hash: [md5] # force logging of md5 checksums
-
-
 Rules
 ~~~~~
 
index df3e57d0ea4616a0490c0549ddb5c4935604fef1..bab883260ebfd0648bee242cfb57c0c17b24fe07 100644 (file)
@@ -8,7 +8,6 @@ Configuration
 
 In the suricata yaml:
 
-
 ::
 
 
@@ -19,16 +18,22 @@ In the suricata yaml:
          force-hash: [md5]  # force logging of md5 checksums
          #waldo: file.waldo # waldo file to store the file_id across runs
 
-Optionally, for JSON output:
-
+For JSON output:
 
 ::
 
+    outputs:
+      - eve-log:
+        enabled: yes
+        filetype: regular #regular|syslog|unix_dgram|unix_stream|redis
+        filename: eve.json
+        types:
+          - files:
+            force-magic: no   # force logging magic on all logged files
+            # force logging of checksums, available hash functions are md5,
+            # sha1 and sha256
+            #force-hash: [md5]
 
-   - file-log:
-     enabled: yes
-     filename: files-json.log
-     append: no
 
 Other settings affecting :doc:`file-extraction`
 
index 79b49c8adf002d4db46556bbd6f36dc7380a42db..28eb33f4456e7a0b349eda9254e53c8ec811279b 100644 (file)
@@ -286,7 +286,6 @@ ippair-storage.c ippair-storage.h \
 ippair-timeout.c ippair-timeout.h \
 log-dnslog.c log-dnslog.h \
 log-droplog.c log-droplog.h \
-log-file.c log-file.h \
 log-filestore.c log-filestore.h \
 log-cf-common.c log-cf-common.h \
 log-httplog.c log-httplog.h \
diff --git a/src/log-file.c b/src/log-file.c
deleted file mode 100644 (file)
index 045588f..0000000
+++ /dev/null
@@ -1,464 +0,0 @@
-/* Copyright (C) 2007-2013 Open Information Security Foundation
- *
- * You can copy, redistribute or modify this Program under the terms of
- * the GNU General Public License version 2 as published by the Free
- * Software Foundation.
- *
- * This program is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- * GNU General Public License for more details.
- *
- * You should have received a copy of the GNU General Public License
- * version 2 along with this program; if not, write to the Free Software
- * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA
- * 02110-1301, USA.
- */
-
-/**
- * \file
- *
- * \author Victor Julien <victor@inliniac.net>
- *
- * Log files we track.
- *
- */
-
-#include "suricata-common.h"
-#include "debug.h"
-#include "detect.h"
-#include "pkt-var.h"
-#include "conf.h"
-
-#include "threadvars.h"
-#include "tm-modules.h"
-
-#include "threads.h"
-
-#include "app-layer-parser.h"
-
-#include "detect-filemagic.h"
-
-#include "stream.h"
-
-#include "util-print.h"
-#include "util-unittest.h"
-#include "util-privs.h"
-#include "util-debug.h"
-#include "util-atomic.h"
-#include "util-file.h"
-#include "util-time.h"
-
-#include "output.h"
-
-#include "log-file.h"
-#include "util-logopenfile.h"
-
-#include "app-layer-htp.h"
-#include "app-layer-smtp.h"
-#include "util-decode-mime.h"
-#include "util-memcmp.h"
-#include "stream-tcp-reassemble.h"
-
-#define MODULE_NAME "LogFileLog"
-
-#define DEFAULT_LOG_FILENAME "files-json.log"
-
-typedef struct LogFileLogThread_ {
-    LogFileCtx *file_ctx;
-    /** LogFileCtx has the pointer to the file and a mutex to allow multithreading */
-    uint32_t file_cnt;
-} LogFileLogThread;
-
-static void LogFileMetaGetUri(FILE *fp, const Packet *p, const File *ff)
-{
-    HtpState *htp_state = (HtpState *)p->flow->alstate;
-    if (htp_state != NULL) {
-        htp_tx_t *tx = AppLayerParserGetTx(IPPROTO_TCP, ALPROTO_HTTP, htp_state, ff->txid);
-        if (tx != NULL) {
-            HtpTxUserData *tx_ud = htp_tx_get_user_data(tx);
-            if (tx_ud != NULL) {
-                if (tx_ud->request_uri_normalized != NULL) {
-                    PrintRawJsonFp(fp,
-                                   bstr_ptr(tx_ud->request_uri_normalized),
-                                   bstr_len(tx_ud->request_uri_normalized));
-                    return;
-                }
-            }
-        }
-    }
-
-    fprintf(fp, "<unknown>");
-}
-
-static void LogFileMetaGetHost(FILE *fp, const Packet *p, const File *ff)
-{
-    HtpState *htp_state = (HtpState *)p->flow->alstate;
-    if (htp_state != NULL) {
-        htp_tx_t *tx = AppLayerParserGetTx(IPPROTO_TCP, ALPROTO_HTTP, htp_state, ff->txid);
-        if (tx != NULL && tx->request_hostname != NULL) {
-            PrintRawJsonFp(fp, (uint8_t *)bstr_ptr(tx->request_hostname),
-                           bstr_len(tx->request_hostname));
-            return;
-        }
-    }
-
-    fprintf(fp, "<unknown>");
-}
-
-static void LogFileMetaGetReferer(FILE *fp, const Packet *p, const File *ff)
-{
-    HtpState *htp_state = (HtpState *)p->flow->alstate;
-    if (htp_state != NULL) {
-        htp_tx_t *tx = AppLayerParserGetTx(IPPROTO_TCP, ALPROTO_HTTP, htp_state, ff->txid);
-        if (tx != NULL) {
-            htp_header_t *h = NULL;
-            h = (htp_header_t *)htp_table_get_c(tx->request_headers,
-                                                "Referer");
-            if (h != NULL) {
-                PrintRawJsonFp(fp, (uint8_t *)bstr_ptr(h->value),
-                               bstr_len(h->value));
-                return;
-            }
-        }
-    }
-
-    fprintf(fp, "<unknown>");
-}
-
-static void LogFileMetaGetUserAgent(FILE *fp, const Packet *p, const File *ff)
-{
-    HtpState *htp_state = (HtpState *)p->flow->alstate;
-    if (htp_state != NULL) {
-        htp_tx_t *tx = AppLayerParserGetTx(IPPROTO_TCP, ALPROTO_HTTP, htp_state, ff->txid);
-        if (tx != NULL) {
-            htp_header_t *h = NULL;
-            h = (htp_header_t *)htp_table_get_c(tx->request_headers,
-                                                "User-Agent");
-            if (h != NULL) {
-                PrintRawJsonFp(fp, (uint8_t *)bstr_ptr(h->value),
-                               bstr_len(h->value));
-                return;
-            }
-        }
-    }
-
-    fprintf(fp, "<unknown>");
-}
-
-static void LogFileMetaGetSmtp(FILE *fp, const Packet *p, const File *ff)
-{
-    SMTPState *state = (SMTPState *) p->flow->alstate;
-    if (state != NULL) {
-        SMTPTransaction *tx = AppLayerParserGetTx(IPPROTO_TCP, ALPROTO_SMTP, state, ff->txid);
-        if (tx == NULL || tx->msg_tail == NULL)
-            return;
-
-        /* Message Id */
-        if (tx->msg_tail->msg_id != NULL) {
-
-            fprintf(fp, "\"message-id\": \"");
-            PrintRawJsonFp(fp, (uint8_t *) tx->msg_tail->msg_id,
-                    (int) tx->msg_tail->msg_id_len);
-            fprintf(fp, "\", ");
-        }
-
-        /* Sender */
-        MimeDecField *field = MimeDecFindField(tx->msg_tail, "from");
-        if (field != NULL) {
-            fprintf(fp, "\"sender\": \"");
-            PrintRawJsonFp(fp, (uint8_t *) field->value,
-                    (int) field->value_len);
-            fprintf(fp, "\", ");
-        }
-    }
-}
-
-/**
- *  \internal
- *  \brief Write meta data on a single line json record
- */
-static void LogFileWriteJsonRecord(LogFileLogThread *aft, const Packet *p, const File *ff, int ipver)
-{
-    SCMutexLock(&aft->file_ctx->fp_mutex);
-
-    /* As writes are done via the LogFileCtx, check for rotation here. */
-    if (aft->file_ctx->rotation_flag) {
-        aft->file_ctx->rotation_flag = 0;
-        if (SCConfLogReopen(aft->file_ctx) != 0) {
-            SCLogWarning(SC_ERR_FOPEN, "Failed to re-open log file. "
-                "Logging for this module will be disabled.");
-        }
-    }
-
-    /* Bail early if no file pointer to write to (in the unlikely
-     * event file rotation failed. */
-    if (aft->file_ctx->fp == NULL) {
-        SCMutexUnlock(&aft->file_ctx->fp_mutex);
-        return;
-    }
-
-    FILE *fp = aft->file_ctx->fp;
-    char timebuf[64];
-    AppProto alproto = FlowGetAppProtocol(p->flow);
-
-    CreateTimeString(&p->ts, timebuf, sizeof(timebuf));
-
-    fprintf(fp, "{ ");
-
-    if (ff->file_store_id > 0)
-        fprintf(fp, "\"id\": %u, ", ff->file_store_id);
-
-    fprintf(fp, "\"timestamp\": \"");
-    PrintRawJsonFp(fp, (uint8_t *)timebuf, strlen(timebuf));
-    fprintf(fp, "\", ");
-    if (p->pcap_cnt > 0) {
-        fprintf(fp, "\"pcap_pkt_num\": %"PRIu64", ", p->pcap_cnt);
-    }
-
-    fprintf(fp, "\"ipver\": %d, ", ipver == AF_INET ? 4 : 6);
-
-    char srcip[46], dstip[46];
-    Port sp, dp;
-    switch (ipver) {
-        case AF_INET:
-            PrintInet(AF_INET, (const void *)GET_IPV4_SRC_ADDR_PTR(p), srcip, sizeof(srcip));
-            PrintInet(AF_INET, (const void *)GET_IPV4_DST_ADDR_PTR(p), dstip, sizeof(dstip));
-            break;
-        case AF_INET6:
-            PrintInet(AF_INET6, (const void *)GET_IPV6_SRC_ADDR(p), srcip, sizeof(srcip));
-            PrintInet(AF_INET6, (const void *)GET_IPV6_DST_ADDR(p), dstip, sizeof(dstip));
-            break;
-        default:
-            strlcpy(srcip, "<unknown>", sizeof(srcip));
-            strlcpy(dstip, "<unknown>", sizeof(dstip));
-            break;
-    }
-    sp = p->sp;
-    dp = p->dp;
-
-    fprintf(fp, "\"srcip\": \"%s\", ", srcip);
-    fprintf(fp, "\"dstip\": \"%s\", ", dstip);
-    fprintf(fp, "\"protocol\": %" PRIu32 ", ", p->proto);
-    if (PKT_IS_TCP(p) || PKT_IS_UDP(p)) {
-        fprintf(fp, "\"sp\": %" PRIu16 ", ", sp);
-        fprintf(fp, "\"dp\": %" PRIu16 ", ", dp);
-    }
-
-    if (alproto == ALPROTO_HTTP) {
-        fprintf(fp, "\"http_uri\": \"");
-        LogFileMetaGetUri(fp, p, ff);
-        fprintf(fp, "\", ");
-
-        fprintf(fp, "\"http_host\": \"");
-        LogFileMetaGetHost(fp, p, ff);
-        fprintf(fp, "\", ");
-
-        fprintf(fp, "\"http_referer\": \"");
-        LogFileMetaGetReferer(fp, p, ff);
-        fprintf(fp, "\", ");
-
-        fprintf(fp, "\"http_user_agent\": \"");
-        LogFileMetaGetUserAgent(fp, p, ff);
-        fprintf(fp, "\", ");
-    } else if (p->flow->alproto == ALPROTO_SMTP) {
-        /* Only applicable to SMTP */
-        LogFileMetaGetSmtp(fp, p, ff);
-    }
-
-    fprintf(fp, "\"filename\": \"");
-    PrintRawJsonFp(fp, ff->name, ff->name_len);
-    fprintf(fp, "\", ");
-#ifdef HAVE_MAGIC
-    fprintf(fp, "\"magic\": \"");
-    if (ff->magic) {
-        PrintRawJsonFp(fp, (uint8_t *)ff->magic, strlen(ff->magic));
-    } else {
-        fprintf(fp, "unknown");
-    }
-    fprintf(fp, "\", ");
-#endif
-    switch (ff->state) {
-        case FILE_STATE_CLOSED:
-            fprintf(fp, "\"state\": \"CLOSED\", ");
-#ifdef HAVE_NSS
-            if (ff->flags & FILE_MD5) {
-                fprintf(fp, "\"md5\": \"");
-                size_t x;
-                for (x = 0; x < sizeof(ff->md5); x++) {
-                    fprintf(fp, "%02x", ff->md5[x]);
-                }
-                fprintf(fp, "\", ");
-            }
-            if (ff->flags & FILE_SHA1) {
-                fprintf(fp, "\"sha1\": \"");
-                size_t x;
-                for (x = 0; x < sizeof(ff->sha1); x++) {
-                    fprintf(fp, "%02x", ff->sha1[x]);
-                }
-                fprintf(fp, "\", ");
-            }
-            if (ff->flags & FILE_SHA256) {
-                fprintf(fp, "\"sha256\": \"");
-                size_t x;
-                for (x = 0; x < sizeof(ff->sha256); x++) {
-                    fprintf(fp, "%02x", ff->sha256[x]);
-                }
-                fprintf(fp, "\", ");
-            }
-#endif
-            break;
-        case FILE_STATE_TRUNCATED:
-            fprintf(fp, "\"state\": \"TRUNCATED\", ");
-            break;
-        case FILE_STATE_ERROR:
-            fprintf(fp, "\"state\": \"ERROR\", ");
-            break;
-        default:
-            fprintf(fp, "\"state\": \"UNKNOWN\", ");
-            break;
-    }
-    fprintf(fp, "\"stored\": %s, ", ff->flags & FILE_STORED ? "true" : "false");
-    fprintf(fp, "\"size\": %"PRIu64" ", FileTrackedSize(ff));
-    fprintf(fp, "}\n");
-    fflush(fp);
-    SCMutexUnlock(&aft->file_ctx->fp_mutex);
-}
-
-static int LogFileLogger(ThreadVars *tv, void *thread_data, const Packet *p,
-                         const File *ff, uint8_t dir)
-{
-    SCEnter();
-    LogFileLogThread *aft = (LogFileLogThread *)thread_data;
-    int ipver = -1;
-
-    if (PKT_IS_IPV4(p)) {
-        ipver = AF_INET;
-    } else if (PKT_IS_IPV6(p)) {
-        ipver = AF_INET6;
-    } else {
-        return 0;
-    }
-
-    BUG_ON(ff->flags & FILE_LOGGED);
-
-    SCLogDebug("ff %p", ff);
-
-    LogFileWriteJsonRecord(aft, p, ff, ipver);
-
-    aft->file_cnt++;
-    return 0;
-}
-
-static TmEcode LogFileLogThreadInit(ThreadVars *t, const void *initdata, void **data)
-{
-    LogFileLogThread *aft = SCMalloc(sizeof(LogFileLogThread));
-    if (unlikely(aft == NULL))
-        return TM_ECODE_FAILED;
-    memset(aft, 0, sizeof(LogFileLogThread));
-
-    if (initdata == NULL)
-    {
-        SCLogDebug("Error getting context for LogFile. \"initdata\" argument NULL");
-        SCFree(aft);
-        return TM_ECODE_FAILED;
-    }
-
-    /* Use the Ouptut Context (file pointer and mutex) */
-    aft->file_ctx = ((OutputCtx *)initdata)->data;
-
-    *data = (void *)aft;
-    return TM_ECODE_OK;
-}
-
-static TmEcode LogFileLogThreadDeinit(ThreadVars *t, void *data)
-{
-    LogFileLogThread *aft = (LogFileLogThread *)data;
-    if (aft == NULL) {
-        return TM_ECODE_OK;
-    }
-
-    /* clear memory */
-    memset(aft, 0, sizeof(LogFileLogThread));
-
-    SCFree(aft);
-    return TM_ECODE_OK;
-}
-
-static void LogFileLogExitPrintStats(ThreadVars *tv, void *data)
-{
-    LogFileLogThread *aft = (LogFileLogThread *)data;
-    if (aft == NULL) {
-        return;
-    }
-
-    SCLogInfo("(%s) Files logged: %" PRIu32 "", tv->name, aft->file_cnt);
-}
-
-/**
- *  \internal
- *
- *  \brief deinit the log ctx and write out the waldo
- *
- *  \param output_ctx output context to deinit
- */
-static void LogFileLogDeInitCtx(OutputCtx *output_ctx)
-{
-    LogFileCtx *logfile_ctx = (LogFileCtx *)output_ctx->data;
-    LogFileFreeCtx(logfile_ctx);
-    free(output_ctx);
-}
-
-/** \brief Create a new http log LogFileCtx.
- *  \param conf Pointer to ConfNode containing this loggers configuration.
- *  \return NULL if failure, LogFileCtx* to the file_ctx if succesful
- * */
-static OutputInitResult LogFileLogInitCtx(ConfNode *conf)
-{
-    OutputInitResult result = { NULL, false };
-    LogFileCtx *logfile_ctx = LogFileNewCtx();
-    if (logfile_ctx == NULL) {
-        SCLogDebug("Could not create new LogFileCtx");
-        return result;
-    }
-
-    if (SCConfLogOpenGeneric(conf, logfile_ctx, DEFAULT_LOG_FILENAME, 1) < 0) {
-        LogFileFreeCtx(logfile_ctx);
-        return result;
-    }
-
-    OutputCtx *output_ctx = SCCalloc(1, sizeof(OutputCtx));
-    if (unlikely(output_ctx == NULL))
-        return result;
-
-    output_ctx->data = logfile_ctx;
-    output_ctx->DeInit = LogFileLogDeInitCtx;
-
-    const char *force_filestore = ConfNodeLookupChildValue(conf, "force-filestore");
-    if (force_filestore != NULL && ConfValIsTrue(force_filestore)) {
-        FileForceFilestoreEnable();
-        SCLogInfo("forcing filestore of all files");
-    }
-
-    const char *force_magic = ConfNodeLookupChildValue(conf, "force-magic");
-    if (force_magic != NULL && ConfValIsTrue(force_magic)) {
-        FileForceMagicEnable();
-        SCLogInfo("forcing magic lookup for logged files");
-    }
-
-    FileForceHashParseCfg(conf);
-    FileForceTrackingEnable();
-
-    result.ctx = output_ctx;
-    result.ok = true;
-    SCReturnCT(result, "OutputInitResult");
-}
-
-void LogFileLogRegister (void)
-{
-    OutputRegisterFileModule(LOGGER_FILE, MODULE_NAME, "file-log",
-        LogFileLogInitCtx, LogFileLogger, LogFileLogThreadInit,
-        LogFileLogThreadDeinit, LogFileLogExitPrintStats);
-
-    SCLogDebug("registered");
-}
diff --git a/src/log-file.h b/src/log-file.h
deleted file mode 100644 (file)
index b856204..0000000
+++ /dev/null
@@ -1,29 +0,0 @@
-/* Copyright (C) 2007-2011 Open Information Security Foundation
- *
- * You can copy, redistribute or modify this Program under the terms of
- * the GNU General Public License version 2 as published by the Free
- * Software Foundation.
- *
- * This program is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- * GNU General Public License for more details.
- *
- * You should have received a copy of the GNU General Public License
- * version 2 along with this program; if not, write to the Free Software
- * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA
- * 02110-1301, USA.
- */
-
-/**
- * \file
- *
- * \author Victor Julien <victor@inliniac.net>
- */
-
-#ifndef __LOG_FILELOG_H__
-#define __LOG_FILELOG_H__
-
-void LogFileLogRegister(void);
-
-#endif /* __LOG_FILELOG_H__ */
index c6b70641c7799758cb950196575f86ef14e6327e..7239abfa2c1f954ef06f357aa177021f4e90ddc1 100644 (file)
@@ -50,7 +50,6 @@
 
 #include "output.h"
 
-#include "log-file.h"
 #include "log-filestore.h"
 #include "util-logopenfile.h"
 
index 25b05d959cadef3ca2f3382e2a7f1a1f102e5e2c..09d3510ba4dff613cd2502ec2b622a2e2e5d6040 100644 (file)
@@ -52,7 +52,6 @@
 #include "util-byte.h"
 #include "util-validate.h"
 
-#include "log-file.h"
 #include "util-logopenfile.h"
 
 #include "output.h"
index 6d96b6724262985b6a7c0044fff80b2de878ad4f..547b02230edcb007970c8e5a6935abdf2559f818 100644 (file)
@@ -60,7 +60,6 @@
 #include "output-json-tls.h"
 #include "output-json-ssh.h"
 #include "log-pcap.h"
-#include "log-file.h"
 #include "output-json-file.h"
 #include "output-json-smtp.h"
 #include "output-json-stats.h"
@@ -1070,7 +1069,6 @@ void OutputRegisterLoggers(void)
     /* pcap log */
     PcapLogRegister();
     /* file log */
-    LogFileLogRegister();
     JsonFileLogRegister();
     LogFilestoreRegister();
     OutputFilestoreRegister();
index e9f0f53a887144fd601d9c0a1adea29f7e0106af..d6e0c0aad7dc0d2ea37438dfb0af0bc343a838b4 100644 (file)
@@ -747,7 +747,14 @@ void RunModeInitializeOutputs(void)
             continue;
         }
 
-        if (strncmp(output->val, "unified-", sizeof("unified-") - 1) == 0) {
+        if (strcmp(output->val, "file-log") == 0) {
+            SCLogWarning(SC_ERR_NOT_SUPPORTED,
+                    "file-log is no longer supported,"
+                    " use eve.files instead "
+                    "(see https://redmine.openinfosecfoundation.org/issues/2376"
+                    " for an explanation)");
+            continue;
+        } else if (strncmp(output->val, "unified-", sizeof("unified-") - 1) == 0) {
             SCLogWarning(SC_ERR_NOT_SUPPORTED,
                     "Unified1 is no longer supported,"
                     " use Unified2 instead "
index 153fb3645d5dec98142122d7cf406fb7d8317938..70337fba6f8a9542823b5f0d22ea12237e5d7e6a 100644 (file)
@@ -429,7 +429,6 @@ typedef enum {
     LOGGER_DROP,
     LOGGER_JSON_ALERT,
     LOGGER_JSON_DROP,
-    LOGGER_FILE,
     LOGGER_FILE_STORE,
     LOGGER_JSON_FILE,
     LOGGER_TCP_DATA,
index 6e32c05e4cbb4485be47a42e5f8402abac26187a..0aa96837784c3a557de1c713328adfbcf8777a52 100644 (file)
@@ -1319,7 +1319,6 @@ const char * PacketProfileLoggertIdToString(LoggerId id)
         CASE_CODE (LOGGER_JSON_TEMPLATE);
         CASE_CODE (LOGGER_TLS_STORE);
         CASE_CODE (LOGGER_TLS);
-        CASE_CODE (LOGGER_FILE);
         CASE_CODE (LOGGER_FILE_STORE);
         CASE_CODE (LOGGER_JSON_FILE);
         CASE_CODE (LOGGER_TCP_DATA);
index 3fe7f311c4ccccb2bce2ebad5835dd3152d478a9..a1a05e9405f1f763dfe527956a6855fee008a730 100644 (file)
@@ -532,18 +532,6 @@ outputs:
       #max-open-files: 1000
       include-pid: no # set to yes to include pid in file names
 
-  # output module to log files tracked in a easily parsable JSON format
-  - file-log:
-      enabled: no
-      filename: files-json.log
-      append: yes
-      #filetype: regular # 'regular', 'unix_stream' or 'unix_dgram'
-
-      force-magic: no   # force logging magic on all logged files
-      # force logging of checksums, available hash functions are md5,
-      # sha1 and sha256
-      #force-hash: [md5]
-
   # Log TCP data after stream normalization
   # 2 types: file or dir. File logs into a single logfile. Dir creates
   # 2 files per TCP session and stores the raw TCP data into them.