Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Disabling agent account doesn’t terminate active connections - Fix #5055

Open
wants to merge 16 commits into
base: develop/6
Choose a base branch
from

Conversation

SudhanshuBawane
Copy link

closed 2608

Description

To fix the behavior of the agent not getting disconnected while user is disabled.

Fixed

The agent is now getting disconnected as soon as the user is disabled unlike before.

Added

  • The new watcher configuration for user has been added.
  • The session code has been modified to consume the new user configuration into the bus.

Change verification

To verify the change as normal run backend and agent, as soon as both are running and connected, Disable the user via sensuctl command. In backend you will see a shutting down a agent session prompt and in agent will fail to do a handshake with bad credential prompt.

SudhanshuBawane and others added 16 commits March 7, 2024 18:47
Signed-off-by: SudhanshuBawane <[email protected]>
Signed-off-by: SudhanshuBawane <[email protected]>
Signed-off-by: SudhanshuBawane <[email protected]>
Signed-off-by: SudhanshuBawane <[email protected]>
Signed-off-by: SudhanshuBawane <[email protected]>
Signed-off-by: SudhanshuBawane <[email protected]>
Signed-off-by: SudhanshuBawane <[email protected]>
Signed-off-by: SudhanshuBawane <[email protected]>
Signed-off-by: SudhanshuBawane <[email protected]>
Signed-off-by: SudhanshuBawane <[email protected]>
Signed-off-by: SudhanshuBawane <[email protected]>
Signed-off-by: SudhanshuBawane <[email protected]>
Signed-off-by: SudhanshuBawane <[email protected]>
Signed-off-by: SudhanshuBawane <[email protected]>
Signed-off-by: SudhanshuBawane <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants