[GEODE-9900] Make sure all commands are sending back AuthenticationExpiredException as is

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 1.15.0
• Fix Version/s: 1.15.0
• Component/s: client/server, security
• Labels: GeodeOperationAPI pull-request-available unreleased
• Assignee: Joris Melchior
• Reporter: Jinmei Liao
• Created: 14/Dec/21 21:19
• Updated: 22/Jun/22 20:45
• Resolved: 06/Jan/22 15:48

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.

Issue Links

• links to: GitHub Pull Request #7207