[ 
https://issues.apache.org/jira/browse/KNOX-3457?focusedWorklogId=1041648&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-1041648
 ]

ASF GitHub Bot logged work on KNOX-3457:
----------------------------------------

                Author: ASF GitHub Bot
            Created on: 15/Sep/26 12:16
            Start Date: 15/Sep/26 12:16
    Worklog Time Spent: 10m 
      Work Description: smolnar82 opened a new pull request, #1399:
URL: https://github.com/apache/knox/pull/1399

   [KNOX-3457](https://issues.apache.org/jira/browse/KNOX-3457) - Remove unused 
delegation.requested.scope.enabled provider parameter
   
   ## What changes were proposed in this pull request?
   
   Removes the dead `delegation.requested.scope.enabled` provider parameter 
introduced with KNOX-3441. It was read into a field and exposed via 
`isDelegationRequestedScopeEnabled()`, but the getter had no production callers 
— scope is never parsed from the exchange request (`TokenExchangeHandler` 
always passes an empty `requestedScopes` set), so the flag had no behavioral 
effect.
   
   - `JWTFederationFilter`: removed the constant and its comment block, the 
`delegationRequestedScopeEnabled` field, the init-param read, and the 
`isDelegationRequestedScopeEnabled()` getter.
   - `JWTFederationFilterDelegationConfigTest`: dropped the scope assertions 
from the shared cases, deleted the dedicated scope test, and removed the scope 
property from the all-flags test.
   - `HadoopAuthFilterTest`: removed the now-obsolete mock stub for the 
parameter.
   
   Pure dead-code removal — no behavior change. Actually implementing 
requested-scope for token exchange is tracked separately.
   
   ## How was this patch tested?
   
   - Repo-wide grep confirms no remaining references to the constant, field, 
getter, or `"delegation.requested.scope.enabled"` string.
   - `mvn -pl 
gateway-provider-security-jwt,gateway-provider-security-hadoopauth -am test 
-Dtest=JWTFederationFilterDelegationConfigTest,HadoopAuthFilterTest`: 
`JWTFederationFilterDelegationConfigTest` 7/7 and `HadoopAuthFilterTest` 12/12 
pass.
   
   ## Integration Tests
   
   N/A
   




Issue Time Tracking
-------------------

            Worklog Id:     (was: 1041648)
    Remaining Estimate: 0h
            Time Spent: 10m

> Remove unused delegation.requested.scope.enabled provider parameter
> -------------------------------------------------------------------
>
>                 Key: KNOX-3457
>                 URL: https://issues.apache.org/jira/browse/KNOX-3457
>             Project: Apache Knox
>          Issue Type: Sub-task
>          Components: Server
>    Affects Versions: 3.1.0
>            Reporter: Sandor Molnar
>            Assignee: Sandor Molnar
>            Priority: Major
>             Fix For: 3.1.0
>
>          Time Spent: 10m
>  Remaining Estimate: 0h
>
> The delegation.requested.scope.enabled provider parameter 
> ({{JWTFederationFilter.DELEGATION_REQUESTED_SCOPE_ENABLED}}) is read into a 
> field and exposed via {{isDelegationRequestedScopeEnabled()}}, but the getter 
> has no production callers, it is referenced only by config-parse tests. Scope 
> is never parsed from the exchange request, so the flag has no behavioral 
> effect.
> Scope: Remove the constant, field, init-param read, and getter from 
> {{JWTFederationFilter}}; drop the scope cases from 
> {{JWTFederationFilterDelegationConfigTest}}; and remove the corresponding 
> mock stub in {{HadoopAuthFilterTest}}. Purely dead-code removal; no behavior 
> change.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

Reply via email to