Skip to content

ECHO-1032 fix: an upload never fails because of its file's name - #1346

Merged
ussaama merged 1 commit into
mainfrom
fix/long-upload-file-names
Oct 26, 2026
Merged

ussaama merged 1 commit into
mainfrom
fix/long-upload-file-names

Conversation

@ussaama

@ussaama ussaama commented Oct 26, 2026

Copy link
Copy Markdown
Contributor

A long file name pushed the stored chunk path past varchar(255) and the upload failed after the file arrived ("Je bestand is binnengekomen, maar we konden het niet opslaan").

  • conversation_chunk.path and conversation.participant_name are text (0042, no rows rewritten)
  • an upload's key keeps at most 80 safe characters of the name plus its extension, so names with "..", spaces or decomposed accents work too
  • initiate clips the participant name and user agent to 255 characters

A long file name pushed the stored chunk path past varchar(255) and the
upload failed after the file arrived ("Je bestand is binnengekomen, maar
we konden het niet opslaan").

- conversation_chunk.path and conversation.participant_name are text
  (0042, no rows rewritten)
- an upload's key keeps at most 80 safe characters of the name plus its
  extension, so names with "..", spaces or decomposed accents work too
- initiate clips the participant name and user agent to 255 characters
@linear

linear Bot commented Oct 26, 2026

Copy link
Copy Markdown

ECHO-1032

@ussaama
ussaama merged commit f9f043d into main Oct 26, 2026
39 of 41 checks passed
@ussaama
ussaama deployed to pr-preview October 26, 2026 13:39 — with GitHub Actions Active
@github-actions

Copy link
Copy Markdown
Contributor

Now on staging, deployed 2026-10-26 13:52 UTC.

@github-actions

Copy link
Copy Markdown
Contributor

Released in v3.5.2.

This branch was successfully deployed

1 active deployment
pr-preview — 20e3d5cc Deployed Oct 26, 2026 by ussaama via 51-teardown-pr-preview #1022
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