-
-
Notifications
You must be signed in to change notification settings - Fork 20
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
FEATURE: Inter-active Menus #133
Comments
This could also be extended to configuring |
build
and inline
menu driven interactionbuild
and inline
interactive menus
TODO:
|
@houmie This should help you out :-) |
This sounds promising. I will test this soon. Well done! |
Hello mate, I have just started the testing, the last commit on master is Do I still have to do Thanks |
If you are only testing then you can do either.
|
Great. I will test everything and report here: Removal:
Option 1:
and option 2:
However I will now focus more on option 3 and 4, Since the two top options are rather legacy. ;-) |
option 3:
option 4:
At this point I don't know if I should Ctrl+C and go back to create the group before hand via |
I have carried on, let see if the group is created for me:
May I ask what is the purpose of tying down the key to the MAC address of the server? I suppose this could be useful when you want to lock the key only to one device. Meaning only that device can connect to the server. If that key leaks, and someone would use it on another device, then it will not be able to connect. Am I correct? |
I carried on and left the Mac address blank, but it led to the first error:
The filename has to lose |
You are trying to create a client TLS-Crypt-v2 key and you are asked for the eg: You would enter: When you build a server TLS-Crypt-V2 key you will not be asked for hardware-addresses. |
Regarding the Thanks for testing and feedback! TLS-Crypt-V2 client keys (and sub-keys) are the very point of Easy-TLS, so I will improve that interactive section to make things more clear. |
Also, you can verify the client TLS-Crypt-V2 metadata by opening the file for it in The metadata included in the client inline file contains no security sensitive data. The metadata is protected by the TLS-Crypt-v2 key itself. It connot be read or tampered with by a client. |
First, thank you for your help, time and feedback. User feedback is always the best motivator! For that reason, I have decided to take a slightly different approach. Easy-TLS is useful for a moment to inline etc. but it is (or could be) very useful for TLS Crypt V2 key management. However, there is no established standard for me to work to, except to try to emulate something like full X509. Instead, by focusing on the interactive menus, I can educate the user as to these new terms at the same time as using them. Hopefully, by clearing away the confusion of all the command options, the script will be easier to use. If you do have questions or ideas please let me know but for now you can relax ;-) |
No problem my friend. I love this project and happy to help testing it whenever you need it. :-) |
That is the idea. Each client key is unique and so the initial handshake should look different to a scanner, if a different client key is used. Thanks for testing and feedback! |
Mhhh no, it doesn't react to it and shows the help menu. Can't find |
If you need anything else specifically tested, let me know. :-) |
Try like so:
And using short options:
Line 296 in ac4158c
Line 305 in ac4158c
Also, you should not need to specify |
I would like to know your feedback regarding each of the inter-active menus. |
Ah yes. Apologies. I did something silly. My SSH session was broken, and when I went back to server I forgot to set $CLIENT and $SERVER variables, and hence it didn't work. I thought it was because of I confirm |
As per the issue title: Add But I am only teasing you. All your help and feedback is invaluable! 1 user test is infinity % more than 0 user tests 💯 |
Alrighty: Let's test This is the case where the server crypt2 key already exists:
I have now renamed it and tried again:
I'm now looking into the other menus. |
Actually Now let's try building the client.
This was also very straight forward. Both files look ok. |
I found one bug it seems:
:) |
You have inadvertently found two bugs! The one above is fixed. However, by your action:
You have exposed a weakness in the indexing process, which I completely over-looked! And now I am going to have to write a new layer of verification to stop that happening in future! The problem is, by renaming the server key, you have effectively destroyed the integrity of the inline-index .. Thanks! 🗡️ But seriously, thanks for testing and feedback, your help is invaluable. 🍻 Edit: FTR, the way Easy-TLS is supposed to work is to use: |
No problem. Any time. :) |
Good morning, Alrighty, let's go through some scenarios:
Build
|
inline without having a client in place I'm not sure about Diffy-Hellman. I don't think I've ever used it before. What benefit would be there in sense of Crypt-2?
|
Creating a client abruptly ends when selecting don't create an inline:
|
Hi,
I agree with you that inlining a client should not bother with Diffe-Hellman and that is easy to fix. I guess there are still some sharp edges which can be smoothed off but your testing did not reveal any errors. Unless I missed it ? Thank you so much for your time and feedback, having a second opinion is invaluable! |
@houmie If you get time, please let me know if it now looks any better. |
Hey @TinCanTech, No worries. Happy to help. Let me pull the latest and fire it off again. |
|
First entry: DRFTGY Only then it comes back complaining that first entry was invalid. Then it breaks the loop. No chance to recover. Not a biggie at all. Just mentioning. |
That is exactly why I ask for your feedback ;-) I can do the Thanks again 👍 |
If you fancy having another shot, I really appreciate your input. The I've tested it fairly thoroughly but there is always something I've over looked ;-) |
Yeah sure, no problem. Sorry I fell asleep last night. I will report back as soon as I can get to it again. :-) Yeah it's easier for someone else to test the code, than yourself. Otherwise you follow the established known paths and won't find the issues. |
Only minor changes to other code. Signed-off-by: Richard Bonhomme <[email protected]>
build
and inline
interactive menus
EG:
./easytls build
The text was updated successfully, but these errors were encountered: