-
Notifications
You must be signed in to change notification settings - Fork 298
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
External property on kCBLDatabaseChangeNotification when CBL is remote #1100
Comments
Turns out this is already hooked up, via the |
I could use For bonus points I could also append the socket's peer IP address, like |
I think there is still one path that will not show up correctly. If you put an attachment through the REST API, it will create a new revision of a document with isExternal set to NO because it takes the putRevision path instead of the forceInsert path: https://github.com/couchbase/couchbase-lite-ios/blob/master/Source/CBL_Router+Handlers.m#L1083 https://github.com/couchbase/couchbase-lite-ios/blob/master/Source/CBL_SQLiteStorage.m#L1725 |
Also if a bulk docs request comes in with new_edits = true https://github.com/couchbase/couchbase-lite-ios/blob/master/Source/CBL_Router+Handlers.m#L285 Note that the source is passed in but never used in either case |
Steps to reproduce:
kCBLDatabaseChangeNotification
handler always have asource
URL that's null andexternal
is falseWhen the replication is CBL <-> Sync Gateway, the
external
field has the expected value but when CBL becomes the remote it's always false on the database event.The text was updated successfully, but these errors were encountered: