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

Server update #30

Closed
wants to merge 7 commits into from
Closed

Server update #30

wants to merge 7 commits into from

Conversation

palainp
Copy link
Member

@palainp palainp commented Aug 19, 2021

Updated the server code. Currently able to connect to and from OpenSSH_8.0p1.

I still have a problem that I don't know how to deal with well: the guard (is_keyed new_keys_* in lib/server.ml are still being triggered every time, so far I commented on them.

I have not currently implemented Hostkey.sexp_of_pub for ED25519, the warning appears when using awa_test_client but does not seem to disturb communication too much.

@palainp
Copy link
Member Author

palainp commented Aug 25, 2021

I followed the issue regarding calls to guard to a situation where Cipher is an AEAD algorithm and therefore the negotiate function allows the Hmac to be plaintext. I corrected by adding a test in the is_plaintext function to indicate that this is a correct situation.

@hannesm
Copy link
Member

hannesm commented Aug 25, 2021

Thanks @palainp -- I'm the next 10 days away from my computer, and will try to review your changes early September.

@hannesm
Copy link
Member

hannesm commented Oct 28, 2021

thanks, this looks fine. I merged it manually (and applied a minor style comment).

@hannesm hannesm closed this Oct 28, 2021
hannesm added a commit to hannesm/opam-repository that referenced this pull request Oct 28, 2021
CHANGES:

* support rsa-sha2 and ed25519 in server code (mirage/awa-ssh#29 mirage/awa-ssh#30 @palainp)
* awa_test_client: add --key argument (mirage/awa-ssh#28 @hannesm, suggested in mirage/awa-ssh#27 by
  @dgjustice @palainp)
* Avoid deprecated Cstruct.len, avoid astring (@hannesm)
* Drop rresult dependency (mirage/awa-ssh#34 @hannesm)
@palainp palainp deleted the server-update branch January 20, 2022 08:26
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

Successfully merging this pull request may close these issues.

2 participants