github-starred/n8n
2
0
Fork 0
mirror of https://github.com/n8n-io/n8n.git synced 2025-12-05 19:27:26 -06:00
Code Issues 1.3k Packages Projects Releases 100 Wiki Activity

[PR #22053] fix(core): Update state column type (no-changelog) #22015

New Issue
Open
opened 2025-11-20 06:22:43 -06:00 by GiteaMirror · 0 comments
GiteaMirror commented 2025-11-20 06:22:43 -06:00
Owner
Copy Link

📋 Pull Request Information

Original PR: https://github.com/n8n-io/n8n/pull/22053
Author: @RicardoE105
Created: 11/19/2025
Status: 🔄 Open

Base: master ← Head: ado-4444-bug-fix-issue-doing-oauth-flow-for-some-instances

📝 Commits (5)

  • 1e6c9ac update state column type
  • fb6383e update migration
  • 97e1b72 fix migration
  • 8b895e8 fix migration
  • 9ffffa8 Merge branch 'master' into ado-4444-bug-fix-issue-doing-oauth-flow-for-some-instances

📊 Changes

4 files changed (+75 additions, -0 deletions)

View changed files

➕ packages/@n8n/db/src/migrations/common/1763572724000-ChangeOAuthStateColumnToUnboundedVarchar.ts (+69 -0)
📝 packages/@n8n/db/src/migrations/mysqldb/index.ts (+2 -0)
📝 packages/@n8n/db/src/migrations/postgresdb/index.ts (+2 -0)
📝 packages/@n8n/db/src/migrations/sqlite/index.ts (+2 -0)

📄 Description

Summary

This column is generated by Lovable (the client), and it appears they use some instance metadata to calculate it. Because of this, the flow works in some instances but not in others, as the state value can exceed the initial VARCHAR(256) limit. This is why it worked fine during the internal testing of the feature.

Related Linear tickets, Github issues, and Community forum posts

https://linear.app/n8n/issue/ADOP-10/bug-fix-issue-doing-oauth-flow-for-some-instances

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)

🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/n8n-io/n8n/pull/22053 **Author:** [@RicardoE105](https://github.com/RicardoE105) **Created:** 11/19/2025 **Status:** 🔄 Open **Base:** `master` ← **Head:** `ado-4444-bug-fix-issue-doing-oauth-flow-for-some-instances` --- ### 📝 Commits (5) - [`1e6c9ac`](https://github.com/n8n-io/n8n/commit/1e6c9aca9ab54f3f65e8c01ff872fc4ddab38bb7) update state column type - [`fb6383e`](https://github.com/n8n-io/n8n/commit/fb6383e42b627b14afd11a40a6b8f523294a099b) update migration - [`97e1b72`](https://github.com/n8n-io/n8n/commit/97e1b7289075ee910654062e2ddefff1563b323d) fix migration - [`8b895e8`](https://github.com/n8n-io/n8n/commit/8b895e8b1aa0a89de33bfbfdbc41da4ff9ac56c2) fix migration - [`9ffffa8`](https://github.com/n8n-io/n8n/commit/9ffffa8ece085000fc559644cae9bb09dc72ad6b) Merge branch 'master' into ado-4444-bug-fix-issue-doing-oauth-flow-for-some-instances ### 📊 Changes **4 files changed** (+75 additions, -0 deletions) <details> <summary>View changed files</summary> ➕ `packages/@n8n/db/src/migrations/common/1763572724000-ChangeOAuthStateColumnToUnboundedVarchar.ts` (+69 -0) 📝 `packages/@n8n/db/src/migrations/mysqldb/index.ts` (+2 -0) 📝 `packages/@n8n/db/src/migrations/postgresdb/index.ts` (+2 -0) 📝 `packages/@n8n/db/src/migrations/sqlite/index.ts` (+2 -0) </details> ### 📄 Description ## Summary This column is generated by Lovable (the client), and it appears they use some instance metadata to calculate it. Because of this, the flow works in some instances but not in others, as the state value can exceed the initial VARCHAR(256) limit. This is why it worked fine during the internal testing of the feature. ## Related Linear tickets, Github issues, and Community forum posts https://linear.app/n8n/issue/ADOP-10/bug-fix-issue-doing-oauth-flow-for-some-instances ## Review / Merge checklist - [x] PR title and summary are descriptive. ([conventions](../blob/master/.github/pull_request_title_conventions.md)) <!-- **Remember, the title automatically goes into the changelog. Use `(no-changelog)` otherwise.** --> - [ ] [Docs updated](https://github.com/n8n-io/n8n-docs) or follow-up ticket created. - [ ] Tests included. <!-- A bug is not considered fixed, unless a test is added to prevent it from happening again. A feature is not complete without tests. --> - [ ] PR Labeled with `release/backport` (if the PR is an urgent fix that needs to be backported) --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
GiteaMirror added the
pull-request
 label 2025-11-20 06:22:43 -06:00
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
Backlog
bug-report
Bump
Bumped
closed:duplicate
closed:enhancement/feature
closed:incomplete-template
closed:info
closed:non-english
closed:support-issue
closed:working-as-expected
Duplicate
in linear
In Review
n8n team
Needs Feedback
Needs Feedback:Unable to reproduce
node/improvement
node/issue
node/new
pull-request
Mirrored from GitHub Pull Request
Released
Stale
status:complete
status:in-linear
status:pending-assignment
status:team-assigned
team:adore
team:ai
team:cats
team:nodes
team:payday
triage:complete
triage:in-progress
triage:needs-info
triage:needs-reproduction
triage:pending
triage:stalled
ui
Upcoming Release
No Label
pull-request
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
GiteaMirror ninjasurge
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/n8n#22015
No description provided.
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?