Guzzle mit CONNECT methode

xXSchrandXx · Dec 1st 2021

Moin,

ich würde gerne mit Guzzle über die Proxy mit einer RCON kommunizieren.

Jedoch habe ich keine Ahnung wie man Guzzle richtig benutzt.

Zurzeit erstelle ich einfach ein stream_socket_client (https://github.com/xXSchrandXx/de.xxschrandxx.wsc.minecraft-api). Dabei habe ich keine Ahnung davon.

Falls es nicht geht (was ich eigentlich denke), würde ich mich auch über Feedback zu dem Verbinden mit der Proxy freuen.

Danke für eure Zeit.

Mit freundlichen Grüßen

xXSchrandXx

Syntafin · Dec 2nd 2021

ich weiß jetzt nicht wie man Guzzle im WCF aufruft, daher mache ich hier einfach Mal mehrere Codeschnipsel, wie ich es angehen würde:

PHP

<?php


namespace App\Http\Controller;


use Guzzle\Client as Guzzle;


// Rest der Klasse
Guzzle::request('GET', '/foo/bar', ['proxy' => 'protocoll://adress']);

Display More

Ansonsten, wie oben und dann als Bleistift:

PHP

$client = new Client([
        'proxy' => 'p://127.1:3128',
]);

Habe das jetzt nur eben am Handy gemacht, daher keine Haftung für Tippfehler!

xXSchrandXx · Dec 2nd 2021

Um den Client aufzurufen habe ich zum testen das benutzt:

PHP

$client = wcf\system\io\HttpFactory::getDefaultClient();

Mein Problem ist nur, dass ich die Verbindung zur Proxy nicht nur für eine Anfrage benutzen möchte.

Ich würde gerne nach dem Connect noch weitere Daten senden und Antworten bekommen.

(Ich glaube mit der wcf\util\HTTPRequest funktioniert das auch nicht.)

**Tim Düsterhus** · Dec 2nd 2021

Hallo,

Guzzle ist ein HTTP-Client. Entsprechend können damit nur (sinnvoll) HTTP-Requests gesendet werden. Der Ansatz in der Datei auf GitHub sieht grundsätzlich richtig aus. Das ist aber alles ein bisschen fast'n'loose:

Es wird nicht berücksichtigt, ob der Proxy überhaupt gesetzt (nicht leer) ist.
Die Proxy-URL wird nicht sauber geparsed, sondern einfach nur mit dem Holzhammer das Protokoll ersetzt. Am besten mit: https://github.com/WoltLab/WCF…es/lib/util/Url.class.php
Die TLS-Validierung wird abgeschaltet (TLS funktioniert aber eh nicht, weil generell tcp gesetzt wird).
Die HTTP-Antwort wird nicht sauber geparsed, sondern einfach nur mal mit einem regulären Ausdruck auf grobe Konformität geprüft.

xXSchrandXx · Dec 2nd 2021

Moin,

danke für deine Hilfe.

Quote from Tim Düsterhus

Die HTTP-Antwort wird nicht sauber geparsed, sondern einfach nur mal mit einem regulären Ausdruck auf grobe Konformität geprüft.

Wie parsed man Diese denn? Einfach explode('\n', $rsp) und dann das Array richtig packen mit $key: $value ist damit nicht gemeint, oder?

**Tim Düsterhus** · Dec 2nd 2021

Hallo,

am einfachsten liest du die Antwort zeilenweise mit fgets bis du eine leere Zeile siehst (nicht fread, das endet nämlich nicht bei Zeilen). Dann endet nämlich die HTTP-Antwort.

Die erste gelesene Zeile ist die Status-Line, die interessiert dich. Da kannst du wohl einfach ^HTTP/([0-9]\.[0-9]) ([0-9]{3}) (.*)\r\n$ drauf. Die HTTP-Version findest du in der ersten Capturing-Gruppe, den Status in der zweiten. Die Version solltest du gegen 1.0 oder 1.1 prüfen. Den Status-Code prüfst du dann, dass der im Bereich von 200 bis 299 liegt.

Die restlichen gelesenen Zeilen kannst du vermutlich einfach ignorieren, du musst sie aber lesen, da sie andernfalls als potentielle RCON-Antwort interpretiert würden.

xXSchrandXx · Dec 2nd 2021

Moin,

kann man bestimmen, wie lang die Zeilen maximal werden?

Zurzeit habe ich das gefühlt, dass fread sehr viel schneller war als fgets.

Daher weiß ich nicht ob für meine Zwecke preg_split('/\n/', fread($this->proxy, 1024)) oder while (!empty($buffer = fgets($this->proxy))) besser wäre.

**Tim Düsterhus** · Dec 2nd 2021

Hallo,

ich kann dir jetzt nicht die PHP-Standardbibliothek erklären. Ich möchte aber zu bedenken geben, dass es auf die Performance des Codes nur in den wenigstens Fällen ankommt. Für die RCON-Kommunikation ist das vermutlich völlig egal. Verwende das was am Ende den lesbarsten Code erzeugt: Und das ist fgets. Wobei die Schleife in deinem Beitrag genau das falsche macht. Du willst lesen, bis zu eine leere Zeile hast. Nicht bis du das Ende des Streams erreicht hast. Schließlich musst du nach dem Proxy-Handshake ja auch noch die eigentliche RCON-Logik implementieren.

xXSchrandXx · Dec 2nd 2021

Auch wenn ich nie die komplette Antwort brauche, ist es also so besser?

PHP

//        $rawRsp = preg_split('/\n/', fread($this->proxy, 1024));

        $rawRsp = [];
        while (!empty($buffer = @fgets($this->proxy))) {
            array_push($rawRsp, $buffer);
        }

        if (empty($rawRsp)) {
            throw new MinecraftException('Response empty.');
        }

        $rsp = [];
        foreach ($rawRsp as $rawLine) {
            $line = preg_split('/:\s*/', $rawLine);
            if (isset($line[1])) {
                $rsp[$line[0]] = $line[1];
            }
        }

        $a = preg_split('/\s/', $rawRsp[0], 2);
        $b = preg_split('/\//', $a[0]);
        $rsp['scheme'] = $b[0];
        $rsp['version'] = $b[1];
        $rsp['statusCode'] = intval($a[1]);

        if (preg_match('/(0-9)\.(0-9)/', $rsp['version'])) {
            throw new MinecraftException('Unknown Version, Version: ' . $rsp['version']);
        }

        if (!(200 >= $rsp['statusCode'] && $rsp['statusCode'] <= 299)) {
            throw new MinecraftException('HTML error ' . $rsp['statusCode']);
        }

Display More

**Tim Düsterhus** · Dec 3rd 2021

Hallo,

ich habe Schwierigkeiten dem Code zu folgen und weiß was da passieren soll. Die Logik ist auf jeden Fall nicht korrekt, weil die Status-Zeile durch Leerzeichen getrennt wird, nicht durch irgendeine beliebige Form von Whitespace.

Ungetestet im WYSIWYG runtergeschrieben:

PHP

<?php
$statusLine = \fgets($this->proxy);
if (!$statusLine) {
 throw new \Exception('Missing status line');
}

if (!\preg_match('/^HTTP\/([0-9]\.[0-9]) ([0-9]{3}) (.*)\r\n$/', $statusLine, $matches)) {
 throw new \Exception('Invalid status line');
}

if ($matches[1] !== '1.0' && $matches[1] !== '1.1') {
 throw new \Exception(\sprintf("Incorrect HTTP version. Expected 1.0 or 1.1, got '%s'.", $matches[1]));
}

if (!(200 <= $matches[2] && $matches[2] < 300)) {
 throw new \Exception(\sprintf("Status code '%d' does not indicate success.", $matches[2]));
}

// Consume the remaining header lines.
while (($line = \fgets($this->proxy)) !== "\r\n") {
 if ($line === false) {
  throw new \Exception('Encountered EOF while searching for the end of proxy response headers');
 }
}

Display More

Guzzle mit CONNECT methode

Participate now!

Share

Similar Threads

Frage zur Styleinstallation

"Failed to HTTPRequest" obwohl Hoster alles neu angepasst hat

HTTPRequest - Timeout funktioniert nicht die 2.

Log voll mit "Failed to HTTPRequest"