refactor(auth): remove lastLogoutTimestamp from markLoggedOut method

[chsami]

No description provided.

[coderabbitai]

Caution

Review failed

The pull request is closed.

Walkthrough

A modification to the markLoggedOut() method in LoginManager.java removes the assignment to lastLogoutTimestamp. The method continues to clear the LOGIN_ATTEMPT_ACTIVE flag but no longer records when the logout occurred.

Inconsistent Logic

There appears to be a logical inconsistency: the method is named markLoggedOut() which semantically implies recording a logout event, yet it no longer updates lastLogoutTimestamp. This disconnect between the method's purpose (marking a logout) and its implementation (no longer tracking when the logout happened) may indicate unintended behavior.

✨ Finishing touches

• 📝 Generate docstrings

---

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 7d0eefa and 3688924.

📒 Files selected for processing (1)

• runelite-client/src/main/java/net/runelite/client/plugins/microbot/util/security/LoginManager.java (0 hunks)

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}