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

CA Private Key in Client connection Data Package? #683

Open
bobdrummond opened this issue Mar 12, 2024 · 2 comments
Open

CA Private Key in Client connection Data Package? #683

bobdrummond opened this issue Mar 12, 2024 · 2 comments
Assignees
Labels
Priority priority issue
Milestone

Comments

@bobdrummond
Copy link

Hi, I'm a new user just getting started with ATAK. I was trying to figure out how to change the connectString0 in the generated client packages, and everything I tried to change in config files was ineffective.

While digging into the packages and code
FreeTAKServer/core/util/certificate_generation.py I noticed that the CA p12 file zipped into client packages has the CA Private Key included. Is that by design or an accident from reusing _generate_certificate() on the CA?

I manually stripped the CA Private Key out of a p12 file with openssl, made a new zip, and tested on a clean client, and it seems to connect and work. Is there something I'm missing or is this an unnecessary security leak?

@brothercorvo brothercorvo added this to the 2.2 milestone Apr 21, 2024
@brothercorvo brothercorvo added the Priority priority issue label Apr 21, 2024
@brothercorvo
Copy link
Collaborator

@naman108 is this one fixed?

@brothercorvo brothercorvo modified the milestones: 2.2, 2.3 May 5, 2024
@naman108
Copy link
Collaborator

hey @bobdrummond, thanks so much for this report, could you share what version you're using, and send an example of a cert with the private key bundled because, from what I can tell from inspecting certs with openssl, the only private key is that of the client bundled in the client p12 and no private key in the second p12.

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

When branches are created from issues, their pull requests are automatically linked.

4 participants