[ 
https://issues.apache.org/jira/browse/MINIFI-184?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15856339#comment-15856339
 ] 

ASF GitHub Bot commented on MINIFI-184:
---------------------------------------

Github user phrocker commented on a diff in the pull request:

    https://github.com/apache/nifi-minifi-cpp/pull/41#discussion_r99357485
  
    --- Diff: cmake/FindOpenSSL.cmake ---
    @@ -0,0 +1,28 @@
    +#  OPENSSL_ROOT_DIR - Set this variable to the root installation of OpenSSL
    +#
    +# Read-Only variables:
    +#  OPENSSL_FOUND - system has the OpenSSL library
    +#  OPENSSL_INCLUDE_DIR - the OpenSSL include directory
    +#  OPENSSL_LIBRARIES - The libraries needed to use OpenSSL
    +#  OPENSSL_VERSION - This is set to $major.$minor.$revision$path (eg. 
0.9.8s)
    +
    +find_path(OPENSSL_INCLUDE_DIR
    +    NAMES openssl/ssl.h
    --- End diff --
    
    Did you consider BOTAN? That's a purely C++/Crypto library ( one of many ). 
Any particular reason we are introducing a C library when C++ libraries exist? 


> Add Security Support for Site to Site
> -------------------------------------
>
>                 Key: MINIFI-184
>                 URL: https://issues.apache.org/jira/browse/MINIFI-184
>             Project: Apache NiFi MiNiFi
>          Issue Type: Improvement
>          Components: Core Framework
>    Affects Versions: cpp-0.2.0
>            Reporter: bqiu
>            Assignee: bqiu
>             Fix For: cpp-0.2.0
>
>
> Add Security Support



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

Reply via email to