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

Combine Informations #1

Open
thomas-v2 opened this issue Feb 10, 2024 · 6 comments
Open

Combine Informations #1

thomas-v2 opened this issue Feb 10, 2024 · 6 comments

Comments

@thomas-v2
Copy link

Hi,

I'm working also on a driver, which is only supporting TLS (using OpenSSL). But I've done most of the high level stuff (low level is in rough shape). Maybe we can combine it? I would be willing to change the license from LPGL to a more freely one.

@bonk-dev
Copy link
Owner

I'm going to upload this lib to Nuget soon so you will be able to use it (no need to change the license, since HarpoS7 is using MIT). I can help you with integrating it into your driver later on.

@mycroes
Copy link

mycroes commented Feb 13, 2024

May I chime in as well? I'm the maintainer of S7NetPlus and creator of Sally7, I'm very interested in at least getting Sally7 to a more modern protocol version. Both libraries are MIT licensed.

@thomas-v2
Copy link
Author

I've got a running version, but it does only support TLS supporting CPUs:

https://github.com/thomas-v2/S7CommPlusDriver

Even if you want to connect to a password protected CPU with TLS, you'd need parts of the authentication used in this HarpoS7 library. CPU password authentication uses a different string to build another derivative key, as far as I know.

@bonk-dev
Copy link
Owner

bonk-dev commented Feb 13, 2024

@thomas-v2 I didn't look into the password auth yet, but during my reverse-engineering sessions I did see a different derive function - I will take a look at it when I have some free time later.

@mycroes Sure! I do know S7NetPlus - I've contributed some code myself. I'm working on releasing an actual public key store so that anyone can download the lib from nuget and use it as is, without dumping anything.

@mycroes
Copy link

mycroes commented Feb 14, 2024

@bonk-dev Yeah I recognized your name and profile pic. I think I've also been in (short) contact with Thomas in 2019, but back then he didn't feel much for supporting work on a C# implementation 😉. Anyway, I still have a lot of catching up to do on all this, I currently have no clue how different S7ComPlus is from the legacy S7 protocol, but it's something I'd very much like to support.

@bonk-dev
Copy link
Owner

@thomas-v2 I finally found some time to look into password auth. I was planning on making a doc on how it works but I'm drained for now. I pushed my changes to main and uploaded the new version to nuget.

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

3 participants