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

ASF GitHub Bot commented on NIFI-4382:
--------------------------------------

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

    https://github.com/apache/nifi/pull/2177#discussion_r141157943
  
    --- Diff: 
nifi-commons/nifi-properties/src/main/java/org/apache/nifi/util/NiFiProperties.java
 ---
    @@ -886,18 +893,70 @@ public String getOidcPreferredJwsAlgorithm() {
         }
     
         /**
    +     * Returns whether Knox SSO is enabled.
    +     *
    +     * @return whether Knox SSO is enabled
    +     */
    +    public boolean isKnoxSsoEnabled() {
    +        return !StringUtils.isBlank(getKnoxUrl());
    +    }
    +
    +    /**
    +     * Returns the Knox URL.
    +     *
    +     * @return Knox URL
    +     */
    +    public String getKnoxUrl() {
    +        return getProperty(SECURITY_USER_KNOX_URL);
    +    }
    +
    +    /**
    +     * Gets the configured Knox Audiences.
    +     *
    +     * @return Knox audiences
    +     */
    +    public Set<String> getKnoxAudiences() {
    +        final String rawAudiences = 
getProperty(SECURITY_USER_KNOX_AUDIENCES);
    +        if (StringUtils.isBlank(rawAudiences)) {
    +            return null;
    --- End diff --
    
    While it's not possible to realize the distinction when configuring the 
properties file and using isBlank, I typically use null for unset while an 
empty Set would indicate the value is configured with no values. I'm happy to 
make this change if you want me to.


> Add KnoxSSO support to NiFi
> ---------------------------
>
>                 Key: NIFI-4382
>                 URL: https://issues.apache.org/jira/browse/NIFI-4382
>             Project: Apache NiFi
>          Issue Type: Bug
>          Components: Core Framework
>            Reporter: Jeff Storck
>            Assignee: Jeff Storck
>
> Add support for KnoxSSO to NiFi.
> Reference documentation: 
> http://knox.apache.org/books/knox-0-13-0/dev-guide.html#KnoxSSO+Integration



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

Reply via email to