Skip to content

Conversation

@vsimon
Copy link

@vsimon vsimon commented Apr 23, 2025

Fixes #122

what

This exposes the disconnect_on_session_timeout option

why

To be able to have the client VPN session be disconnected after the maximum session_timeout_hours is reached.

references

#122

@vsimon vsimon requested review from a team as code owners April 23, 2025 19:53
@mergify mergify bot added the triage Needs triage label Apr 23, 2025
@vsimon vsimon force-pushed the disconnect-timeout branch 2 times, most recently from fc9295c to 0e5f709 Compare April 23, 2025 20:19
@vsimon
Copy link
Author

vsimon commented Apr 23, 2025

I just looked up that this option came in aws provider 5.90.0, adding this to versions.tf.

hashicorp/terraform-provider-aws#41621

@vsimon vsimon force-pushed the disconnect-timeout branch 2 times, most recently from adfd8c0 to 3724bde Compare April 23, 2025 20:25
@gberenice gberenice added the major Breaking changes (or first stable release) label Apr 24, 2025
@gberenice
Copy link

/terratest

@gberenice
Copy link

@vsimon thanks for your contribution! Could you please run terraform fmt against the code?

@vsimon vsimon force-pushed the disconnect-timeout branch from 3724bde to fe069eb Compare April 25, 2025 16:02
@vsimon
Copy link
Author

vsimon commented Apr 25, 2025

@vsimon thanks for your contribution! Could you please run terraform fmt against the code?

Thanks @gberenice, the format has been run.

@RoseSecurity
Copy link
Contributor

/terratest

@mergify
Copy link

mergify bot commented Jul 24, 2025

💥 This pull request now has conflicts. Could you fix it @vsimon? 🙏

@mergify mergify bot added the conflict This PR has conflicts label Jul 24, 2025
@vsimon vsimon force-pushed the disconnect-timeout branch from fe069eb to a7c121f Compare July 24, 2025 17:04
@mergify mergify bot removed the conflict This PR has conflicts label Jul 24, 2025
@vsimon
Copy link
Author

vsimon commented Jul 24, 2025

This is done

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

major Breaking changes (or first stable release) triage Needs triage

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add disconnect_on_session_timeout option

3 participants