Skip to content

GitLab

  • Projects
  • Groups
  • Snippets
  • Help
    • Loading...
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
  • Sign in / Register
J
jami-lrc
  • Project overview
    • Project overview
    • Details
    • Activity
    • Releases
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
    • Locked Files
  • Issues 21
    • Issues 21
    • List
    • Boards
    • Labels
    • Service Desk
    • Milestones
    • Iterations
  • Requirements
    • Requirements
    • List
  • Security & Compliance
    • Security & Compliance
    • Dependency List
    • License Compliance
  • Operations
    • Operations
    • Incidents
  • Analytics
    • Analytics
    • Insights
    • Issue
    • Repository
    • Value Stream
  • Wiki
    • Wiki
  • Members
    • Members
  • Collapse sidebar
  • Activity
  • Graph
  • Create a new issue
  • Commits
  • Issue Boards
  • savoirfairelinux
  • jami-lrc
  • Issues
  • #449

Closed
Open
Opened May 25, 2018 by Hugo Lefeuvre@hlefeuvreDeveloper

chatview: should display received images only

Whenever an image is received by file transfer, it is stored in a user defined folder and displayed by the chatview. If the image stored at this location changes, the chatview will not notify the user about the changes and silently display the replaced image. This is a bug.

What should be done instead: Implement a mechanism that checks image integrity before displaying it. For example, the database could store a hash of the image and check it before triggering display. If the image isn't available anymore (removed, replaced) then a message placeholder should be displayed, e.g. "Image is not available anymore (deleted, replaced)".

Edited Jul 13, 2018 by Hugo Lefeuvre
To upload designs, you'll need to enable LFS and have admin enable hashed storage. More information
Assignee
Assign to
None
Milestone
None
Assign milestone
Time tracking
None
Due date
None
Reference: savoirfairelinux/ring-lrc#449