Skip to content
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

Remove curl dependency #9

Open
bensmoif opened this issue Sep 17, 2015 · 1 comment
Open

Remove curl dependency #9

bensmoif opened this issue Sep 17, 2015 · 1 comment

Comments

@bensmoif
Copy link
Collaborator

webdav-sync relies very tightly on command-line curl. We'd like to remove that dependency and have a built in async push and pull that does not require calls to curl.

@maboiteaspam
Copy link

As i m interested in this topic, i would like to suggest https://github.com/request/request / or https://github.com/substack/hyperquest modules to implement all http exchange logic.

IMHUnderstanding, in lib/webdav_sync.js we should keep only the public api and replace bodies with suggested modules.
The most problematic being the return value which today is the result of process.exec and won t remains the same.
Seems, to be confirmed, they are unused returns values, making the job easier.

BTW, how about using https://github.com/chalk/chalk to print pretty colors on std*** ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants