9dcb64b53c
Initialize schema version upon import
8 years ago
cfad983d08
Consistently use `ciphertext` instead of `encrypted`
8 years ago
a61c905146
Impliclity decrypt data if `key` is present
8 years ago
94c75762b5
Add `Message.createAttachmentLoader`
8 years ago
43ab320d31
Load attachments before writing them upon export
8 years ago
7eeabb5b1a
Add precondition for `writeAttachment`
8 years ago
0faaf52b71
Import and upgrade messages upon save
8 years ago
cf7355bd2f
Rename `backupToDirectory` to `exportToDirectory`
...
For symmetry reasons.
8 years ago
bf67254cc5
Add `Message.createImporter`
8 years ago
efd673083d
Rename `writeAttachmentData` to `writeNewAttachmentData`
8 years ago
bcbd526b99
Sync read receipt configuration upon relink
...
Fixes https://github.com/signalapp/Signal-Android/issues/7535 .
8 years ago
ce8fe0d345
Add `Signal.Startup` module
8 years ago
fe0306e6be
issue-2023: move opened event out of selected conversation check
...
Going through the git history the existing logic of "dont do this if
it's already selected" was just for audio or video QOL enhancements to
not stop playing when the same conversation is selected.
8 years ago
8474f3cf7f
Use `create*` prefix to clarify curried functions
8 years ago
12cdeee7ec
Rename `count` to `numMessagesPerBatch`
8 years ago
38ac9972e8
Improve termination condition
8 years ago
5b1a0c45df
Replace `getAll` with cursor
...
This reduces the likelihood of IPC message size error.
8 years ago
ed77378293
Fix bug in termination condition
8 years ago
052fb841f6
Allow database name override in migrations
8 years ago
1e04083813
Support database upgrades in `Database.open`
8 years ago
f50ac1c99e
Reduce excessive logging about database status
8 years ago
77b1f3c505
Replace startup with attachment background migration
...
Iterates through `Message::id` index.
8 years ago
c853d1bccd
Remove whole database migrations
...
We should enable this when we are willing to run attachment migration
on startup.
8 years ago
057762806e
Add support for attachment background migration without index
8 years ago
32ac99b439
Prefer ISO date formatting
...
Previously, this was formatted using locale specific settings. This will allow
easier debugging by matching key rotation timestamps against log timestamps.
8 years ago
2e4893f4c1
Clarify implications of introducing new migrations
8 years ago
7413b787b6
Make `migrations` private
8 years ago
b07c66eaa7
Generate attachments from local files
...
This ensures our benchmarking is closer to real-world usage, e.g. images, video,
etc. that are not compressible.
8 years ago
064a40544c
Fix error message formatting
8 years ago
442f5d7295
Use `done` to match JavaScript iterators
...
See: https://developer.mozilla.org/en-US/docs/Web/JavaScript/Guide/Iterators_and_Generators
8 years ago
a4f0c5242f
Document why batch size is one
8 years ago
d59fdddb43
Improve `storage.fetch` error message
8 years ago
805031ade8
Conditionally run post-attachment migrations
...
Introduce placeholder migrations for Backbone models so they never implicitly
run migrations whenever they are `fetch`ed. We prefer to run our migrations
explicitly upon app startup and then let Backbone models be (slightly) dumb(er)
models, without inadvertently triggering migrations.
8 years ago
bfbeedab5c
Temporarily disable post-attachment migration migrations
8 years ago
d9be6a0f94
Destructure Lodash `require`s
8 years ago
b7b6195cfc
Extract IIFE into separate function
8 years ago
11f98474ba
Capture how many messages we have to process
8 years ago
0fdc1140dd
Add `Database.getCount` function
8 years ago
1f8556b049
Remove unused `createCollection`
8 years ago
c67c2a858a
Remove Backbone references for attachment migration
...
It has to run without any other migrations interfering.
8 years ago
0c40f35623
Document disadvantage of fetching messages without index
8 years ago
a4ecf1a9d6
Define constant after creating idle detector
8 years ago
d3c9de4712
Add debug module
...
Lets us generate large conversations with large attachments.
8 years ago
30037e5308
Reduce attachment migration batch size to 1
...
This prevents ‘Maximum IPC message size exceeded’ due to IDB `getAll` operation.
- https://github.com/zincbase/zincdb/issues/17
- https://cs.chromium.org/chromium/src/content/browser/indexed_db/indexed_db_database.cc?q=%22Maximum+IPC+message+size+exceeded%22&sq=package:chromium&l=1160
8 years ago
21147a20a0
Add `sleep` module
8 years ago
08f73b8420
Remove last processed index after attachment migration
8 years ago
696a144ab7
Add `settings.deleteItem`
8 years ago
a18e462817
Move migrations to `Signal.Migrations`
8 years ago
5910f84af4
Remove outdated documentation
8 years ago
efe3cd67fc
Allow attachment migration run on higher database version
8 years ago
921c3dba7c
Skip migrations that have already been applied
8 years ago
417511ffd2
Add `database.getVersion`
8 years ago
c5c94bc3ab
Extract `getMigrationVersions`
8 years ago
6aea36240d
Rename `closeDatabase` to `closeDatabaseConnection`
8 years ago
016432826b
Extract `database` and `settings` modules
8 years ago
5bea894abd
Close database connection via Backbone IDB adapter
8 years ago
f50e9ae364
Log closing connection of database
8 years ago
4ff8bc3357
Use `camelCase` for non-constructors
8 years ago
3720c3f3bb
Improve log message
8 years ago
ce5b450fdb
Log `targetSchemaVersion`
8 years ago
f7f24b5822
Log total number of processed messages
8 years ago
eca930770c
Remove hard-coded database connection settings
8 years ago
1df6dc8378
Abort processing if there are no more messages
8 years ago
85490fbc98
Disable JSHint for `background.js`
...
It doesn’t recognize `async` and I couldn’t figure out how to ignore a top-level
`async` without cascading errors.
8 years ago
d5d0eabdfd
Remove usage of `storage` module
8 years ago
8966e80284
Improve identifier names
8 years ago
3e2d575506
Document `MessageDataMigrator` module design
8 years ago
3c57dbfb56
Extract `NUM_MESSAGES_PER_BATCH`
8 years ago
85788d3c4a
Match `items` storage format to Backbone adapter
8 years ago
7de7fcf561
Avoid `no-param-reassign` violation
8 years ago
070235b59b
Implement `MessageDataMigrator.processAll`
...
Upgrades schema of all messags upon startup.
8 years ago
b8a0bc3423
Run attachment to disk migration on startup
8 years ago
178a3cc262
Reduce work for verifying transaction completion
8 years ago
b6e978f74c
Implement `MessagesDataMigrator.processAll`
8 years ago
457bf7ab9d
Add `createCollection` function
8 years ago
8ea257ad4d
Use double quotes for identifiers in error messages
8 years ago
172616ca4f
Add log message for dummy migration 18
8 years ago
579b01283e
Replace `wrapDeferred` with `deferredToPromise`
8 years ago
40c40c800a
Prefer `exports`
8 years ago
da144edc56
Manually close database connection after migration
8 years ago
106ce21c49
Remove redundant log message
8 years ago
fcd30cd919
Close database after migration
...
This is not 100% reliable as database connections are closed in a separate
thread according to the documentation:
- https://developer.mozilla.org/en-US/docs/Web/API/IDBDatabase/close
- https://stackoverflow.com/a/18639298
- 80c7a06d5c/backbone-indexeddb.js (L558-L565)
8 years ago
c765422fa1
Extract `deferredToPromise`
8 years ago
e2f1339ab9
Explicitly run post-attachment migrations
8 years ago
d7c8d33edb
Extract `runMigrations`
8 years ago
d16178638e
Split database migrations into pre- and post-attachment migration
...
- Run light-weight migrations before attachment migration.
- Run regular migrations after attachments have been moved to disk.
8 years ago
b24dad23ea
Fix search view conversation reset bug
...
When searching for an existing conversation using a phone number, it’s possible
to click on ‘Start conversation…’ and have that new dummy entry overwrite the
existing conversation.
This change ensures we are always showing a dummy conversation model that is not
part of the conversation collection. Adding it is always idempotent as it goes
through `getOrCreateAndWait`.
8 years ago
08f6886f3e
Strengthen precondition of `ConversationController.getOrCreate`
8 years ago
83c979fb84
Rename `createTemporary` to `dangerouslyCreateAndAdd`
...
Class: `ConversationController`.
This function should not be used in application code as it creates potentially
invalid `Conversation` instances in our global conversation collection. We keep
making it available for testing purposes.
8 years ago
90de9d85a3
Remove type annotation
8 years ago
4c10fcfa17
Reduce threshold until optimization appears to 1s
8 years ago
dede0f1e25
Improve user messaging during initialization
...
- We first show a localized loading message.
- If initialization takes longer than a certain threshold, we show a different
‘optimization’ message.
- If initialization is below the threshold the message change is canceled right
before the regular loading screen.
8 years ago
add19aa732
Add initialization view
8 years ago
f253fef5c6
Explicitly make `onDestroy` have `void` return type
8 years ago
9b112489cd
Upgrade 2 messages every 30 seconds
...
This results in 4 messages per minute or 240 messages per hour.
8 years ago
7c2fde4433
Add `Signal.Workflow` namespace
8 years ago
4a7cc22de3
🎨 Use `window.requestIdleCallback`
8 years ago
5c9f18e1ae
Wire up background message migration
8 years ago
4651cbc2eb
Improve ``Message.upgradeSchema` preconditions
8 years ago
ca28f3030b
Add `MessageMigration` module
8 years ago
8e762797c0
Remove attachment validation
...
Turns out `fileName` is optional and maybe even `contentType`. We should revisit
this at a later point in time.
8 years ago
09ee2d4ea2
Switch to using `requestIdleCallback`
...
Example:
https://developer.mozilla.org/en-US/docs/Web/API/Background_Tasks_API#Example
8 years ago
d13668544d
Rename `IdleListener` to `IdleDetector`
8 years ago
92ace67846
Start idle listener on startup
8 years ago
3140e4d66d
Implement basic `IdleListener`
8 years ago
0c06fff47b
Wire up `Message` / `Attachment` migration functions on startup
...
Makes `migrationContext` obsolete.
8 years ago
867f73b80a
Use double quotes for identifiers in error messages
8 years ago
70cdd2b350
Add `stringToArrayBuffer` utility
...
This way we can avoid an extra dependency.
8 years ago
2ad0fd7db6
Use Lodash `omit` instead of `delete`
8 years ago
ef2be2cf5f
Make `MessageView::onUnload` synchronous
...
Our tests rely on that.
8 years ago
b3251a8de2
Prevent potential NPE
8 years ago
9d25aa4e43
Delete attachment files on disk upon message delete
8 years ago
279b3f81c7
Add `Attachment.deleteData`
8 years ago
833a203559
Remove Vim modeline
8 years ago
e1c1b1aa72
Load attachment data before rendering
...
Prevent double rendering of attachments by multiple entries into
`MessageView::render` using promises.
8 years ago
97e3b49a36
Whitelist `js/views/message_view.js` for ESLint
8 years ago
b5721c2594
Remove Vim modeline
8 years ago
38e2468b2e
Set `schemaVersion` in `handleMessage`
8 years ago
e3dbeba963
Initialize schema version during `Message` construction
8 years ago
9a540d6d18
Load attachment data before sending message
8 years ago
8218e308cd
Improve identifier names
8 years ago
6355c54114
Return relative path from `writeAttachmentData`
...
This will make our app more robust to changes in location on the file system.
8 years ago
2cd3d5ac16
Add `Message` schema version 3
8 years ago
e0428355be
Wire up `writeAttachment`
8 years ago
aa82a2a5fb
Add `context` to `Message.upgradeSchema`
...
Allows us to pass in Electron/Node.js specific functions. This allows better
unit testing in Mocha where we don’t have access to Electron APIs.
8 years ago
ebe2a769c9
Add `Attachment.migrateDataToFileSystem`
8 years ago
1a117e3f78
Fix #2125 Prevent multiple settings modals
8 years ago
cea42bde7d
Encryption support for backup and restore
...
Also moved to the _ prefix in backup.js for all private methods exported
for testing.
8 years ago
6d8f4b7b6e
Backup: zipped messages.json, flat attachments dir
...
Backup creates, in a target directory:
- An attachments folder, with all attachments, each named for their
parent message's id - a GUID. If there is more than one attachment
in a given message, each attachment beyond the first will end with
'-N', zero-indexed.
- A file named messages.zip. It contains exactly what went to disk in
the original export code, but zipped up.
Export is now only 'light,' and in this new messages.zip format.
Import supports both the new format and the old format. If the target
directory has a messages.zip file, we'll treat it as the new format.
Next up: Encrypting attachments and the messages.zip!
8 years ago
153a22f46b
Exclude unnecessary data in export: profiles, replayable errors
8 years ago
c0c758d459
Backup: All exports are now 'light' exports
8 years ago
e583434366
Refactor: Move Backup under window.Signal
8 years ago
186268a1b1
Remove strange eslint-generated triple-parens
8 years ago
81e94c5aa3
Refactor backup.js to use async/await
8 years ago
c01b343bd4
Code formatting updates for backup.js
8 years ago
c88381efe3
Use `async` / `await` to improve readability
8 years ago
2642844c27
Rewrite migration 17 without `idb`
...
We ran into issues when doing async operations inside of an IndexedDB
`onupgradeneeded` handler. The errors were ‘The transaction is not active’ or
‘Transaction has finished’. The following documentation confirmed that
transactions are committed/terminated when control returns to the event loop:
Spec
- https://www.w3.org/TR/IndexedDB/#transaction-lifetime-concept
- https://www.w3.org/TR/IndexedDB/#upgrade-transaction-construct
Stack Overflow
- https://stackoverflow.com/a/11059085
- https://stackoverflow.com/a/27338944
Since the initial database migration is so critical, I decided to avoid `idb`
with promise support for IndexedDB for now, but will reconsider using it for
other tasks in the future to improve readability of IndexedDB code.
8 years ago
db2941cbb0
Measure duration of migration
8 years ago
a619d48fac
Update schema version history
8 years ago
0e20e8e2ea
Use `Errors.toLogFormat`
8 years ago
cd3aee962d
Upgrade message schema before sending
8 years ago
a5edbf8328
Initialize schema as first step in `upgradeSchema`
8 years ago
5d927b73e6
Use `while` loop for IDB cursor iteration
...
Previously, I messily combined promises and callbacks because I thought we
were affected by the microtask issue:
https://github.com/gasi/idb#iteratecursor--iteratekeycursor
ESLint’s `more/no-then` encouraged me to revisit this and it works as expected.
8 years ago
a76a6098c4
Simplify log statement
8 years ago
dbdf6fd880
Run version 17 migration upon startup
8 years ago
182e6ffe10
Add version 17 migration
8 years ago
752c8f97e6
🎨 Format ternaries
8 years ago
8dfaa5619f
Prefix private functions with underscore
8 years ago
ed336d31d7
Move schema versioning from `Attachment` to `Message`
8 years ago
b9e9f5e19a
🎨 `withInheritedSchemaVersion` to `inheritSchemaVersion`
8 years ago
Daniel Gasienica
colefranz
Navdeep Singh
Scott Nonnenberg
Scott Nonnenberg