[ 
https://issues.apache.org/jira/browse/GEODE-9900?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Jinmei Liao updated GEODE-9900:
-------------------------------
    Description: 
As we have discovered in GEODE-9820, some commands (especially CQ commands) are 
not sending back `AuthenticationExpiredException` to the client with 
MessageType.EXCEPTION, hence causing the client not wrapping it in the correct 
form (See `AbstractOp around L300), We need to:
1. go over all the commands and find out what commands are NOT handling the 
`AuthenticationExpiredExcpetion` correctly.
2. fix these commands, catch `AuthenticationExpiredException` specifically and 
do a `writeException` (leave all other exception handling intact)
3. write tests to make sure we are sending the exception to the client.


#1 would help us determine how many commands are in need of this fix and size 
this story correctly.

  was:
As we have discovered in GEODE-9820, some commands (especially CQ commands) are 
not sending back `AuthenticationExpiredException` to the client with 
MessageType.EXCEPTION, hence causing the client not wrapping it in the correct 
form (See `AbstractOp around L300), We need to:
1. go over all the commands and find out what commands are NOT handling the 
`AuthenticationExpiredExcpetion` correctly.
2. fix these commands, catch `AuthenticationExpiredException` specifically and 
do a `writeException` (leave all other exception handling intact)
3. write tests to make sure we are sending the exception to the client.


> Make sure all commands are sending back AuthenticationExpiredException as is
> ----------------------------------------------------------------------------
>
>                 Key: GEODE-9900
>                 URL: https://issues.apache.org/jira/browse/GEODE-9900
>             Project: Geode
>          Issue Type: Bug
>          Components: client/server
>            Reporter: Jinmei Liao
>            Priority: Major
>
> As we have discovered in GEODE-9820, some commands (especially CQ commands) 
> are not sending back `AuthenticationExpiredException` to the client with 
> MessageType.EXCEPTION, hence causing the client not wrapping it in the 
> correct form (See `AbstractOp around L300), We need to:
> 1. go over all the commands and find out what commands are NOT handling the 
> `AuthenticationExpiredExcpetion` correctly.
> 2. fix these commands, catch `AuthenticationExpiredException` specifically 
> and do a `writeException` (leave all other exception handling intact)
> 3. write tests to make sure we are sending the exception to the client.
> #1 would help us determine how many commands are in need of this fix and size 
> this story correctly.



--
This message was sent by Atlassian Jira
(v8.20.1#820001)

Reply via email to