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

exit: Copy default route to ffhb_routing_table #35

Merged
merged 2 commits into from
Nov 25, 2015
Merged

Conversation

corny
Copy link
Member

@corny corny commented Nov 25, 2015

No description provided.

@@ -23,6 +23,10 @@ Role Variables
exit_openvpn_username: Username for openvpn auth
exit_openvpn_password: Passwort for openvpn auth

### default (dangerous)
Copy link
Contributor

Choose a reason for hiding this comment

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

Warum ist das gefaehrlich? Magst du das kurz dazu schreiben?

Copy link
Member Author

Choose a reason for hiding this comment

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

Be aware of the german "Störerhaftung"!
So besser?

@@ -7,7 +7,6 @@
dest=/etc/iproute2/rt_tables
line="252 {{ ffhb_routing_table }}"
regexp="^252 "
state="{{ "absent" if ffhb_routing_table == "default" else "present" }}"
Copy link
Contributor

Choose a reason for hiding this comment

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

Whats the default here?

Copy link
Member Author

Choose a reason for hiding this comment

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

present

mortzu added a commit that referenced this pull request Nov 25, 2015
exit: Copy default route to ffhb_routing_table
@mortzu mortzu merged commit 1cb03d6 into master Nov 25, 2015
@corny corny deleted the exit_default branch November 25, 2015 14:55
mortzu added a commit that referenced this pull request Jun 11, 2022
exit: Copy default route to ffhb_routing_table
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