Merge lp://staging/~jbicha/ubuntu-start-page/add-basic-nm-check-page into lp://staging/ubuntu-start-page

Proposed by Jeremy Bícha
Status: Rejected
Rejected by: Junien Fridrick
Proposed branch: lp://staging/~jbicha/ubuntu-start-page/add-basic-nm-check-page
Merge into: lp://staging/ubuntu-start-page
Diff against target: 5 lines (+1/-0)
1 file modified
www/nm-connectivity-check.txt (+1/-0)
To merge this branch: bzr merge lp://staging/~jbicha/ubuntu-start-page/add-basic-nm-check-page
Reviewer Review Type Date Requested Status
Junien Fridrick Disapprove
Ubuntu Start Page Pending
Review via email: mp+318177@code.staging.launchpad.net

Description of the change

This adds a basic textpage which can be used by NetworkManager to check network connectivity.

The existing http://start.ubuntu.com/connectivity-check.html does not work as well since the first line of the file is
<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 3.2//EN">
which could be sent by a captive portal.

This is for a new feature which is being discussed here:
https://lists.ubuntu.com/archives/ubuntu-devel/2017-February/039696.html

My proposed page just use the default NetworkManager text.

Fedora however uses 'ok'. I don't know if using that string or a similarly short one would make this page more scalable.

To post a comment you must log in.
Revision history for this message
Junien Fridrick (axino) wrote :

We set up a specific domain for this, so this is not useful anymore.

review: Disapprove

Unmerged revisions

150. By Jeremy Bícha

Add basic NM connectivity check page

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