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

TLS key can't be generated for OpenVPN client/server #81

Closed
TTPBruce opened this issue Aug 1, 2023 · 4 comments · Fixed by #89
Closed

TLS key can't be generated for OpenVPN client/server #81

TTPBruce opened this issue Aug 1, 2023 · 4 comments · Fixed by #89
Labels
bug Something isn't working
Milestone

Comments

@TTPBruce
Copy link

TTPBruce commented Aug 1, 2023

The documentation specifies that you can set tls='generate' to generate a TLS key, however this seems to be un-implemented, as the parameter is required to match the correct format and I can't seem to find anything that would parse the keyword "generate".

description: TLS Key. If set to 'generate' it will create a key if one does not already exist.

description: TLS Key. If set to 'generate' it will create a key if one does not already exist.

@genofire
Copy link
Contributor

genofire commented Sep 7, 2023

same problem, i am writing a patch now.

@opoplawski opoplawski added the bug Something isn't working label Dec 23, 2023
@opoplawski opoplawski added this to the 0.6.0 milestone Dec 23, 2023
opoplawski pushed a commit to genofire/pfsensible-core that referenced this issue Jan 4, 2024
opoplawski pushed a commit to genofire/pfsensible-core that referenced this issue Jan 4, 2024
opoplawski added a commit to genofire/pfsensible-core that referenced this issue Jan 4, 2024
@opoplawski
Copy link
Contributor

Another question is how is this to be used in practice? Presumably you need to collect the key, so I think we need to make the modules return the key when generate is used. Sound good? Other ideas?

@TTPBruce
Copy link
Author

TTPBruce commented Jan 4, 2024

Yes, this is how I envisioned using it. Our current processes involve manual copying of the keys, so instead having the module return the generated key would be useful.

@opoplawski
Copy link
Contributor

Okay, PR #107 should return any generated keys.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants