-
Notifications
You must be signed in to change notification settings - Fork 1.3k
fix: change avatar column type to TEXT to support long URLs #1499
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
fix: change avatar column type to TEXT to support long URLs #1499
Conversation
|
@LinkinStars @team can you please review this pr. |
|
@maishivamhoo123 After testing, we found an error. This is an error encountered during testing with MySQL. You may need to verify whether the migration works correctly for different database types. |
@LinkinStars sir i have migrated the version to v2.0.1 and i removed the default tag from Avatar . |
|
@LinkinStars can you please for approve for workflows |
Approved. This PR will be merged in future. |
Type of change
avatar column in user table is too small #1498
Description
This PR changes the
avatarcolumn type fromVARCHARtoTEXTin theusertable.Reason:
Some Google profile image URLs exceed the previous character limit (1024/2048 chars), causing 500 errors during login or registration. Switching to
TEXTensures support for long URLs (up to 65kb).Changes
Userstruct ininternal/entity/user_entity.goto useTEXTfor the avatar field.v1.8.2migration (internal/migrations/v32.go) to alter the database column automatically.Verification
avatarcolumn now succeeds without error.