Skip to content

GitLab

  • Projects
  • Groups
  • Snippets
  • Help
    • Loading...
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in / Register
briar
briar
  • Project overview
    • Project overview
    • Details
    • Activity
    • Releases
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 700
    • Issues 700
    • List
    • Boards
    • Labels
    • Service Desk
    • Milestones
  • Merge Requests 12
    • Merge Requests 12
  • CI / CD
    • CI / CD
    • Pipelines
    • Jobs
    • Schedules
  • Operations
    • Operations
    • Incidents
    • Environments
  • Analytics
    • Analytics
    • CI / CD
    • Repository
    • Value Stream
  • Wiki
    • Wiki
  • Members
    • Members
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • briar
  • briarbriar
  • Issues
  • #1492

Closed
Open
Opened Jan 04, 2019 by akwizgran@akwizgranOwner

Race between sending and validating private message attachments

When writing a private message, we preview the attachments and remove any attachments that can't be loaded (either because the content provider supplies a MIME type with no known extension, or because Glide can't load the preview, or, in the case of a single attachment, because we can't determine the image size). If the send button is clicked before an invalid attachment has been removed from the list, the invalid attachment will be sent. At the moment this causes two problems:

  • Inconsistent behaviour depending on how quickly the attachments load and the user clicks the send button
  • False UI feedback (the "Could not attach image" toast is shown but the attachment is sent)

Once the backend for image attachments has been implemented (#1434 (closed)), we may have additional problems:

  • The adapter item created for the message may not match the attachments that were actually sent
  • We may not be able to display our own messages if invalid attachments were sent

Subtask of #1438 (closed).

Assignee
Assign to
Android 1.3
Milestone
Android 1.3
Assign milestone
Time tracking
None
Due date
None
Reference: briar/briar#1492