Skip to content

fix(auth): surface underlying provider errors for JWT and TwoStep auth failures#6803

Open
hassan254-prog wants to merge 1 commit into
masterfrom
wari/surface-more-two-step-errors
Open

fix(auth): surface underlying provider errors for JWT and TwoStep auth failures#6803
hassan254-prog wants to merge 1 commit into
masterfrom
wari/surface-more-two-step-errors

Conversation

@hassan254-prog

@hassan254-prog hassan254-prog commented Jul 16, 2026

Copy link
Copy Markdown
Contributor

Describe the problem and your solution

  • surface underlying provider errors for JWT and TwoStep auth failures

Review in cubic

@hassan254-prog hassan254-prog self-assigned this Jul 16, 2026

@superagent-security superagent-security Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Superagent found 3 security concern(s).

Comment thread packages/server/lib/controllers/auth/postJwt.ts
Comment thread packages/server/lib/controllers/auth/postTwoStep.ts
Comment thread packages/shared/lib/utils/error.ts Outdated
@superagent-security superagent-security Bot added the pr:flagged PR flagged for review by security analysis. label Jul 16, 2026

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues found across 4 files

Confidence score: 5/5

  • Automated review surfaced no issues in the provided summaries.
  • No files require special attention.

Re-trigger cubic

@hassan254-prog
hassan254-prog force-pushed the wari/surface-more-two-step-errors branch from cd6dca4 to 439c090 Compare July 16, 2026 15:04
@superagent-security superagent-security Bot removed the pr:flagged PR flagged for review by security analysis. label Jul 17, 2026
@NangoHQ NangoHQ deleted a comment from superagent-security Bot Jul 17, 2026
@hassan254-prog
hassan254-prog requested a review from a team July 17, 2026 09:44
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.

1 participant