arpadboda commented on a change in pull request #650: MINIFICPP-1034 - MQTT
processors doesnt work
URL: https://github.com/apache/nifi-minifi-cpp/pull/650#discussion_r326269553
##########
File path: extensions/mqtt/processors/AbstractMQTTProcessor.cpp
##########
@@ -46,32 +46,15 @@ core::Property AbstractMQTTProcessor::SecurityCA("Security
CA", "File or directo
core::Property AbstractMQTTProcessor::SecurityCert("Security Cert", "Path to
client's public key (PEM) used for authentication", "");
core::Property AbstractMQTTProcessor::SecurityPrivateKey("Security Private
Key", "Path to client's private key (PEM) used for authentication", "");
core::Property AbstractMQTTProcessor::SecurityPrivateKeyPassWord("Security
Pass Phrase", "Private key passphrase", "");
-core::Relationship AbstractMQTTProcessor::Success("success", "FlowFiles that
are sent successfully to the destination are transferred to this relationship");
-core::Relationship AbstractMQTTProcessor::Failure("failure", "FlowFiles that
failed to send to the destination are transferred to this relationship");
-void AbstractMQTTProcessor::initialize() {
- // Set the supported properties
- std::set<core::Property> properties;
- properties.insert(BrokerURL);
- properties.insert(CleanSession);
- properties.insert(ClientID);
- properties.insert(UserName);
- properties.insert(PassWord);
- properties.insert(KeepLiveInterval);
- properties.insert(ConnectionTimeOut);
- properties.insert(QOS);
- properties.insert(Topic);
- setSupportedProperties(properties);
- // Set the supported relationships
- std::set<core::Relationship> relationships;
- relationships.insert(Success);
- relationships.insert(Failure);
- setSupportedRelationships(relationships);
- MQTTClient_SSLOptions sslopts_ = MQTTClient_SSLOptions_initializer;
Review comment:
This was just hiding instead of initializing the member. So it had no chance
to work, but it didn't pop up given the security properties are not exported,
so it's never referenced.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
With regards,
Apache Git Services
