Add Cli to WebUI

Message ID	1466187020-29557-1-git-send-email-fff@chrisi01.de
State	Changes Requested
Headers	show Return-Path: <franken-dev-bounces@freifunk.net> From: Christian Dresel <fff@chrisi01.de> To: franken-dev@freifunk.net Subject: [PATCH] Add Cli to WebUI Date: Fri, 17 Jun 2016 20:10:20 +0200 Message-Id: <1466187020-29557-1-git-send-email-fff@chrisi01.de> Precedence: list Reply-To: Christian Dresel <fff@chrisi01.de>, franken-dev@freifunk.net MIME-Version: 1.0 Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: quoted-printable Errors-To: franken-dev-bounces@freifunk.net Sender: "franken-dev" <franken-dev-bounces@freifunk.net>

Christian Dresel June 17, 2016, 6:10 p.m.

Signed-off-by: Christian Dresel <fff@chrisi01.de>
---
 .../fff/fff-web/files/www/ssl/cgi-bin/cmd.html     | 32 ++++++++++++++++++++++
 .../fff/fff-web/files/www/ssl/cgi-bin/header       |  1 +
 2 files changed, 33 insertions(+)
 create mode 100644 src/packages/fff/fff-web/files/www/ssl/cgi-bin/cmd.html

Jan Kraus June 17, 2016, 6:54 p.m.

Hallo Christian,

der Patch ansich passt meiner Meinung nach so.
Allerdings finde ich die Idee nicht so extrem gut. Das verleitet Leute,
die nicht mal SSH bedienen können dazu auf dem Router irgendwas
rumzuspielen.
Als optionales Paket wäre ich dafür, es aufzunehmen. Allerdings nicht im
Standardumfang.

Grüße Jan

Am Freitag, den 17.06.2016, 20:10 +0200 schrieb Christian Dresel:
> Signed-off-by: Christian Dresel <fff@chrisi01.de>
> ---
>  .../fff/fff-web/files/www/ssl/cgi-bin/cmd.html     | 32 ++++++++++++++++++++++
>  .../fff/fff-web/files/www/ssl/cgi-bin/header       |  1 +
>  2 files changed, 33 insertions(+)
>  create mode 100644 src/packages/fff/fff-web/files/www/ssl/cgi-bin/cmd.html
> 
> diff --git a/src/packages/fff/fff-web/files/www/ssl/cgi-bin/cmd.html b/src/packages/fff/fff-web/files/www/ssl/cgi-bin/cmd.html
> new file mode 100644
> index 0000000..e2ebb10
> --- /dev/null
> +++ b/src/packages/fff/fff-web/files/www/ssl/cgi-bin/cmd.html
> @@ -0,0 +1,32 @@
> +#!/usr/bin/haserl
> +
> +<%in /www/ssl/cgi-bin/header %>
> +<%in /www/ssl/cgi-bin/helpers %>
> +<% show_msg %>
> +<form method="POST">
> +<table style="width: 100%;">
> +	<tr>
> +		<td>
> +			<fieldset>
> +				<legend>Kommandozeile</legend>
> +				<table>
> +					<tr><td colspan="2"><input type="text" name="command" /></td>
> +					<td colspan="2"><input type="submit" name="cmd" value="Ausf&uuml;hren" /></td></tr>
> +				</table>
> +				<table>
> +					<tr><td colspan="2"><% ${POST_command} %></td></tr>
> +				</table>
> +			</fieldset>
> +			<fieldset>
> +				<legend>Beispiel Kommandos</legend>
> +				<table>
> +					<tr><td colspan="2"><b>Ausf&uuml;rbares Kommandos</b></td><td colspan="2"><b>Erkl&auml;rung</b></td></tr>
> +					<tr><td colspan="2">batctl o</td><td colspan="2">Zeigt alle Batman Nachbarn an</td></tr>
> +					<tr><td colspan="2">batctl gwl</td><td colspan="2">Zeigt alle Gateways in der Batman Gatewayselection an</td></tr>
> +				</table>
> +			</fieldset>
> +		</td>
> +	</tr>
> +</table>
> +</form>
> +<%in /www/ssl/cgi-bin/footer %>
> diff --git a/src/packages/fff/fff-web/files/www/ssl/cgi-bin/header b/src/packages/fff/fff-web/files/www/ssl/cgi-bin/header
> index 4cecfcd..5844d07 100755
> --- a/src/packages/fff/fff-web/files/www/ssl/cgi-bin/header
> +++ b/src/packages/fff/fff-web/files/www/ssl/cgi-bin/header
> @@ -36,6 +36,7 @@ HOSTNAME=$(uci get -q 'system.@system[0].hostname')
>  			nav_entry settings.html		"Einstellungen"
>  			nav_entry ports.html		"Anschl&uuml;sse"
>  			nav_entry wifiscan.html		"Wifi Scan"
> +                        nav_entry cmd.html         	"Kommandozeile"
>  			nav_entry upgrade.html		"Upgrade"
>  			nav_entry password.html		"Password"
>  			nav_entry reboot.html		"Neustart"
> -- 
> 2.1.4
>

