Skip to content
Snippets Groups Projects

batches: File upload error handling

Merged Erik Seliger requested to merge rc/batches-file-name-error into main

Created by: Piszmog

Two recent issues were found.

  • When a file exceeded the size limit, the file name was not in the error. This made it hard to tell which file is causing the issue
  • The error being returned was still set after logging as a warn. This made it so the program still errored out

Thread of context.

Test plan

Existing tests cover changes.

Screenshot 2023-08-07 at 13 41 54

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
Please register or sign in to reply
Loading