Merge lp://staging/~jmacmahon/wicd/wired-enc into lp://staging/wicd/1.6

Proposed by Joe MacMahon
Status: Merged
Merged at revision: 716
Proposed branch: lp://staging/~jmacmahon/wicd/wired-enc
Merge into: lp://staging/wicd/1.6
Diff against target: 631 lines (+289/-110)
9 files modified
curses/netentry_curses.py (+114/-47)
curses/wicd-curses.py (+1/-1)
encryption/templates/active_wired (+1/-0)
encryption/templates/wired_8021x (+17/-0)
gtk/netentry.py (+110/-42)
wicd/misc.py (+15/-5)
wicd/networking.py (+15/-7)
wicd/wicd-daemon.py (+2/-1)
wicd/wnettools.py (+14/-7)
To merge this branch: bzr merge lp://staging/~jmacmahon/wicd/wired-enc
Reviewer Review Type Date Requested Status
David Paleino Approve
Review via email: mp+90615@code.staging.launchpad.net

Description of the change

Wired encryption templates have been implemented in the daemon, GTK interface and ncurses interface.

Included is an encryption template for 802.1x wired networks.

Encryption settings may be saved to and loaded from wired profiles.

To post a comment you must log in.
712. By David Paleino

Finally really fix the issue with \x00-ESSIDs :(

Revision history for this message
David Paleino (dpaleino) wrote :

Oh, thanks.

I don't have any kind of such setup handy, so I can't really test it. I'm confident you did, though :)

I'll review the code ASAP.

Thanks for your contribution,
David

713. By Launchpad Translations on behalf of wicd-devel

Launchpad automatic translations update.

Revision history for this message
Joe MacMahon (jmacmahon) wrote :

Yes it works perfectly with the setup I have in university halls,
however I'm not overly confident all 802.1x setups are the same, so a
new template may be required when such an issue becomes apparent.

Regardless, the framework is now in place in the code to support such
templates.

Thanks very much,
Joe

On 31/01/12 17:28, David Paleino wrote:
> Oh, thanks.
>
> I don't have any kind of such setup handy, so I can't really test it. I'm confident you did, though :)
>
> I'll review the code ASAP.
>
> Thanks for your contribution,
> David

714. By David Paleino

Try to better handle the double-escaped unicode ESSIDs

715. By Launchpad Translations on behalf of wicd-devel

Launchpad automatic translations update.

716. By David Paleino

Merged wired-encryption implementation by Joe MacMahon. Thanks!

Revision history for this message
David Paleino (dpaleino) wrote :

Ok, the code seems good, even if I, unfortunately, I can't test it.

I'm confident you did, therefore merging into mainline. We'll see when I release 1.7.1 what kind of bugs pop up :)

Thanks for your contribution,
David

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches

to status/vote changes: