Skip to content

Conversation

@BGZStephen
Copy link
Contributor

Summary

Related Linear tickets, Github issues, and Community forum posts

Review / Merge checklist

  • PR title and summary are descriptive. (conventions)
  • Docs updated or follow-up ticket created.
  • Tests included.
  • PR Labeled with release/backport (if the PR is an urgent fix that needs to be backported)

@n8n-assistant n8n-assistant bot added core Enhancement outside /nodes-base and /editor-ui n8n team Authored by the n8n team labels Dec 8, 2025
@codecov
Copy link

codecov bot commented Dec 8, 2025

❌ 1 Tests Failed:

Tests completed Failed Passed Skipped
15529 1 15528 0
View the top 1 failed test(s) by shortest run time
OauthService generateAOauth2AuthUri should skip userId in CSRF state when skipAuthOnOAuthCallback is true
Stack Traces | 0.002s run time
Error: expect(jest.fn()).toHaveBeenCalledWith(...expected)

Expected: ObjectContaining {"cid": "1"}
Received: "user-id"

Number of calls: 1
    at Object.<anonymous> (.../oauth/__tests__/oauth.service.test.ts:1037:36)
    at processTicksAndRejections (node:internal/process/task_queues:105:5)

To view more test analytics, go to the Test Analytics Dashboard
📋 Got 3 mins? Take this short survey to help us improve Test Analytics.

@blacksmith-sh
Copy link

blacksmith-sh bot commented Dec 8, 2025

Found 1 test failure on Blacksmith runners:

Failure

Test View Logs
OauthService/
OauthService generateAOauth2AuthUri should skip userId in CSRF state when skipAuthOnOAu
thCallback is true
View Logs

Fix in Cursor

@currents-bot
Copy link

currents-bot bot commented Dec 8, 2025

E2E Tests: n8n tests passed after 10m 5.4s

🟢 576 · 🔴 0 · ⚪️ 43 · 🟣 1

View Run Details

Run Details

  • Project: n8n

  • Groups: 2

  • Framework: Playwright

  • Run Status: Passed

  • Commit: d92ad82

  • Spec files: 129

  • Overall tests: 619

  • Duration: 10m 5.4s

  • Parallelization: 9

Groups

GroupId Results Spec Files Progress
multi-main:ui 🟢 519 · 🔴 0 · ⚪️ 43 · 🟣 1 120 / 120
multi-main:ui:isolated 🟢 57 · 🔴 0 · ⚪️ 0 9 / 9


This message was posted automatically by currents.dev | Integration Settings

@BGZStephen BGZStephen changed the title feat: dynamic credentials authorization endpoint for oauth2 feat: Dynamic credentials authorization endpoint for oauth2 Dec 8, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

core Enhancement outside /nodes-base and /editor-ui n8n team Authored by the n8n team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants