inadyn 1.99.3 non-functional

Bug #1270539 reported by Michael Johnson
10
This bug affects 1 person
Affects Status Importance Assigned to Milestone
inadyn (Ubuntu)
Fix Released
High
Unassigned
Saucy
Fix Released
High
Unassigned

Bug Description

[SRU REQUEST]

[IMPACT]

Base64 encoding of the HTTP basic authorisation token was broken
resulting in updates being rejected with 401 Unauthorised.

Log file fragment:
(Hostname and IP address edited)

Tue Aug 6 20:33:09 2013: Sending IP# update to DDNS server, connecting to dynupdate.no-ip.com(8.23.224.120)
Tue Aug 6 20:33:10 2013: Sending alias table update to DDNS server:
Tue Aug 6 20:33:10 2013: GET /nic/update?hostname=myhostname.no-ip.biz&myip=11.22.33.44 HTTP/1.0
Host: dynupdate.no-ip.com
Authorization: Basic 3t
User-Agent: inadyn/1.99.3 <email address hidden>

Tue Aug 6 20:33:10 2013: Fatal error in DDNS server response:
Tue Aug 6 20:33:10 2013: [401 Unauthorized] This service requires basic http authentication
Tue Aug 6 20:33:10 2013: DDNS server response:
Tue Aug 6 20:33:10 2013: HTTP/1.0 401 Unauthorized

[TEST CASE]

Version saucy doesn't authorize correctly, version in trusty does.

[REGRESSION POTENTIAL]

none

[OTHER INFO]
https://github.com/troglobit/inadyn/pull/40 is the upstream issue.

Revision history for this message
Andreas Moog (ampelbein) wrote :

Confirmed, setting the general bug to "Fix Released" (1.99.4 is in trusty) and adding a saucy salamander task).

Changed in inadyn (Ubuntu):
importance: Undecided → High
status: New → Fix Released
Changed in inadyn (Ubuntu Saucy):
importance: Undecided → High
Andreas Moog (ampelbein)
description: updated
Revision history for this message
Andreas Moog (ampelbein) wrote :

I uploaded a fix and now we have to wait for the SRU team to approve the upload.

Changed in inadyn (Ubuntu Saucy):
status: New → In Progress
Revision history for this message
Brian Murray (brian-murray) wrote : Please test proposed package

Hello Michael, or anyone else affected,

Accepted inadyn into saucy-proposed. The package will build now and be available at http://launchpad.net/ubuntu/+source/inadyn/1.99.3-1ubuntu0.1 in a few hours, and then in the -proposed repository.

Please help us by testing this new package. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation how to enable and use -proposed. Your feedback will aid us getting this update out to other Ubuntu users.

If this package fixes the bug for you, please add a comment to this bug, mentioning the version of the package you tested, and change the tag from verification-needed to verification-done. If it does not fix the bug for you, please add a comment stating that, and change the tag to verification-failed. In either case, details of your testing will help us make a better decision.

Further information regarding the verification process can be found at https://wiki.ubuntu.com/QATeam/PerformingSRUVerification . Thank you in advance!

Changed in inadyn (Ubuntu Saucy):
status: In Progress → Fix Committed
tags: added: verification-needed
Revision history for this message
Michael Johnson (mj-revmj) wrote :

Sorry it took me so long to get back around to this, but I can confirm that the proposed package is working properly.

Revision history for this message
Michael Johnson (mj-revmj) wrote :

Here are the details from my system to show it functioning (log obfuscated to remove identifying information):

$ lsb_release -a
No LSB modules are available.
Distributor ID: Ubuntu
Description: Ubuntu 13.10
Release: 13.10
Codename: saucy

$ dpkg -l inadyn
Desired=Unknown/Install/Remove/Purge/Hold
| Status=Not/Inst/Conf-files/Unpacked/halF-conf/Half-inst/trig-aWait/Trig-pend
|/ Err?=(none)/Reinst-required (Status,Err: uppercase=bad)
||/ Name Version Architecture Description
+++-=======================================-========================-========================-===================================================================================
ii inadyn 1.99.3-1ubuntu0.1 amd64 Simple and small DynDNS client written in the C language

