acidburn0zzz / ics-openvpn

Automatically exported from code.google.com/p/ics-openvpn
1 stars 0 forks source link

Won .ovpn files - parsing #266

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
Version: OpenVPN For Android 0.6.17 from Play Store

Steps to reproduce the problem:
1.On Android phone in any browser, download .ovpn files from privatetunnel.com 
account.
2.Import .ovpn using folder icon
3.See error msg

Expected output: successful import

Actual output: error log: "No endtag <//span> for starttag </span>" (truncated 
for clarity).

Mobile phone: LG MS323

Android Version and stock ROM: 4.4.2, LGMS32310B

Original issue reported on code.google.com by fhaus...@gmail.com on 18 Jul 2014 at 9:41

GoogleCodeExporter commented 9 years ago
Title should be: 
Won't import PT .ovpn files - parsing

Original comment by fhaus...@gmail.com on 18 Jul 2014 at 9:44

GoogleCodeExporter commented 9 years ago
I think your ovpn file is somehow broken. Judging from the error message you 
have imported a html file.

Original comment by arne@rfc2549.org on 18 Jul 2014 at 10:20

GoogleCodeExporter commented 9 years ago
At the time I checked today, all .ovpn files downloaded from privatetunnel.com 
follow this general format:

<div class="rt-joomla ">
    <div class="rt-article">

        <p><span style="font-size: 12.222222328186035px; line-height: 15.796296119689941px;"># Automatically generated OpenVPN client config file
# Generated on Fri Jul 18 06:48:09 2014 by us-il-chi-001
(Many more #configuration lines)
...
# zafXhJd85Gju0bZstM
# -----END RSA SIGNATURE-----
</span></p>
</div>
</div>

Chrome, Opera Mini and Dolphin each download the .ovpn files identically.

The official client, OpenVPN Connect
Connect, parses each of these .ovpn files without error, effectively ignoring 
the html.

I'm suggesting that OpenVPN For Android should also parse these files, 
improving the user experience and interoperability.

As a side note, if, at install&try time, this client had successfully imported, 
I would not have moved on to the official client.

Original comment by fhaus...@gmail.com on 18 Jul 2014 at 11:50

GoogleCodeExporter commented 9 years ago
I am not comfortable with ignoring unknown configuration. That configuration 
file is just not a valid configuration file. I suggest that you contact 
privatetunnel.com support.

I will not implement a html parser to parse these completely broken files.

Original comment by arne@rfc2549.org on 18 Jul 2014 at 11:55