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

Feature: Add few missing entry support to openvpn_server #152

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Inocious
Copy link

@Inocious Inocious commented Jan 8, 2025

This commit will add remote_cert_tls, inactive_seconds, keepalive_interval, keepalive_timeout and exit_notify features to pfsensible.core.pfsense_openvpn_server module

Copy link
Contributor

@opoplawski opoplawski left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the PR. We'll need version_added doc fields for the new options - let's go with 0.6.2 for now. Also a changelog fragment is needed.

@@ -91,6 +91,10 @@
description: Enforce a match between the common name of the client certificate and the username given at login.
default: false
type: bool
remote_cert_tls:
description: Enforce that only hosts with a client certificate can connect (EKU: "TLS Web Client Authentication").
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This may need to get quoted differently to be valid YAML.

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