DNS should send ACK to maker address posting

[mojoX911]

Currently, the maker has no way of knowing if their address posting request are successfully stored by DNS.

make the DNS send back an ACK message to maker.

[Levi0804]

After updating/inserting the address:

coinswap/src/market/directory.rs

Lines 486 to 488 in 6ee6dd6

	);
	directory.updated_address_map((metadata.url, metadata.proof.bond.outpoint))?;
	}

we can send ACK and then handle reading it on the maker side. Am I correct?

[Shourya742]

After updating/inserting the address:

coinswap/src/market/directory.rs

Lines 486 to 488 in 6ee6dd6

 ); 
 directory.updated_address_map((metadata.url, metadata.proof.bond.outpoint))?; 

}
we can send ACK and then handle reading it on the maker side. Am I correct?

based on successful operation send an ACK or NACK. Just expand the result and send the payload accordingly.

[Levi0804]

if let Err(e) = directory.updated_address_map((metadata.url, metadata.proof.bond.outpoint)) {
    log::warn!(
        "NACK. Posting request by maker was rejected by DNS: {:?}",
        e
    );
    send_message(stream, &"NACK")?;
    return Err(e);
} 
log::info!("ACK. Posting request by maker was accepted by DNS.");
send_message(stream, &"ACK")?;

so something like this, right?

[Shourya742]

if let Err(e) = directory.updated_address_map((metadata.url, metadata.proof.bond.outpoint)) {
log::warn!(
"NACK. Posting request by maker was rejected by DNS: {:?}",
e
);
send_message(stream, &"NACK")?;
return Err(e);
}
log::info!("ACK. Posting request by maker was accepted by DNS.");
send_message(stream, &"ACK")?;
so something like this, right?

have response type as enum and use match

[Levi0804]

I created a PR #414 Please let me know your thoughts on it.