Christian Dresel June 17, 2016, 7:14 p.m.

hi Jan

aber ist Freifunk nicht dafür da "rumzuspielen"? Soll nicht
experiementiert werden? Ist der Einstieg dorthin nicht leichter wenn man
am WebUI mal mit ein batctl o anfangen kann? Was kann man dort wirklich
kaputt machen (ja rm -rf / ich weiß...)

Der Wunsch kam aus dem Thread "Was alles ins WebUI soll" und da ich das
zuvor schon fertig hatte, hab ich einfach mal ein Patch draus gemacht.
Ob es nun aufgenommen wird oder nicht müsste man dann... hmm wie geht
man jetzt mit sowas um wenn es manche wollen und andere nicht?

mfg

Christian

On 17.06.2016 20:54, mayosemmel wrote:
> Hallo Christian,
>
> der Patch ansich passt meiner Meinung nach so.
> Allerdings finde ich die Idee nicht so extrem gut. Das verleitet Leute,
> die nicht mal SSH bedienen können dazu auf dem Router irgendwas
> rumzuspielen.
> Als optionales Paket wäre ich dafür, es aufzunehmen. Allerdings nicht im
> Standardumfang.
>
> Grüße Jan
>
> Am Freitag, den 17.06.2016, 20:10 +0200 schrieb Christian Dresel:
>> Signed-off-by: Christian Dresel <fff@chrisi01.de>
>> ---
>>  .../fff/fff-web/files/www/ssl/cgi-bin/cmd.html     | 32 ++++++++++++++++++++++
>>  .../fff/fff-web/files/www/ssl/cgi-bin/header       |  1 +
>>  2 files changed, 33 insertions(+)
>>  create mode 100644 src/packages/fff/fff-web/files/www/ssl/cgi-bin/cmd.html
>>
>> diff --git a/src/packages/fff/fff-web/files/www/ssl/cgi-bin/cmd.html b/src/packages/fff/fff-web/files/www/ssl/cgi-bin/cmd.html
>> new file mode 100644
>> index 0000000..e2ebb10
>> --- /dev/null
>> +++ b/src/packages/fff/fff-web/files/www/ssl/cgi-bin/cmd.html
>> @@ -0,0 +1,32 @@
>> +#!/usr/bin/haserl
>> +
>> +<%in /www/ssl/cgi-bin/header %>
>> +<%in /www/ssl/cgi-bin/helpers %>
>> +<% show_msg %>
>> +<form method="POST">
>> +<table style="width: 100%;">
>> +	<tr>
>> +		<td>
>> +			<fieldset>
>> +				<legend>Kommandozeile</legend>
>> +				<table>
>> +					<tr><td colspan="2"><input type="text" name="command" /></td>
>> +					<td colspan="2"><input type="submit" name="cmd" value="Ausf&uuml;hren" /></td></tr>
>> +				</table>
>> +				<table>
>> +					<tr><td colspan="2"><% ${POST_command} %></td></tr>
>> +				</table>
>> +			</fieldset>
>> +			<fieldset>
>> +				<legend>Beispiel Kommandos</legend>
>> +				<table>
>> +					<tr><td colspan="2"><b>Ausf&uuml;rbares Kommandos</b></td><td colspan="2"><b>Erkl&auml;rung</b></td></tr>
>> +					<tr><td colspan="2">batctl o</td><td colspan="2">Zeigt alle Batman Nachbarn an</td></tr>
>> +					<tr><td colspan="2">batctl gwl</td><td colspan="2">Zeigt alle Gateways in der Batman Gatewayselection an</td></tr>
>> +				</table>
>> +			</fieldset>
>> +		</td>
>> +	</tr>
>> +</table>
>> +</form>
>> +<%in /www/ssl/cgi-bin/footer %>
>> diff --git a/src/packages/fff/fff-web/files/www/ssl/cgi-bin/header b/src/packages/fff/fff-web/files/www/ssl/cgi-bin/header
>> index 4cecfcd..5844d07 100755
>> --- a/src/packages/fff/fff-web/files/www/ssl/cgi-bin/header
>> +++ b/src/packages/fff/fff-web/files/www/ssl/cgi-bin/header
>> @@ -36,6 +36,7 @@ HOSTNAME=$(uci get -q 'system.@system[0].hostname')
>>  			nav_entry settings.html		"Einstellungen"
>>  			nav_entry ports.html		"Anschl&uuml;sse"
>>  			nav_entry wifiscan.html		"Wifi Scan"
>> +                        nav_entry cmd.html         	"Kommandozeile"
>>  			nav_entry upgrade.html		"Upgrade"
>>  			nav_entry password.html		"Password"
>>  			nav_entry reboot.html		"Neustart"
>> -- 
>> 2.1.4
>>

