Visitar URL original
[ticket/17714] Handle avatar extension case-insensitively in avatar c… by rubencm · Pull Request #7039 · phpbb/phpbb · GitHub
Skip to content

[ticket/17714] Handle avatar extension case-insensitively in avatar c… - #7039

Open
rubencm wants to merge 1 commit into
phpbb:masterfrom
rubencm:ticket/17714
Open

rubencm wants to merge 1 commit into
phpbb:masterfrom
rubencm:ticket/17714

Conversation

@rubencm

@rubencm rubencm commented Sep 17, 2026 •

Copy link
Copy Markdown
Member

…ontroller

Ensure avatar extensions are converted to lowercase before validating against allowed extensions so avatars with uppercase extensions are served properly.

PHPBB-17714

Checklist:

  • Correct branch: master for new features; 3.3.x for fixes
  • Tests pass
  • Code follows coding guidelines: master and 3.3.x
  • Commit follows commit message format

Tracker ticket:

https://tracker.phpbb.com/browse/PHPBB-17714

@rubencm
rubencm marked this pull request as draft September 18, 2026 08:40
@rubencm
rubencm marked this pull request as ready for review September 18, 2026 14:52
@marc1706

marc1706 commented Oct 3, 2026

Copy link
Copy Markdown
Member

Should we maybe also handle files that might have been saved with uppercase extension in extension_guesser::map_extension_to_type()? Right now this does not properly handle uppercase extensions.

@marc1706 marc1706 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

see comment

…ontroller

Ensure avatar extensions are converted to lowercase before validating
against allowed extensions so avatars with uppercase extensions are
served properly.

PHPBB-17714
@rubencm

rubencm commented Oct 31, 2026

Copy link
Copy Markdown
Member Author

see comment

done

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.

2 participants