$ cat /var/log/inadyn/inadyn.log | sed -e 's/[0-9]\+\.[0-9]\+\.[0-9]\+\.[0-9]\+/X.X.X.X/g' -e 's/[^ ]\+\(no-ip\.org\)/XXXX.\1/g' | head -58
Wed Feb 12 21:22:32 2014: Inadyn version 1.99.3 -- Dynamic DNS update client.
Wed Feb 12 21:22:32 2014: Resolving hostname XXXX.no-ip.org => IP# X.X.X.X
Wed Feb 12 21:22:32 2014: Checking for IP# change, connecting to ip1.dynupdate.no-ip.com(X.X.X.X)
Wed Feb 12 21:22:32 2014: Querying DDNS server for my public IP#:
Wed Feb 12 21:22:32 2014: GET / HTTP/1.0
Host: ip1.dynupdate.no-ip.com
User-Agent: inadyn/1.99.3 <email address hidden>

Wed Feb 12 21:22:32 2014: IP server response:
Wed Feb 12 21:22:32 2014: HTTP/1.1 200 OK
Server: nginx
Date: Thu, 13 Feb 2014 03:21:30 GMT
Content-Type: text/html
Connection: close

X.X.X.X
Wed Feb 12 21:22:32 2014: Current public IP# X.X.X.X
Wed Feb 12 21:22:32 2014: Update needed for alias XXXX.no-ip.org, new IP# X.X.X.X
Wed Feb 12 21:22:32 2014: Sending IP# update to DDNS server, connecting to dynupdate.no-ip.com(X.X.X.X)
Wed Feb 12 21:22:33 2014: Sending alias table update to DDNS server:
Wed Feb 12 21:22:33 2014: GET XXXX.no-ip.org&myip=X.X.X.X HTTP/1.0
Host: dynupdate.no-ip.com
Authorization: Basic dGhlcmV2bWo6ckBncEVGVEdINURjTzlCNg==
User-Agent: inadyn/1.99.3 <email address hidden>

Wed Feb 12 21:22:33 2014: Successful alias table update for XXXX.no-ip.org => new IP# X.X.X.X
Wed Feb 12 21:22:33 2014: DDNS server response:
Wed Feb 12 21:22:33 2014: HTTP/1.1 200 OK
Date: Thu, 13 Feb 2014 03:21:30 GMT
Server: Apache/2
Content-Location: update.php
Vary: negotiate
TCN: choice
Content-Length: 20
Connection: close
Content-Type: text/plain; charset=UTF-8

good X.X.X.X
Wed Feb 12 21:27:34 2014: .
Wed Feb 12 21:27:34 2014: Checking for IP# change, connecting to ip1.dynupdate.no-ip.com(X.X.X.X)
Wed Feb 12 21:27:34 2014: Querying DDNS server for my public IP#:
Wed Feb 12 21:27:34 2014: GET / HTTP/1.0
Host: ip1.dynupdate.no-ip.com
User-Agent: inadyn/1.99.3 <email address hidden>

Wed Feb 12 21:27:34 2014: IP server response:
Wed Feb 12 21:27:34 2014: HTTP/1.1 200 OK
Server: nginx
Date: Thu, 13 Feb 2014 03:26:31 GMT
Content-Type: text/html
Connection: close

X.X.X.X
Wed Feb 12 21:27:34 2014: No IP# change detected, still at X.X.X.X
Wed Feb 12 21:27:34 2014: Current public IP# X.X.X.X

Revision history for this message
Michael Johnson (mj-revmj) wrote :

Ok, well, I notice too late that I left the base 64 credentials in the log output. But no worries, it was not a password I used anywhere else and I have changed it.

Andreas Moog (ampelbein)
tags: added: verification-done
removed: verification-needed
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package inadyn - 1.99.3-1ubuntu0.1

---------------
inadyn (1.99.3-1ubuntu0.1) saucy-proposed; urgency=low

  * Grab patch from upstream to fix basic HTTP authentication. (LP: #1270539)
 -- Andreas Moog <email address hidden> Sun, 19 Jan 2014 11:31:29 +0100

Changed in inadyn (Ubuntu Saucy):
status: Fix Committed → Fix Released
Revision history for this message
Brian Murray (brian-murray) wrote : Update Released

The verification of the Stable Release Update for inadyn has completed successfully and the package has now been released to -updates. Subsequently, the Ubuntu Stable Release Updates Team is being unsubscribed and will not receive messages about this bug report. In the event that you encounter a regression using the package from -updates please report a new bug using ubuntu-bug and tag the bug report regression-update so we can easily find any regresssions.

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.