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

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

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

    https://github.com/apache/nifi/pull/2177#discussion_r141146066
  
    --- 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 --
    
    Does this need to be `null` or could it be an empty `Set` to allow for 
simpler iteration on the consuming end?


> 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