From 6c96a47f13ebcb790ee8de7cbef8549336fa57ed Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Adrien=20B=C3=A9raud?= <adrien.beraud@savoirfairelinux.com>
Date: Wed, 16 Oct 2019 14:27:54 -0400
Subject: [PATCH] c wrapper: update dhtcnode, add dht_op_token_delete

---
 c/opendht.cpp    | 46 ++++++++++++++++++----------------------------
 tools/dhtcnode.c | 34 ++++++++++++++++++++++++++++++----
 2 files changed, 48 insertions(+), 32 deletions(-)

diff --git a/c/opendht.cpp b/c/opendht.cpp
index 408b640b..13819c35 100644
--- a/c/opendht.cpp
+++ b/c/opendht.cpp
@@ -85,8 +85,7 @@ int dht_publickey_unpack(dht_publickey* pk, const uint8_t* dat, size_t dat_size)
     return 0;
 }
 
-int dht_publickey_pack(dht_publickey* pk, char* out, size_t* outlen)
-{
+int dht_publickey_pack(dht_publickey* pk, char* out, size_t* outlen) {
     return gnutls_pubkey_export(reinterpret_cast<dht::crypto::PublicKey*>(pk)->pk, GNUTLS_X509_FMT_DER, out, outlen);
 }
 
@@ -102,32 +101,27 @@ dht_pkid dht_publickey_get_long_id(const dht_publickey* pk) {
     return h;
 }
 
-bool dht_publickey_check_signature(const dht_publickey* pk, const char* data, size_t data_size, const char* signature, size_t signature_size)
-{
+bool dht_publickey_check_signature(const dht_publickey* pk, const char* data, size_t data_size, const char* signature, size_t signature_size) {
     return reinterpret_cast<const dht::crypto::PublicKey*>(pk)->checkSignature((const uint8_t*)data, data_size, (const uint8_t*)signature, signature_size);
 }
 
-dht_blob* dht_publickey_encrypt(const dht_publickey* pk, const char* data, size_t data_size)
-{
+dht_blob* dht_publickey_encrypt(const dht_publickey* pk, const char* data, size_t data_size) {
     auto rdata = new dht::Blob;
     *rdata = reinterpret_cast<const dht::crypto::PublicKey*>(pk)->encrypt((const uint8_t*)data, data_size);
     return (dht_blob*)rdata;
 }
 
-dht_privatekey* dht_privatekey_generate(unsigned key_length_bits)
-{
+dht_privatekey* dht_privatekey_generate(unsigned key_length_bits) {
     if (key_length_bits == 0)
         key_length_bits = 4096;
     return reinterpret_cast<dht_privatekey*>(new dht::crypto::PrivateKey(dht::crypto::PrivateKey::generate(key_length_bits)));
 }
 
-dht_privatekey* dht_privatekey_import(const uint8_t* dat, size_t dat_size, const char* password)
-{
+dht_privatekey* dht_privatekey_import(const uint8_t* dat, size_t dat_size, const char* password) {
     return reinterpret_cast<dht_privatekey*>(new dht::crypto::PrivateKey(dat, dat_size, password));
 }
 
-dht_publickey* dht_privatekey_get_publickey(const dht_privatekey* key)
-{
+dht_publickey* dht_privatekey_get_publickey(const dht_privatekey* key) {
     return reinterpret_cast<dht_publickey*>(new dht::crypto::PublicKey(reinterpret_cast<const dht::crypto::PrivateKey*>(key)->getPublicKey()));
 }
 
@@ -140,26 +134,22 @@ void dht_runner_delete(dht_runner* runner) {
     delete reinterpret_cast<dht::DhtRunner*>(runner);
 }
 
-void dht_runner_run(dht_runner* r, in_port_t port)
-{
+void dht_runner_run(dht_runner* r, in_port_t port) {
     auto runner = reinterpret_cast<dht::DhtRunner*>(r);
     runner->run(port, {}, true);
 }
 
-void dht_runner_ping(dht_runner* r, struct sockaddr* addr, socklen_t addr_len)
-{
+void dht_runner_ping(dht_runner* r, struct sockaddr* addr, socklen_t addr_len) {
     auto runner = reinterpret_cast<dht::DhtRunner*>(r);
     runner->bootstrap(dht::SockAddr(addr, addr_len));
 }
 
-void dht_runner_bootstrap(dht_runner* r, const char* host, const char* service)
-{
+void dht_runner_bootstrap(dht_runner* r, const char* host, const char* service) {
     auto runner = reinterpret_cast<dht::DhtRunner*>(r);
     runner->bootstrap(host, service);
 }
 
-void dht_runner_get(dht_runner* r, const dht_infohash* h, dht_get_cb cb, dht_done_cb done_cb, void* cb_user_data)
-{
+void dht_runner_get(dht_runner* r, const dht_infohash* h, dht_get_cb cb, dht_done_cb done_cb, void* cb_user_data) {
     auto runner = reinterpret_cast<dht::DhtRunner*>(r);
     auto hash = reinterpret_cast<const dht::InfoHash*>(h);
     runner->get(*hash, [cb,cb_user_data](std::shared_ptr<dht::Value> value){
@@ -170,8 +160,7 @@ void dht_runner_get(dht_runner* r, const dht_infohash* h, dht_get_cb cb, dht_don
     });
 }
 
-dht_op_token* dht_runner_listen(dht_runner* r, const dht_infohash* h, dht_value_cb cb, void* cb_user_data)
-{
+dht_op_token* dht_runner_listen(dht_runner* r, const dht_infohash* h, dht_value_cb cb, void* cb_user_data) {
     auto runner = reinterpret_cast<dht::DhtRunner*>(r);
     auto hash = reinterpret_cast<const dht::InfoHash*>(h);
     auto fret = new std::future<size_t>;
@@ -185,16 +174,18 @@ dht_op_token* dht_runner_listen(dht_runner* r, const dht_infohash* h, dht_value_
     return (dht_op_token*)fret;
 }
 
-void dht_runner_cancel_listen(dht_runner* r, const dht_infohash* h, dht_op_token* t)
-{
+void dht_runner_cancel_listen(dht_runner* r, const dht_infohash* h, dht_op_token* t) {
     auto runner = reinterpret_cast<dht::DhtRunner*>(r);
     auto hash = reinterpret_cast<const dht::InfoHash*>(h);
     auto token = reinterpret_cast<std::future<size_t>*>(t);
     runner->cancelListen(*hash, std::move(*token));
 }
 
-void dht_runner_put(dht_runner* r, const dht_infohash* h, const dht_value* v, dht_done_cb done_cb, void* cb_user_data)
-{
+void dht_op_token_delete(dht_op_token* token) {
+    delete reinterpret_cast<std::future<size_t>*>(token);
+}
+
+void dht_runner_put(dht_runner* r, const dht_infohash* h, const dht_value* v, dht_done_cb done_cb, void* cb_user_data) {
     auto runner = reinterpret_cast<dht::DhtRunner*>(r);
     auto hash = reinterpret_cast<const dht::InfoHash*>(h);
     auto value = reinterpret_cast<const ValueSp*>(v);
@@ -204,8 +195,7 @@ void dht_runner_put(dht_runner* r, const dht_infohash* h, const dht_value* v, dh
     });
 }
 
-void dht_runner_shutdown(dht_runner* r, dht_shutdown_cb done_cb, void* cb_user_data)
-{
+void dht_runner_shutdown(dht_runner* r, dht_shutdown_cb done_cb, void* cb_user_data) {
     auto runner = reinterpret_cast<dht::DhtRunner*>(r);
     runner->shutdown([done_cb, cb_user_data](){
         if (done_cb)
diff --git a/tools/dhtcnode.c b/tools/dhtcnode.c
index 9a0b2f79..8ea9bb61 100644
--- a/tools/dhtcnode.c
+++ b/tools/dhtcnode.c
@@ -1,30 +1,56 @@
-#include "c/opendht_c.h"
+#include <c/opendht_c.h>
+
 #include <stdio.h>
+#include <string.h>
+#include <unistd.h>
+
+bool dht_value_callback(const dht_value* value, bool expired, void* user_data)
+{
+    dht_runner* runner = (dht_runner*)user_data;
+    dht_data_view data = dht_value_get_data(value);
+    printf("Value callback %s: %.*s.\n", expired ? "expired" : "new", (int)data.size, data.data);
+}
 
 bool dht_get_callback(const dht_value* value, void* user_data)
 {
     dht_runner* runner = (dht_runner*)user_data;
-    printf("Get callback.");
+    dht_data_view data = dht_value_get_data(value);
+    printf("Get callback: %.*s.\n", (int)data.size, data.data);
 }
 
 bool dht_done_callback(bool ok, void* user_data)
 {
     dht_runner* runner = (dht_runner*)user_data;
-    printf("Done callback. %s", ok ? "Success !" : "Failure :-(");
+    printf("Done callback. %s\n", ok ? "Success !" : "Failure :-(");
 }
 
 int main()
 {
     dht_runner* runner = dht_runner_new();
+    dht_runner_run(runner, 4040);
 
     dht_infohash h;
     dht_infohash_random(&h);
 
     printf("random hash: %s\n", dht_infohash_print(&h));
 
+    // Put data
+    const char* data_str = "yo, this is some data";
+    dht_value* val = dht_value_new(data_str, strlen(data_str));
+    dht_runner_put(runner, &h, val, dht_done_callback, runner);
+    dht_value_unref(val);
+
+    // Get data
     dht_runner_get(runner, &h, dht_get_callback, dht_done_callback, runner);
 
-    dht_runner_delete(runner);
+    // Listen for data
+    dht_op_token* token = dht_runner_listen(runner, &h, dht_value_callback, runner);
+
+    sleep(1);
 
+    dht_runner_cancel_listen(runner, &h, token);
+    dht_op_token_delete(token);
+
+    dht_runner_delete(runner);
     return 0;
 }
-- 
GitLab