Webinterface Autokorrekturen

highTower · 9. August 2022 um 04:32

Ich würde vorschlagen, bei ajenti in den Schüler Lehrer und gerätelisten die Eingaben mehr zu kontrollieren und gegebenenfalls automatisiert zu korrigieren.

Beispiele dafür sind groß geschriebene Gerätenamen und IP-Adresse mit einer führenden null. Beispielsweise wird die IP-Adresse 10. 210.056.1 leider nicht korrigiert und limbo kommt damit nicht klar.

Aufgefallen ist mir weiterhin das spannende Zertifikat in der ajenti ssl-conf vielleicht kann man ja auch ein letsencrypt Plugin anbieten.

Arnaud · 9. August 2022 um 07:26

Hallo @highTower ( wir pflegen lieber die Begrüßung hier ),

Danke, das ist sogar auf meine kleine Todoliste, hatte ich vergessen, aber das schaue ich mir erst ende August wieder. Der fehlerhafte Regexp ist hier :

github.com

linuxmuster/linuxmuster-webui7/blob/a1692e20ed19a271c553884d9b29085975d64cd5/usr/lib/linuxmuster-webui/plugins/lmn_common/resources/js/services/validation.coffee#L137


      
          
              validMac = validMac && (this.externVar['devices'].filter(this.findval('mac', mac)).length < 2)
          
              if !validMac
                  return error_msg
              return true
          
          # Regexp for ip address, and tests if no duplicate
          this.isValidIP = (ip) ->
              error_msg = ip + gettext(' is not valid or duplicated')
              regExp = /^(25[0-5]|2[0-4][0-9]|[01]?[0-9][0-9]?)\.(25[0-5]|2[0-4][0-9]|[01]?[0-9][0-9]?)\.(25[0-5]|2[0-4][0-9]|[01]?[0-9][0-9]?)\.(25[0-5]|2[0-4][0-9]|[01]?[0-9][0-9]?)$/ ## TODO all IPs allowed, and 010.1.1.1
              validIP = regExp.test(ip) && (this.externVar['devices'].filter(this.findval('ip', ip)).length < 2)
              if !validIP
                  return error_msg
              return true
          
          # Hostnames not empty, only with alphanumeric chars and "-", and tests if no duplicate
          this.isValidHost = (hostname, test_length = true) ->
              error_msg = hostname + gettext(' does not contain valid chars, is duplicated or too long (>15 chars)')
              regExp = /^[a-zA-Z0-9\-]+$/
              validHostname = regExp.test(hostname) && (this.externVar['devices'].filter(this.findval('hostname', hostname)).length < 2)

Man sieht damit, dass 0 als Präfix akzeptiert ist ( problem ist [01]? ).

Es gibt für die Webui enorm viele Wünsche, und nicht so viele Hände. Das mit dem let’s encrypt wäre ein gutes „First issue“. Eigentlich ist alles vorbereitet in Ajenti um es zu entwickeln :

github.com

ajenti/ajenti/blob/master/ajenti-core/aj/https_redirect.py

from gevent import pywsgi, spawn

def http_to_https(target_url):

    def app(environ, start_response):

        if environ['PATH_INFO'] == "/":
            start_response(
                            '301 Move Permanently',
                            [
                                ('Location', target_url),
                                ('Connection', 'close'),
                                ('Cache-control', 'private')
                            ])
            return [b"""
    <html>
    <body>
        Migrating to HTTPS protocol.
        Please go to <a href='{self.target}'>{self.target}</a> to use Ajenti.
    </body>

Diese Datei wurde abgeschnitten. Original zeigen

Man kann z.B. eine andere URL hinzufügen :

if environ[‘PATH_INFO’].startswith(“/.well-known/acme-challenge/”) and ‘…’ not in environ[‘PATH_INFO’]:
    try:
        with open(f"…/acme{environ[‘PATH_INFO’]}", ‘rb’) as challenge:
            key = challenge.read()
        start_response(‘200 OK’, [(‘Content-Type’, ‘text/html’)])
        return [key]
    except FileNotFoundError:
    # Will lead to a 404 error
        pass

Das ist natürlich nicht komplett, das wäre nur einen Anfang.

Gruß

Arnaud

highTower · 9. August 2022 um 12:15

Hallo Arnaud,

sorry - wird gemacht.

Das klingt doch erstmal gut!

Bzgl. Lets-Encrypt müsste ich mich erst mit den Spielregeln von Ajenti & Co vertraut machen. Was nutzt ihr für eine IDE und Work Enviroment?

Und beim SSL-Zertifikat ist es unüblich, dass key und cert in einer Datei sind.

Viele Grüße

highTower

Arnaud · 9. August 2022 um 13:09

Hallo higTower,

Ich verwende persönnlich PyCharm. Ich habe einen Skript geschrieben, um Ajenti + DevTool automatisch zu installieren. Davon habe ich nicht viel Feedback gekriegt, ich hoffe es funktioniert :

github.com

ajenti/ajenti/blob/master/scripts/install-dev.sh

#!/bin/bash
if [ "$(id -u)" != "0" ]; then
   echo "This script must be run as root" 1>&2
   exit 1
fi

DISTRO=
OS=
PYTHON3=$(which python3)

msg()
{
    message=$1
    echo
    # Bold and green font
    echo -e "\e[1m\e[92m$message\e[39m\e[0m"
    echo
}

if grep 'Debian' /etc/issue > /dev/null 2>&1 ; then

Diese Datei wurde abgeschnitten. Original zeigen

Python Abhängigkeiten werden in virtual env installiert ( in /opt ).

Ja, das stört mich auch, ich wollte es trennen, was jetzt auch machbar ist, aber das ist ein Breaking change, und die Umstellung muss gut vorbereitet sein. Als ich im Projekt angefangen habe war es so, und ich habe viele Zeit gebracht um das ganze zu lernen. Die Umstellung auf Python3 war wegen SSL relativ kompliziert und seit dem habe ich das nicht bearbeitet (außer für SIcherheitsgründen : Replace deprecated digest sha1 with sha256. Breaks actual client cert. · ajenti/ajenti@960ab68 · GitHub ).

Gruß

Arnaud

Arnaud · 29. August 2022 um 13:23

Hallo,

Das ist erledigt :

Gruß

Arnaud