Skip to content

fix: widen OAuth signature columns from VARCHAR(255) to VARCHAR(512)#117

Merged
hrntknr merged 1 commit intomainfrom
fix/widen-oauth-signature-columns
Mar 3, 2026
Merged

fix: widen OAuth signature columns from VARCHAR(255) to VARCHAR(512)#117
hrntknr merged 1 commit intomainfrom
fix/widen-oauth-signature-columns

Conversation

@hrntknr
Copy link
Copy Markdown
Member

@hrntknr hrntknr commented Mar 3, 2026

Summary

Widen all 7 OAuth-related GORM model fields from size:255 to size:512.
OAuth token signatures regularly exceed 255 characters (~420 chars observed
with Google OAuth + PostgreSQL), causing INSERT failures.

Fixes #111

Type of Change

  • fix: A bug fix

Related Issues

Closes #111

Co-authored-by: Alex Leonov [email protected]
Co-authored-by: Arnaud Barisain-Monrose [email protected]

OAuth token signatures regularly exceed 255 characters (~420 chars
observed with Google OAuth + PostgreSQL), causing INSERT failures:
  ERROR: value too long for type character varying(255) (SQLSTATE 22001)

Widen all 7 affected GORM model fields from size:255 to size:512.

Fixes #111

Co-authored-by: Alex Leonov <[email protected]>
Co-authored-by: Arnaud Barisain-Monrose <[email protected]>
Copilot AI review requested due to automatic review settings March 3, 2026 12:59
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR addresses PostgreSQL INSERT failures when OAuth token signatures exceed VARCHAR(255) by widening the relevant GORM model column sizes to VARCHAR(512) in the SQL repository schema.

Changes:

  • Increase GORM size from 255512 for OAuth-related primary keys and signature fields in the SQL-backed repository models.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@codecov
Copy link
Copy Markdown

codecov Bot commented Mar 3, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@hrntknr hrntknr merged commit 68437b1 into main Mar 3, 2026
12 checks passed
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.

Postgre backend doesn't reliably work

2 participants