You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@nifi.apache.org by sz...@apache.org on 2022/05/11 13:20:59 UTC

[nifi-minifi-cpp] 02/02: MINIFICPP-1785 Fix CWEL attributes to be valid in HTTP requests

This is an automated email from the ASF dual-hosted git repository.

szaszm pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/nifi-minifi-cpp.git

commit 2b065b4eb1332d014280007fd0045c1c76e40790
Author: Gabor Gyimesi <ga...@gmail.com>
AuthorDate: Wed May 11 15:10:06 2022 +0200

    MINIFICPP-1785 Fix CWEL attributes to be valid in HTTP requests
    
    Closes #1287
    Signed-off-by: Marton Szasz <sz...@apache.org>
---
 extensions/windows-event-log/ConsumeWindowsEventLog.cpp            | 4 ++--
 extensions/windows-event-log/tests/ConsumeWindowsEventLogTests.cpp | 4 ++--
 2 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/extensions/windows-event-log/ConsumeWindowsEventLog.cpp b/extensions/windows-event-log/ConsumeWindowsEventLog.cpp
index 7a0033340..7cc7c2fdc 100644
--- a/extensions/windows-event-log/ConsumeWindowsEventLog.cpp
+++ b/extensions/windows-event-log/ConsumeWindowsEventLog.cpp
@@ -696,8 +696,8 @@ void ConsumeWindowsEventLog::putEventRenderFlowFileToSession(const EventRender&
   auto commitFlowFile = [&] (const std::shared_ptr<core::FlowFile>& flowFile, const std::string& content, const std::string& mimeType) {
     session.writeBuffer(flowFile, content);
     session.putAttribute(flowFile, core::SpecialFlowAttribute::MIME_TYPE, mimeType);
-    session.putAttribute(flowFile, "Timezone name", timezone_name_);
-    session.putAttribute(flowFile, "Timezone offset", timezone_offset_);
+    session.putAttribute(flowFile, "timezone.name", timezone_name_);
+    session.putAttribute(flowFile, "timezone.offset", timezone_offset_);
     session.getProvenanceReporter()->receive(flowFile, provenanceUri_, getUUIDStr(), "Consume windows event logs", 0ms);
     session.transfer(flowFile, Success);
   };
diff --git a/extensions/windows-event-log/tests/ConsumeWindowsEventLogTests.cpp b/extensions/windows-event-log/tests/ConsumeWindowsEventLogTests.cpp
index 029818374..90e728a93 100644
--- a/extensions/windows-event-log/tests/ConsumeWindowsEventLogTests.cpp
+++ b/extensions/windows-event-log/tests/ConsumeWindowsEventLogTests.cpp
@@ -159,8 +159,8 @@ TEST_CASE("ConsumeWindowsEventLog can consume new events", "[onTrigger]") {
     REQUIRE(LogTestController::getInstance().contains("<EventData><Data>Event one</Data></EventData>"));
 
     // make sure timezone attributes are present
-    REQUIRE(LogTestController::getInstance().contains("key:Timezone offset value:"));
-    REQUIRE(LogTestController::getInstance().contains("key:Timezone name value:"));
+    REQUIRE(LogTestController::getInstance().contains("key:timezone.offset value:"));
+    REQUIRE(LogTestController::getInstance().contains("key:timezone.name value:"));
   }
 
   SECTION("Read two events") {