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

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

                Author: ASF GitHub Bot
            Created on: 14/Jun/21 07:27
            Start Date: 14/Jun/21 07:27
    Worklog Time Spent: 10m 
      Work Description: smolnar82 opened a new pull request #458:
URL: https://github.com/apache/knox/pull/458


   ## What changes were proposed in this pull request?
   
   In some cases, it's better to return `401/UNAUTHORIZED` than 
`400/BAD_REQUEST`.
   
   ## How was this patch tested?
   
   Ran JUnit tests and executed manual testing locally.
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
[email protected]


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

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

> Consolidate HTTP error codes in JWT federation filter
> -----------------------------------------------------
>
>                 Key: KNOX-2621
>                 URL: https://issues.apache.org/jira/browse/KNOX-2621
>             Project: Apache Knox
>          Issue Type: Bug
>          Components: Server
>    Affects Versions: 1.6.0
>            Reporter: Sandor Molnar
>            Assignee: Sandor Molnar
>            Priority: Critical
>             Fix For: 1.6.0
>
>          Time Spent: 10m
>  Remaining Estimate: 0h
>
> In 
> {{org.apache.knox.gateway.provider.federation.jwt.filter.AbstractJWTFilter.validateToken(...)}}
>  there are multiple validation takes place and Knox returns 
> {{400/BAD_REQUEST}} in some places where {{401/UNAUTHORIZED}} sould have been 
> used (for instance, when a token is expired).



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Reply via email to