Skip to content

Increase avatar URL length limit to 1024 characters#32

Merged
IamMuuo merged 2 commits intomasterfrom
staging
Mar 14, 2026
Merged

Increase avatar URL length limit to 1024 characters#32
IamMuuo merged 2 commits intomasterfrom
staging

Conversation

@IamMuuo
Copy link
Copy Markdown
Contributor

@IamMuuo IamMuuo commented Mar 14, 2026

Summary by CodeRabbit

  • Bug Fixes
    • Increased the maximum length allowed for user avatar URLs to support longer image paths.

IamMuuo and others added 2 commits March 14, 2026 13:16
- Some user profile pictures from oauth are larger than 512 chars long
- Now weve doubled the length to 1024 to prevent such scenarios from happening
@IamMuuo IamMuuo merged commit a2aadf2 into master Mar 14, 2026
3 of 4 checks passed
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Mar 14, 2026

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 17394218-038f-467a-811a-5d88f0f6999d

📥 Commits

Reviewing files that changed from the base of the PR and between 926efae and 24174e9.

📒 Files selected for processing (3)
  • users/migrations/0002_alter_user_avatar_url.py
  • users/migrations/0003_alter_user_avatar_url.py
  • users/models.py

📝 Walkthrough

Walkthrough

Two Django migrations are added to modify the User model's avatar_url field, followed by a model definition update. The field is converted to a URLField with blank and null allowances, and the maximum length is increased from 500 to 1024 characters.

Changes

Cohort / File(s) Summary
User avatar_url field migrations
users/migrations/0002_alter_user_avatar_url.py, users/migrations/0003_alter_user_avatar_url.py
Sequential data migrations altering the avatar_url field to a URLField with blank and null values, culminating in a max_length increase to 1024 characters.
User model definition
users/models.py
Updated User.avatar_url field max_length from 500 to 1024, maintaining null and blank permissions.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~5 minutes

Poem

🐰 A field grows longer, like carrots in spring,
From five hundred to one-zero-two-four, ⬆️
Two migrations dance, then the model's in sync,
URLs flourish with plenty more room to explore!
— The CodeRabbit Warren

✨ Finishing Touches
  • 📝 Generate docstrings (stacked PR)
  • 📝 Generate docstrings (commit on current branch)
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch staging
📝 Coding Plan
  • Generate coding plan for human review comments

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

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