Jan Kraus June 17, 2016, 7:17 p.m.

Hi Christia,

eventuell unterschätze ich auch nur die Leute.
Ich muss auf Arbeit unter anderem First Level Support machen. Bei uns
hängen so Sprüche im Büro wie:
"Dumm ist der, der dummes tut" oder "Manche Leute brauchen einen High
Five, ins Gesicht, mit einem Stuhl"
Daraus lässt sich jetzt sicher einiges schließen ;-)

Wenn noch 1-2 der anderen das für ne gute Idee halten, werde ich nicht
das streiten anfangen. Ich benutze das Web-IF sowieso nur fürs Position
setzten und danach nie wieder ;-)

Grüße Jan

Am Freitag, den 17.06.2016, 21:14 +0200 schrieb Christian Dresel:
> hi Jan
> 
> aber ist Freifunk nicht dafür da "rumzuspielen"? Soll nicht
> experiementiert werden? Ist der Einstieg dorthin nicht leichter wenn man
> am WebUI mal mit ein batctl o anfangen kann? Was kann man dort wirklich
> kaputt machen (ja rm -rf / ich weiß...)
> 
> Der Wunsch kam aus dem Thread "Was alles ins WebUI soll" und da ich das
> zuvor schon fertig hatte, hab ich einfach mal ein Patch draus gemacht.
> Ob es nun aufgenommen wird oder nicht müsste man dann... hmm wie geht
> man jetzt mit sowas um wenn es manche wollen und andere nicht?
> 
> mfg
> 
> Christian
> 
> On 17.06.2016 20:54, mayosemmel wrote:
> > Hallo Christian,
> >
> > der Patch ansich passt meiner Meinung nach so.
> > Allerdings finde ich die Idee nicht so extrem gut. Das verleitet Leute,
> > die nicht mal SSH bedienen können dazu auf dem Router irgendwas
> > rumzuspielen.
> > Als optionales Paket wäre ich dafür, es aufzunehmen. Allerdings nicht im
> > Standardumfang.
> >
> > Grüße Jan
> >
> > Am Freitag, den 17.06.2016, 20:10 +0200 schrieb Christian Dresel:
> >> Signed-off-by: Christian Dresel <fff@chrisi01.de>
> >> ---
> >>  .../fff/fff-web/files/www/ssl/cgi-bin/cmd.html     | 32 ++++++++++++++++++++++
> >>  .../fff/fff-web/files/www/ssl/cgi-bin/header       |  1 +
> >>  2 files changed, 33 insertions(+)
> >>  create mode 100644 src/packages/fff/fff-web/files/www/ssl/cgi-bin/cmd.html
> >>
> >> diff --git a/src/packages/fff/fff-web/files/www/ssl/cgi-bin/cmd.html b/src/packages/fff/fff-web/files/www/ssl/cgi-bin/cmd.html
> >> new file mode 100644
> >> index 0000000..e2ebb10
> >> --- /dev/null
> >> +++ b/src/packages/fff/fff-web/files/www/ssl/cgi-bin/cmd.html
> >> @@ -0,0 +1,32 @@
> >> +#!/usr/bin/haserl
> >> +
> >> +<%in /www/ssl/cgi-bin/header %>
> >> +<%in /www/ssl/cgi-bin/helpers %>
> >> +<% show_msg %>
> >> +<form method="POST">
> >> +<table style="width: 100%;">
> >> +	<tr>
> >> +		<td>
> >> +			<fieldset>
> >> +				<legend>Kommandozeile</legend>
> >> +				<table>
> >> +					<tr><td colspan="2"><input type="text" name="command" /></td>
> >> +					<td colspan="2"><input type="submit" name="cmd" value="Ausf&uuml;hren" /></td></tr>
> >> +				</table>
> >> +				<table>
> >> +					<tr><td colspan="2"><% ${POST_command} %></td></tr>
> >> +				</table>
> >> +			</fieldset>
> >> +			<fieldset>
> >> +				<legend>Beispiel Kommandos</legend>
> >> +				<table>
> >> +					<tr><td colspan="2"><b>Ausf&uuml;rbares Kommandos</b></td><td colspan="2"><b>Erkl&auml;rung</b></td></tr>
> >> +					<tr><td colspan="2">batctl o</td><td colspan="2">Zeigt alle Batman Nachbarn an</td></tr>
> >> +					<tr><td colspan="2">batctl gwl</td><td colspan="2">Zeigt alle Gateways in der Batman Gatewayselection an</td></tr>
> >> +				</table>
> >> +			</fieldset>
> >> +		</td>
> >> +	</tr>
> >> +</table>
> >> +</form>
> >> +<%in /www/ssl/cgi-bin/footer %>
> >> diff --git a/src/packages/fff/fff-web/files/www/ssl/cgi-bin/header b/src/packages/fff/fff-web/files/www/ssl/cgi-bin/header
> >> index 4cecfcd..5844d07 100755
> >> --- a/src/packages/fff/fff-web/files/www/ssl/cgi-bin/header
> >> +++ b/src/packages/fff/fff-web/files/www/ssl/cgi-bin/header
> >> @@ -36,6 +36,7 @@ HOSTNAME=$(uci get -q 'system.@system[0].hostname')
> >>  			nav_entry settings.html		"Einstellungen"
> >>  			nav_entry ports.html		"Anschl&uuml;sse"
> >>  			nav_entry wifiscan.html		"Wifi Scan"
> >> +                        nav_entry cmd.html         	"Kommandozeile"
> >>  			nav_entry upgrade.html		"Upgrade"
> >>  			nav_entry password.html		"Password"
> >>  			nav_entry reboot.html		"Neustart"
> >> -- 
> >> 2.1.4
> >>
> 
>

