nyxxxie/serenity

changes_comments should have compound foreign key (file_id, hash)

nanokatze opened this issue · 2 comments

There can exist same change hash for two or more different files. Example:

  1. you open two files with exactly same content
  2. their base changes are both 0
  3. erase first 4 bytes in both
  4. change hash is sha256(base + file_pos + change_type + data)
  5. get collision

@fst3a is this fixed with sqlalchemy changes in feature/implement_project?