Ticket #335 (closed defect: fixed)

Opened 4 months ago

Last modified 4 months ago

OpenVPN up script is borken

Reported by: anonymous Assigned to: Fabian Jaeger
Priority: major Milestone: Shimo 2.1
Component: OpenVPN Version: 2.0
Keywords: Cc:

Description

I just upgraded to 2.0.4.3/361 and noticed the OpenVPN up script is broken. 

$ diff broken_openven_up.sh working_openvpn_up.sh
107,108c107,108
< d.add ServerAddresses * $vDNS $OLDDNS1 $OLDDNS2
< d.add DomainName $domain $OLDDOMAIN
---
> d.add ServerAddresses * $vDNS
> d.add DomainName $domain

Please revert back the previous working version.

Thanks,

Lonnie

Attachments

Change History

07/29/08 16:31:44 changed by Fabian Jaeger

  • status changed from new to assigned.
  • owner set to Fabian Jaeger.

(follow-up: ↓ 3 ) 07/29/08 16:59:27 changed by Fabian Jaeger

I will add both version to the next release and let the user decide which of them to use. Some users wanted the feature that the DNS servers are merged and therefore I added the new script...

(in reply to: ↑ 2 ; follow-up: ↓ 4 ) 07/29/08 18:55:58 changed by lonnie

Replying to Fabian Jaeger:

I will add both version to the next release and let the user decide which of them to use. Some users wanted the feature that the DNS servers are merged and therefore I added the new script...

Correct me if I am wrong, but in many cases the local DNS is not even reachable, for example when a "redirect-gateway def1" is pushed at the server.

Shouldn't people that want split DNS do this with a Custom Config and not with the up/down scripts?

Lonnie

(in reply to: ↑ 3 ) 08/01/08 17:49:03 changed by Fabian Jaeger

Replying to lonnie:

Correct me if I am wrong, but in many cases the local DNS is not even reachable, for example when a "redirect-gateway def1" is pushed at the server. Shouldn't people that want split DNS do this with a Custom Config and not with the up/down scripts?

That's true. But why not offer the user an easy option to merge local and remote network settings? There will be a simple checkbox if you want to enable this convenience feature...

08/03/08 20:15:26 changed by Fabian Jaeger

  • status changed from assigned to closed.
  • resolution set to fixed.

Add/Change #335 (OpenVPN up script is borken)




Change Properties
Action