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

Add the local id and remote id in key protobuffer message #25

Open
kaiduanx opened this issue Jan 15, 2022 · 0 comments
Open

Add the local id and remote id in key protobuffer message #25

kaiduanx opened this issue Jan 15, 2022 · 0 comments

Comments

@kaiduanx
Copy link
Contributor

Now the following code decides the site id of A or B.

https://github.com/Open-QKD-Network/qkd-net/blob/issue-21-zp/qnl/key-routing-service/src/main/java/com/uwaterloo/iqc/qnl/qll/cqptoolkit/server/KeyTransferServer.java#L87

This will not work when we have more than 2 nodes in the system.

Please add local and remote as string to key protobuffer message.

https://github.com/Open-QKD-Network/qkd-net/blob/issue-21-zp/qnl/key-routing-service/src/main/proto/KeyTransfer.proto#L7 key

message Key {
bytes key = 1;
uint64 seqID = 2;
string localId = 3;
string remoteId = 4;
}

LocalId and remoteId is the id of dummy driver, it is set in the dummydriver.json when we run dummy driver.

Please figure out by yourself how to change the code in cqptoolkit to pass the local id and remote id to OpenQKDNetwork.

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

No branches or pull requests

1 participant