Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Rewrite livesync implementation #929

Merged
merged 7 commits into from
Feb 12, 2018

Conversation

Plamen5kov
Copy link
Contributor

@Plamen5kov Plamen5kov commented Jan 23, 2018

Related Issue

problem
livesync doesn't work the same way on all devices because of permission changes in upper API levels

solution
make livesync work with file content passing through unix sockets

expected behavior
files that are pushed to socket are created or deleted, depending on operation

Protocol:
create: (operation)(fileNameLength)(fileName)(fileContentLength)(fileContent)
Example of message sent through socket: 800007./a.txt0000000011fileContent

delete: (operation)(fileNameLength)(fileName)
Example of message sent through socket: 700003./a

Create example explained:
operation: 8 - create, 7 - delete (8)
fileNamelength: exactly 5 bytes (00007)
fileName: relative to app folder (./a.txt)
fileContentLength: exactly 10 bytes (0000000011)
fileContent: byte buffer (fileContent)

This PR is based on this branch.

@ns-bot
Copy link

ns-bot commented Jan 23, 2018

💔

@Plamen5kov Plamen5kov force-pushed the plamen5kov/parse-buffers branch from f41d06d to 7297e59 Compare January 23, 2018 12:26
@ns-bot
Copy link

ns-bot commented Jan 23, 2018

💔

@Plamen5kov Plamen5kov changed the title Plamen5kov/parse buffers Rewrite livesync implementation Jan 23, 2018
@ns-bot
Copy link

ns-bot commented Jan 24, 2018

💔

3 similar comments
@ns-bot
Copy link

ns-bot commented Jan 25, 2018

💔

@ns-bot
Copy link

ns-bot commented Jan 25, 2018

💔

@ns-bot
Copy link

ns-bot commented Jan 25, 2018

💔

@ns-bot
Copy link

ns-bot commented Jan 26, 2018

💔

@Plamen5kov Plamen5kov force-pushed the plamen5kov/parse-buffers branch from cbd2845 to 7998b18 Compare January 26, 2018 12:37
@ns-bot
Copy link

ns-bot commented Jan 26, 2018

💔

2 similar comments
@ns-bot
Copy link

ns-bot commented Jan 26, 2018

💔

@ns-bot
Copy link

ns-bot commented Jan 26, 2018

💔

@Plamen5kov Plamen5kov force-pushed the plamen5kov/parse-buffers branch from 079391b to 46a0087 Compare January 26, 2018 14:38
@ns-bot
Copy link

ns-bot commented Jan 26, 2018

💔

@Plamen5kov Plamen5kov force-pushed the plamen5kov/parse-buffers branch from 46a0087 to 09bb963 Compare January 29, 2018 14:18
@ns-bot
Copy link

ns-bot commented Jan 29, 2018

💔

@Plamen5kov Plamen5kov force-pushed the plamen5kov/parse-buffers branch from 09bb963 to 155aa2b Compare February 12, 2018 11:49
@ns-bot
Copy link

ns-bot commented Feb 12, 2018

💔

@Plamen5kov Plamen5kov merged commit 155aa2b into plamen5kov/revert Feb 12, 2018
@Plamen5kov Plamen5kov mentioned this pull request Feb 12, 2018
@Natalia-Hristova Natalia-Hristova added this to the 4.0.0 (TBD) milestone Feb 20, 2018
@etabakov etabakov deleted the plamen5kov/parse-buffers branch March 14, 2018 07:31
@vtrifonov vtrifonov modified the milestones: 4.0.0, 4.2.0 Jul 26, 2018
# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants