Skip to content

[stable33] fix(http): avoid iconv for header ascii fallback#60722

Merged
AndyScherzinger merged 1 commit into
stable33from
backport/60649/stable33
Jun 17, 2026
Merged

[stable33] fix(http): avoid iconv for header ascii fallback#60722
AndyScherzinger merged 1 commit into
stable33from
backport/60649/stable33

Conversation

@backportbot

@backportbot backportbot Bot commented May 26, 2026

Copy link
Copy Markdown

Backport of PR #60649

iconv transliteration is locale- and config-dependent and fails silently
on some setups. UnicodeString::ascii() from symfony/string uses a built-in
transliteration table backed by symfony/polyfill-intl-normalizer, so it
works on all setups without requiring optional PHP extensions.

Assisted-by: Claude:claude-sonnet-4-6
Signed-off-by: Christoph Wurst <1374172+ChristophWurst@users.noreply.github.com>
@backportbot backportbot Bot requested review from a team, SebastianKrupinski and hamza221 as code owners May 26, 2026 07:23
@backportbot backportbot Bot requested review from Altahrim, Antreesy, ArtificialOwl, ChristophWurst, provokateurin and salmart-dev and removed request for a team May 26, 2026 07:23
@backportbot backportbot Bot added bug 3. to review Waiting for reviews regression labels May 26, 2026
@backportbot backportbot Bot added this to the Nextcloud 33.0.4 milestone May 26, 2026
@ChristophWurst ChristophWurst added 4. to release Ready to be released and/or waiting for tests to finish and removed 3. to review Waiting for reviews labels May 26, 2026
This was referenced Jun 1, 2026
@AndyScherzinger AndyScherzinger merged commit 5c0c52c into stable33 Jun 17, 2026
163 of 174 checks passed
@AndyScherzinger AndyScherzinger deleted the backport/60649/stable33 branch June 17, 2026 17:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

4. to release Ready to be released and/or waiting for tests to finish bug regression

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants