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

Secure Session example code #381

Merged
merged 3 commits into from
Feb 15, 2019
Merged

Conversation

ilammy
Copy link
Collaborator

@ilammy ilammy commented Feb 15, 2019

Here is an example of using Secure Session to implement a minimal networking application. Don't do anything fancy: a simple echo server with fixed keys and basic error handling should be enough.

We need to illustrate the following points:

  • how to initiate connections and exchange data
  • how to use both APIs of SecureSession
  • how to implement SecureSessionTransport correctly

There are also some specific comments on the session termination which has to be handled on the application layer.

Here is an example of using Secure Session to implement a minimal
networking application. Don't do anything fancy: a simple echo server
with fixed keys and basic error handling should be enough.

We need to illustrate the following points:

  - how to initiate connections and exchange data
  - how to use both APIs of SecureSession
  - how to implement SecureSessionTransport correctly

There are also some specific comments on the session termination which
has to be handled on the application layer.
@ilammy ilammy added docs 📚 Documentation, both offline and online W-RustThemis 🦀 Wrapper: Rust-Themis, Rust API, Cargo crates labels Feb 15, 2019
@ilammy
Copy link
Collaborator Author

ilammy commented Feb 15, 2019

Please give this a look to ensure that I did not write anything stupid in comments and do not teach bad practices to the users.

Copy link
Contributor

@ignatk ignatk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks OK to me, although I'm not very good at reading rust code yet...

Copy link
Collaborator

@Lagovas Lagovas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

Copy link
Contributor

@vixentael vixentael left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

On the current moment looks / feels good!

@ilammy ilammy merged commit 8fd3264 into cossacklabs:master Feb 15, 2019
@ilammy ilammy deleted the ssession-example branch February 19, 2019 19:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs 📚 Documentation, both offline and online W-RustThemis 🦀 Wrapper: Rust-Themis, Rust API, Cargo crates
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants