ancwrd1 / snx-rs

Open Source Linux Client For Check Point VPN Tunnels
GNU Affero General Public License v3.0
116 stars 8 forks source link

Issue when multiple DNS suffixes #15

Closed NicolasFR closed 8 months ago

NicolasFR commented 8 months ago

Hi, First, thank you very much for your work. I encountered a dns resolution problem for the vpn suffixes.

In the logs there is this message: DEBUG snx_rs::platform::linux::xfrm: Adding acquired DNS suffixes: ["XXXXX.fr;XXXX.fr"] instead of seeing ["XXXXX.fr","XXXX.fr"]

When I check resolvectl for VTI interface in dns suffixes I see that the ";" becomes a "\059" : DNS Domain: XXXXX.fr\059XXXXX.fr

I think it has to do with this line : https://github.com/ancwrd1/snx-rs/blob/70781a9e51033b06886b39826515ca00bfc993e1/src/tunnel/connector.rs#L323

By replacing the comma with a semicolon, then in the logs there are the 2 suffixes.

Can I propose a pull request ?