Tim Niemeyer June 18, 2016, 8:55 a.m.

Hi

Am 17. Juni 2016 20:54:39 MESZ, schrieb mayosemmel <mayosemmel@googlemail.com>:
>Hallo Christian,
>
>der Patch ansich passt meiner Meinung nach so.
>Allerdings finde ich die Idee nicht so extrem gut. Das verleitet Leute,
>die nicht mal SSH bedienen können dazu auf dem Router irgendwas
>rumzuspielen.
>Als optionales Paket wäre ich dafür, es aufzunehmen. Allerdings nicht
>im
>Standardumfang.

Ihr habt beide Recht. Wie wäre es mit "Experten" Modus?

Tim
>
>Grüße Jan
>
>Am Freitag, den 17.06.2016, 20:10 +0200 schrieb Christian Dresel:
>> Signed-off-by: Christian Dresel <fff@chrisi01.de>
>> ---
>>  .../fff/fff-web/files/www/ssl/cgi-bin/cmd.html     | 32
>++++++++++++++++++++++
>>  .../fff/fff-web/files/www/ssl/cgi-bin/header       |  1 +
>>  2 files changed, 33 insertions(+)
>>  create mode 100644
>src/packages/fff/fff-web/files/www/ssl/cgi-bin/cmd.html
>> 
>> diff --git a/src/packages/fff/fff-web/files/www/ssl/cgi-bin/cmd.html
>b/src/packages/fff/fff-web/files/www/ssl/cgi-bin/cmd.html
>> new file mode 100644
>> index 0000000..e2ebb10
>> --- /dev/null
>> +++ b/src/packages/fff/fff-web/files/www/ssl/cgi-bin/cmd.html
>> @@ -0,0 +1,32 @@
>> +#!/usr/bin/haserl
>> +
>> +<%in /www/ssl/cgi-bin/header %>
>> +<%in /www/ssl/cgi-bin/helpers %>
>> +<% show_msg %>
>> +<form method="POST">
>> +<table style="width: 100%;">
>> +	<tr>
>> +		<td>
>> +			<fieldset>
>> +				<legend>Kommandozeile</legend>
>> +				<table>
>> +					<tr><td colspan="2"><input type="text" name="command" /></td>
>> +					<td colspan="2"><input type="submit" name="cmd"
>value="Ausf&uuml;hren" /></td></tr>
>> +				</table>
>> +				<table>
>> +					<tr><td colspan="2"><% ${POST_command} %></td></tr>
>> +				</table>
>> +			</fieldset>
>> +			<fieldset>
>> +				<legend>Beispiel Kommandos</legend>
>> +				<table>
>> +					<tr><td colspan="2"><b>Ausf&uuml;rbares Kommandos</b></td><td
>colspan="2"><b>Erkl&auml;rung</b></td></tr>
>> +					<tr><td colspan="2">batctl o</td><td colspan="2">Zeigt alle
>Batman Nachbarn an</td></tr>
>> +					<tr><td colspan="2">batctl gwl</td><td colspan="2">Zeigt alle
>Gateways in der Batman Gatewayselection an</td></tr>
>> +				</table>
>> +			</fieldset>
>> +		</td>
>> +	</tr>
>> +</table>
>> +</form>
>> +<%in /www/ssl/cgi-bin/footer %>
>> diff --git a/src/packages/fff/fff-web/files/www/ssl/cgi-bin/header
>b/src/packages/fff/fff-web/files/www/ssl/cgi-bin/header
>> index 4cecfcd..5844d07 100755
>> --- a/src/packages/fff/fff-web/files/www/ssl/cgi-bin/header
>> +++ b/src/packages/fff/fff-web/files/www/ssl/cgi-bin/header
>> @@ -36,6 +36,7 @@ HOSTNAME=$(uci get -q 'system.@system[0].hostname')
>>  			nav_entry settings.html		"Einstellungen"
>>  			nav_entry ports.html		"Anschl&uuml;sse"
>>  			nav_entry wifiscan.html		"Wifi Scan"
>> +                        nav_entry cmd.html         	"Kommandozeile"
>>  			nav_entry upgrade.html		"Upgrade"
>>  			nav_entry password.html		"Password"
>>  			nav_entry reboot.html		"Neustart"
>> -- 
>> 2.1.4
>> 
>
>
>
>------------------------------------------------------------------------

Ralph A. Schmid, dk5ras June 20, 2016, 11:10 a.m.

Moin,

> Ihr habt beide Recht. Wie wäre es mit "Experten" Modus?

Ja, fände ich gut. Meinetwegen noch 'ne Warnung dazu, und fertig. Gibt es ja
bei allen möglichen Geräten, bis hin zu FRITZ-Boxen und derlei...
 
> Tim

Viele Grüße

Ralph.

Details

Commit Message

Patch hide | download patch | download mbox

Comments