Skip to content

[client] introduce JWT refresh tokens#39

Merged
capcom6 merged 1 commit intomasterfrom
client/jwt-refresh-tokens
Mar 3, 2026
Merged

[client] introduce JWT refresh tokens#39
capcom6 merged 1 commit intomasterfrom
client/jwt-refresh-tokens

Conversation

@capcom6
Copy link
Copy Markdown
Member

@capcom6 capcom6 commented Mar 2, 2026 •
edited by coderabbitai Bot
Loading

Summary by CodeRabbit

  • New Features
    • Authentication responses now include optional refresh token support for improved session token management.

@codecov
Copy link
Copy Markdown

codecov Bot commented Mar 2, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Mar 2, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 85fddb9 and 52df4c3.

📒 Files selected for processing (1)
  • smsgateway/dto_auth.go

Walkthrough

A new RefreshToken field of type string was added to the TokenResponse struct in the authentication DTO file. The field includes a JSON tag for serialization with an optional omitempty directive.

Changes

Cohort / File(s) Summary
DTO Modifications
smsgateway/dto_auth.go		 Added RefreshToken string field to TokenResponse struct with JSON tag refresh_token,omitempty for optional token serialization.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The pull request title '[client] introduce JWT refresh tokens' accurately describes the main change: adding a RefreshToken field to the TokenResponse struct to support JWT refresh token functionality.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

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.

@capcom6 capcom6 merged commit 22cfdb8 into master Mar 3, 2026
8 checks passed
@capcom6 capcom6 deleted the client/jwt-refresh-tokens branch March 3, 2026 03:09
@coderabbitai coderabbitai Bot mentioned this pull request Mar 3, 2026
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@capcom6