lordgamez commented on code in PR #1842:
URL: https://github.com/apache/nifi-minifi-cpp/pull/1842#discussion_r1695212591
##########
libminifi/src/core/flow/StructuredConfiguration.cpp:
##########
@@ -989,6 +989,9 @@ std::string StructuredConfiguration::getOptionalField(const
Node& node, const st
return default_value;
}
+ if (result.isSequence()) {
+ return (*result.begin()).getString().value();
Review Comment:
Good point, I think it's possible and we can use the default in that case.
Added a fix and test in d86c2b9145b3b7f0c44ec05d7d2c66bc28b321a6
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
