[
https://issues.apache.org/jira/browse/KNOX-2762?focusedWorklogId=781429&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-781429
]
ASF GitHub Bot logged work on KNOX-2762:
----------------------------------------
Author: ASF GitHub Bot
Created on: 14/Jun/22 23:49
Start Date: 14/Jun/22 23:49
Worklog Time Spent: 10m
Work Description: harshiljhaveri opened a new pull request, #594:
URL: https://github.com/apache/knox/pull/594
## What changes were proposed in this pull request?
Whitespaces around delimiters were causing problems while specifying
composite auth provider names. The parseProviderNames function has been
modified to trim provider names before passing it to the Deployment Contributor
function.
## How was this patch tested?
Ran automated unit/integration tests to ensure provider names did not
contain whitespaces. Also ran manual tests to ensure no malfunctions.
[Knox Contributing
Process](https://cwiki.apache.org/confluence/display/KNOX/Contribution+Process#ContributionProcess-GithubWorkflow)
reviewed.
Issue Time Tracking
-------------------
Worklog Id: (was: 781429)
Remaining Estimate: 0h
Time Spent: 10m
> Whitespaces around delimiters in composite provider names gives
> NullPointerException
> ------------------------------------------------------------------------------------
>
> Key: KNOX-2762
> URL: https://issues.apache.org/jira/browse/KNOX-2762
> Project: Apache Knox
> Issue Type: Bug
> Components: KnoxCLI
> Affects Versions: 1.6.0
> Reporter: Harshil Jhaveri
> Priority: Minor
> Attachments: NPE gateway log.png
>
> Time Spent: 10m
> Remaining Estimate: 0h
>
> When giving space before the delimiter for composite.provider.names in
> composite authorisation provider. The topology deployment is failing and
> returning a null pointer exception.
> The topology state:
> {color:#e8bf6a}<provider>{color}{color:#e8bf6a}
> <role>{color}authorization{color:#e8bf6a}</role>{color}{color:#e8bf6a}
> <name>{color}CompositeAuthz{color:#e8bf6a}</name>{color}{color:#e8bf6a}
> <enabled>{color}true{color:#e8bf6a}</enabled>{color}{color:#e8bf6a}
> <param>{color}{color:#e8bf6a}
> <name>{color}composite.provider.names{color:#e8bf6a}</name>{color}{color:#e8bf6a}
> <value>{color}AclsAuthz
> ,XASecurePDPKnox{color:#e8bf6a}</value>{color}{color:#e8bf6a}
> </param>{color}{color:#e8bf6a} <param>{color}{color:#e8bf6a}
> <name>{color}AclsAuthz.ranger.acl{color:#e8bf6a}</name>{color}{color:#e8bf6a}
> <value>*{color}{*};{*}{*};{*}{color:#e8bf6a}</value>{color}{color:#e8bf6a}
> </param>{color}{color:#e8bf6a}</provider>{color}
> {color:#e8bf6a}{color:#172b4d}CC:{color} [~pzampino] [~smore] [~smolnar]
> {color}
--
This message was sent by Atlassian Jira
(v8.20.7#820007)
