On 12-01-2020 21:00, Jef Driesen wrote:

On 12/01/2020 20:34, Jan Mulder wrote:
When downloading from an OSTC4, I get the same error. A little
investigation shows that in OstcFirmwareCheck::parseOstcFwVersion() a
very unexpected string is parsed:

"<html>
<head><title>301 Moved Permanently</title></head>
<body>
<center><h1>301 Moved Permanently"

where we expect a string like "[1.5.1]", being the first line in the
http://www.heinrichsweikamp.net/autofirmware/ostc4_changelog.txt file.

So, the problem is not data returned from the DC, but from the webserver
(and totally unprotected parsing at Subsurface end). But, interestingly,
visiting the mentioned changelog file from a browser (or curl), all
looks just fine.

Looks like www.heinrichsweikamp.net redirects to heinrichsweikamp.net, and subsurface doesn't follow the redirect automatically. This probably needs to be configured explicitly in the code somewhere.

Yes, thanks Jef, that's it. Just verified things by directly querying http://heinrichsweikamp.net/autofirmware/ostc4_changelog.txt. That works as intended.

--jan



_______________________________________________
subsurface mailing list
[email protected]
http://lists.subsurface-divelog.org/cgi-bin/mailman/listinfo/subsurface

Reply via email to