Page MenuHomeSolus

Updated to version 1.6.0 (released on 2017-11-15)

Authored by sankasan on Dec 27 2017, 11:34 PM.


Group Reviewers
Triage Team

Bumped to the next stable released version.

Test Plan

Connect to any FortiVPN server using the command line.

Diff Detail

R3698 openfortivpn
No Linters Available
No Unit Test Coverage

Event Timeline

sankasan created this revision.Dec 27 2017, 11:34 PM
JoshStrobl requested changes to this revision.Dec 28 2017, 6:23 PM
JoshStrobl added a subscriber: JoshStrobl.

Bumped to the next stable released version.

Needs a summarized changelog.

This revision now requires changes to proceed.Dec 28 2017, 6:23 PM

Do you expect me to write one as the developer doesn't provide one?

Nah I'll just cry a little on the inside. Can you test this via the network-manager-fortisslvpn package (so I assume NM and GUI bits) to ensure it works w/ openfortivpn 1.6.0? Since it was only tested via the CLI.

I tested this version quite a bit over the last few days. The command line version works mostly as expected. An issue I have had with previous openfortivpn versions on Solus (not on Arch though) is that hostnames don't translate to IP addresses. I see the changes added to /etc/resolv.conf but for some reason the translation doesn't happen. This was an issue in 1.5.0 and still is an issue. I'll follow this up with openfortivpn on github.

The latest released NetworkManager plugin is bugged and doesn't work. I just posted information on this on the mailing-list and apparently because there is already a patch available they don't plan on releasing an updated version anytime soon. I'm not sure how Solus works with downstream patches... I didn't see a way for me to do this in a PR. Anyways the relevant issue for the network-manager-fortisslvpn plugin can be found here:

sankasan added a comment.EditedJan 5 2018, 8:38 PM

After applying a fix for network-manager-fortisslvpn the GUI works equally well with 1.5.0 and 1.6.0. The patch can be found here:

I'm in contact with the openfortivpn developer on the DNS issue.

sankasan added a comment.EditedJan 6 2018, 8:28 PM

Issue resolved.. was a VPN configuration issue. The update is ready to go live!

What are we waiting for?

JoshStrobl accepted this revision.Mar 16 2018, 6:50 PM
This revision is now accepted and ready to land.Mar 16 2018, 6:50 PM
JoshStrobl closed this revision.Mar 17 2018, 1:50 PM

Closing revision. Can't be updated without network-manager-fortisslvpn being updated and its diff wasn't rebased.