Projecte

General

Perfil

Actions

Error #341

tancat

[guifi qmp 3.1] not working

Afegit per guifi pedro fa casi 10 anys. Actualitzat fa més de 9 anys.

Estat:
Tancat
Prioritat:
Normal
Assignat a:
Categoria:
Packages
Versió prevista:
-
Inici:
18-01-2015
Data de venciment:
% realitzat:

100%

Temps previst:

Descripció

I tried with:
https://guifi.net/en/guifi/device/72950/view/unsolclic
and it does not work
[this gives an error with old version of qMp]

with http accept the url:
http://guifi.net/en/guifi/device/72950/view/unsolclic
same for:
http://guifi.net/guifi/device/72950/

results

formatting is failing

<font color='#0a0'>Getting oneclick config: Done! Checking oneclick config: Done! Showing variables: nodename='' devname='' devmodel='Rocket/Nano/Loco' ipv4='' netmask='' zoneid='' </font>

and cannot obtain the values, so get a random ipv4, etc.
[you can get this values correctly from old version of qMp]

Actions #1

Actualitzat per Jorge L. Florit fa casi 10 anys

  • Categoria s'ha establert a Packages
  • Estat ha canviat de Nou a En progrés

Hi, qmp-guifi oneclick has never worked with HTTPS, in fact, wget in OpenWRT doesn't get HTTPS URLs; it is due to a bug with wget in busybox [1].
We have to try an alternative to support HTTPS.

By the way, with HTTP (no secure) in clearance 3.0 works well:
# qmpguifi get_url http://guifi.net/en/guifi/device/72950/view/unsolclic /tmp/guifi_oneclick
Getting oneclick config:
Done!

# qmpguifi check /tmp/guifi_oneclick
Checking oneclick config:
Done!

# qmpguifi print /tmp/guifi_oneclick
Showing variables:
nodename='BCNCoroleu29'
devname='BCNCoroleu29Rd1'
devmodel='AirMaxM5_Rocket/Nano/Loco'
ipv4='10.1.57.129'
netmask='255.255.255.224'
zoneid='StAP'

I have to try with the latest v3.1, but nothing has change in the package for months, so it should still work.

[1] http://lists.busybox.net/pipermail/busybox-cvs/2005-April/019949.html

Actions #2

Actualitzat per guifi pedro fa casi 10 anys

  • Tema ha canviat de [guifi qmp] not working / do not accept https url a [guifi qmp 3.1] not working
Actions #3

Actualitzat per guifi pedro fa casi 10 anys

with a qMp 3.1, I'm sure you have access to it: 10.1.57.128

root@BCNCoroleu29-62ba:~# qmpguifi check /tmp/guifi_oneclick
Checking oneclick config:
Done!
root@BCNCoroleu29-62ba:~# qmpguifi get_url http://guifi.net/en/guifi/device/72950/view/unsolclic /tmp/guifi_oneclick
Getting oneclick config:
Done!
root@BCNCoroleu29-62ba:~# qmpguifi check /tmp/guifi_oneclick
Checking oneclick config:
Done!
root@BCNCoroleu29-62ba:~# qmpguifi print /tmp/guifi_oneclick
Showing variables:
nodename=''
devname=''
devmodel='Rocket/Nano/Loco'
ipv4=''
netmask=''
zoneid=''

Actions #4

Actualitzat per guifi pedro fa casi 10 anys

About the HTTPS error, I suggest a better error output (detect initial 4 characters: https). Say that "HTTPS URL is not valid, please use HTTP" (specially in the web part) instead of:

  1. qmpguifi get_url https://guifi.net/en/guifi/device/72950/view/unsolclic /tmp/guifi_oneclick
    Getting oneclick config:
    qMp: ERROR: Wrong URL specified. USE: '/usr/bin/qmpguifi get_url [http://guifi.net/guifi/device/#####/] [FILE]'

Web error says:
Getting oneclick config: ERROR: Wrong URL specified.

also, I think the web should take this URL as valid (put "http://" if there is no protocol specified):
guifi.net/en/guifi/device/72950/view/unsolclic

The CLI part is ok, because is assumed to be a more experimented user.
Or perhaps we can assume that in general all users that use qMp are experimented and this corrections are not needed.

Actions #5

Actualitzat per Jorge L. Florit fa casi 10 anys

The HTTPS issue should be revised separately and maybe we can change wget to curl. So it will be a feature to support https instead of a bug. Of course it should be nice to differentiate an error in the URL from an error if it's a secure HTTP or not.

The other question is because there is some change in the behaviour of the awk command, it ocurrs in v3.1 because there is a new version of busybox.

root@UPCc6lab104-TL-5cab:~# awk
BusyBox v1.19.4 (2014-05-23 12:27:44 CEST) multi-call binary.

root@BCNCoroleu29-62ba:~# awk
BusyBox v1.22.1 (2014-12-12 10:08:21 CET) multi-call binary.

Actions #6

Actualitzat per Jorge L. Florit fa casi 10 anys

  • % realitzat ha canviat de 0 a 50

The blank fields issue should be solved with commit 148748f3.
If you want to test it without flashing again, you can replace /etc/qmp/qmp_guifi.sh with the fixed file [1].

[1] http://dev.qmp.cat/projects/qmp/repository/revisions/148748f338fd6a6079afd7224f3712cd52e8cd84/raw/packages/qmp-guifi/files/etc/qmp/qmp_guifi.sh

Actions #7

Actualitzat per guifi pedro fa casi 10 anys

now it works. How fast Jorge (less than 24h, what's that! :P), thanks!

- formatting in the web is awful
- another easy way to solve https is change this text:
Specify the guifi.net URL of the device:
with this other:
Specify the guifi.net URL (HTTP) of the device:

seems that curl is not included in qMp 3.1, so perhaps is a bad idea, it means more size for images (thinking about 4MB flash devices), no?

Actions #8

Actualitzat per Jorge L. Florit fa casi 10 anys

  • Estat ha canviat de En progrés a Resolt
  • % realitzat ha canviat de 50 a 100

"- formatting in the web is awful"

It is because of the new theme. It's now fixed in commit cf62ebab [1].

"- another easy way to solve https is change this text [...]"

I noticed that curl is not included, so while wget in busybox doesn't support HTTPS I've added the purposed solution temporarily (in commit eb8c6514 [2]): an error message notifying that HTTPS URLs are not supported yet and the text specifying "HTTP URLs" in the web interface.

[1] http://dev.qmp.cat/projects/qmp/repository/revisions/cf62ebabf0b072a075bb326b3460ceefae383adc
[2] http://dev.qmp.cat/projects/qmp/repository/revisions/eb8c6514dab5bf80e8a96687aee1209e5afa744c

Actions #9

Actualitzat per Jorge L. Florit fa més de 9 anys

  • Estat ha canviat de Resolt a Tancat
Actions

També disponible a: Atom PDF