Skip to content

GitLab

  • Menu
Projects Groups Snippets
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in
  • B Bluedevil
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 2
    • Issues 2
    • List
    • Boards
    • Service Desk
    • Milestones
  • Bugzilla
    • Bugzilla
  • Merge requests 6
    • Merge requests 6
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Releases
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • Plasma
  • Bluedevil
  • Merge requests
  • !58

[sendfile] Show error from send operation to the user

  • Review changes

  • Download
  • Email patches
  • Plain diff
Merged Nicolas Fella requested to merge work/sendfileerror into master Dec 24, 2021
  • Overview 3
  • Commits 1
  • Pipelines 4
  • Changes 4

If the sending fails for whatever reason we say it has failed, but don't show any details, leaving the user wondering what's wrong

Not that dumping a technical error message to the user is great UX, but not showing it is worse IMO

This way the user can at least paste it into Google and maybe get something helpful or show a screenshot in a bug report

Assignee
Assign to
Reviewer
Request review from
Time tracking
Source branch: work/sendfileerror