Skip to content

Sn/3.8.1#423

Merged
vishwab1 merged 4 commits into
release-3.8.1from
sn/3.8.1
May 22, 2026
Merged

Sn/3.8.1#423
vishwab1 merged 4 commits into
release-3.8.1from
sn/3.8.1

Conversation

@snehar-nd

@snehar-nd snehar-nd commented May 22, 2026

Copy link
Copy Markdown
Contributor

📋 Description

JIRA ID:

Admin post deployment issue.

✅ Type of Change

  • 🐞 Bug fix (non-breaking change which resolves an issue)

ℹ️ Additional Information

Allow Multiple logins for the role admin and superadmin.

@coderabbitai

coderabbitai Bot commented May 22, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 69593b70-a8f2-459b-9eba-4d1d85bf0f44

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch sn/3.8.1

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

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

@sonarqubecloud

sonarqubecloud Bot commented May 22, 2026

Copy link
Copy Markdown

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

@vishwab1 vishwab1 merged commit 3e0bbb7 into release-3.8.1 May 22, 2026
5 checks passed
sehjotsinghunthinkable pushed a commit to sehjotsinghunthinkable/Common-API that referenced this pull request Jun 18, 2026
@snehar-nd @claude @sehjotsinghunthinkable
Sn/3.8.1 (PSMRI#423)
f9780ce 
* fix: aam-2313 phone number leading with zero - removed zero

* fix: allow concurrent sessions for admin, superadmin, and supervisor roles

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

* fix: added admin and superadmin for the condition

---------

Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>
sehjotsinghunthinkable pushed a commit to sehjotsinghunthinkable/Common-API that referenced this pull request Jun 18, 2026
@vishwab1 @claude @sehjotsinghunthinkable
fix: use stringRedisTemplate for jti: key read/delete in concurrent s…
4c19907 
…ession logout

redisTemplate has Jackson2JsonRedisSerializer<User> as value serializer, so
reading the plain-string jti: value caused a deserialization failure (statusCode 5000).
jti: keys are written via stringRedisTemplate at login, so reads and deletes must
also use stringRedisTemplate — restoring the behaviour from commit 80fa0e5 that
was accidentally reverted in PSMRI#423.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@vishwab1 vishwab1 vishwab1 approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@snehar-nd @vishwab1