Project

General

Profile

Bug #491

Dynamic DNS upgrade code not working

Added by Chris Buechler over 9 years ago. Updated over 8 years ago.

Status:
Resolved
Priority:
Normal
Assignee:
-
Category:
Configuration Upgrade
Target version:
Start date:
04/08/2010
Due date:
% Done:

0%

Estimated time:
Affected Version:
2.0
Affected Architecture:

Description

Dynamic DNS account is lost after upgrade from 1.2.3 to 2.0.

Associated revisions

Revision e31c90fc (diff)
Added by Ermal Luçi over 9 years ago

Ticket #491. Fix upgrade code. Since dyndns is considered and array item it failed with previous code. Use index of 0 since 1.2.x does not have multiple entries.

Revision 246aceaa (diff)
Added by Seth Mos over 9 years ago

Prevent empry dyndns array items on config upgrade ticket #491

Revision c9e13418 (diff)
Added by Ermal Luçi over 8 years ago

Ticket #491. Do not actually check for enable to exist but for host since enable might be unset explicitly by the user. This should re-enable configs to work.

Revision 7d62c4c8 (diff)
Added by Ermal Luçi over 8 years ago

Ticket #491. Correct username/password name fields so upgrade works correctly.

History

#1 Updated by Jim Pingle over 9 years ago

Here's some before-and-after config clippings for this. Looks like we may just need to move the <dyndns> entry as-is under <dyndnses> and maybe set interface to wan since that was the only choice on 1.2.x

Before:

<dyndns>
    <type>dyndns</type>
    <username>jim-p</username>
    <password>xxxx</password>
    <host>jim-p.is-a-geek.com</host>
    <mx/>
    <enable/>
</dyndns>

And after:

<dyndnses>
    <dyndns>
        <type>dyndns</type>
        <username>jim-p</username>
        <password>xxxx</password>
        <host>jim-p.is-a-geek.com</host>
        <mx/>
        <enable/>
        <interface>wan</interface>
        <descr>DynDNS for DSL</descr>
    </dyndns>
</dyndnses>

#2 Updated by Ermal Luçi over 9 years ago

  • Status changed from New to Feedback

It was there just it wasn't working because 'dyndns' now i considered as a member of an array.

#3 Updated by Seth Mos over 9 years ago

The config upgrade code worked a bit too well, it produced empty dyndns config items on upgrades when none existed before.

#4 Updated by Seth Mos over 9 years ago

  • Status changed from Feedback to Resolved

Seth Mos wrote:

The config upgrade code worked a bit too well, it produced empty dyndns config items on upgrades when none existed before.

#5 Updated by Chris Buechler over 8 years ago

  • Subject changed from Dynamic DNS upgrade code missing to Dynamic DNS upgrade code not working
  • Status changed from Resolved to New

This is reportedly not working, config is gone after upgrade.

#6 Updated by Ermal Luçi over 8 years ago

  • Status changed from New to Feedback

#7 Updated by Chris Buechler over 8 years ago

  • Status changed from Feedback to New

Closer, everything but the username and password is converted properly. It's completely missing the username and password after upgrade.

#8 Updated by Ermal Luçi over 8 years ago

  • Status changed from New to Feedback

Should be ok now. It was using not correct key values to assign.

#9 Updated by Ermal Luçi over 8 years ago

  • Status changed from Feedback to Resolved

Also available in: Atom PDF