Use challenge straight away if available

This reverts commit 7231921b0b.

Examples/View.php

@@ -17,6 +17,7 @@
 	$Info    = Array( );
 	$Rules   = Array( );
 	$Players = Array( );
+	$Exception = null;
 	
 	try
 	{
@@ -87,7 +88,7 @@
 	</div>
 		
 	<div class="container">
-<?php if( $Exception !== '' ): ?>
+<?php if( $Exception !== null ): ?>
 		<div class="panel panel-error">
 			<pre class="panel-body"><?php echo htmlspecialchars( $Exception->__toString( ) ); ?></pre>
 		</div>

README.md

@@ -1,7 +1,5 @@
 # PHP Source Query
 
-[![Build Status](https://img.shields.io/travis/com/xPaw/PHP-Source-Query.svg)](https://travis-ci.com/xPaw/PHP-Source-Query)
-[![Test Coverage](https://img.shields.io/coveralls/github/xPaw/PHP-Source-Query/master.svg)](https://coveralls.io/github/xPaw/PHP-Source-Query)
 [![Packagist Downloads](https://img.shields.io/packagist/dt/xpaw/php-source-query-class.svg)](https://packagist.org/packages/xpaw/php-source-query-class)
 [![Packagist Version](https://img.shields.io/packagist/v/xpaw/php-source-query-class.svg)](https://packagist.org/packages/xpaw/php-source-query-class)
 

SourceQuery/BaseSocket.php

@@ -41,7 +41,6 @@
 		abstract public function Close( ) : void;
 		abstract public function Open( string $Address, int $Port, int $Timeout, int $Engine ) : void;
 		abstract public function Write( int $Header, string $String = '' ) : bool;
-		abstract public function WritePadded( int $Header, string $String = '' ) : bool;
 		abstract public function Read( int $Length = 1400 ) : Buffer;
 		
 		protected function ReadInternal( Buffer $Buffer, int $Length, callable $SherlockFunction ) : Buffer

SourceQuery/Socket.php

@@ -61,26 +61,6 @@
 			return $Length === FWrite( $this->Socket, $Command, $Length );
 		}
 		
-		/**
-		 * Write a request packge to the socket. Pads it up to 1200 bytes to prevent reflective DoS.
-		 *
-		 * @see https://steamcommunity.com/discussions/forum/14/2989789048633291344/
-		 * @return bool Whether fwrite succeeded.
-		 */
-		public function WritePadded( int $Header, string $String = '' ) : bool
-		{
-			$Command = pack( 'ccccca*', 0xFF, 0xFF, 0xFF, 0xFF, $Header, $String );
-			$Length  = strlen( $Command );
-
-			if( $Length < 1200 )
-			{
-				$Command .= str_repeat( "\0", 1200 - $Length );
-				$Length = 1200;
-			}
-
-			return $Length === fwrite( $this->Socket, $Command, $Length );
-		}
-
 		/**
 		 * Reads from socket and returns Buffer.
 		 *

SourceQuery/SourceQuery.php

@@ -200,12 +200,28 @@
 				throw new SocketException( 'Not connected.', SocketException::NOT_CONNECTED );
 			}
 			
-			$this->Socket->WritePadded( self::A2S_INFO, "Source Engine Query\0" );
+			if( $this->Challenge )
+			{
+				$this->Socket->Write( self::A2S_INFO, "Source Engine Query\0" . $this->Challenge );
+			}
+			else
+			{
+				$this->Socket->Write( self::A2S_INFO, "Source Engine Query\0" );
+			}
+
 			$Buffer = $this->Socket->Read( );
-			
 			$Type = $Buffer->GetByte( );
 			$Server = [];
 			
+			if( $Type === self::S2A_CHALLENGE )
+			{
+				$this->Challenge = $Buffer->Get( 4 );
+
+				$this->Socket->Write( self::A2S_INFO, "Source Engine Query\0" . $this->Challenge );
+				$Buffer = $this->Socket->Read( );
+				$Type = $Buffer->GetByte( );
+			}
+
 			// Old GoldSource protocol, HLTV still uses it
 			if( $Type === self::S2A_INFO_OLD && $this->Socket->Engine === self::GOLDSOURCE )
 			{
@@ -365,7 +381,7 @@
 			
 			$this->GetChallenge( self::A2S_PLAYER, self::S2A_PLAYER );
 			
-			$this->Socket->WritePadded( self::A2S_PLAYER, $this->Challenge );
+			$this->Socket->Write( self::A2S_PLAYER, $this->Challenge );
 			$Buffer = $this->Socket->Read( 14000 ); // Moronic Arma 3 developers do not split their packets, so we have to read more data
 			// This violates the protocol spec, and they probably should fix it: https://developer.valvesoftware.com/wiki/Server_queries#Protocol
 			
@@ -411,7 +427,7 @@
 			
 			$this->GetChallenge( self::A2S_RULES, self::S2A_RULES );
 			
-			$this->Socket->WritePadded( self::A2S_RULES, $this->Challenge );
+			$this->Socket->Write( self::A2S_RULES, $this->Challenge );
 			$Buffer = $this->Socket->Read( );
 			
 			$Type = $Buffer->GetByte( );
@@ -452,13 +468,10 @@
 			
 			if( $this->UseOldGetChallengeMethod )
 			{
-				$this->Socket->Write( self::A2S_SERVERQUERY_GETCHALLENGE, "\xFF\xFF\xFF\xFF" );
-			}
-			else
-			{
-				$this->Socket->WritePadded( $Header, "\xFF\xFF\xFF\xFF" );
+				$Header = self::A2S_SERVERQUERY_GETCHALLENGE;
 			}
 			
+			$this->Socket->Write( $Header, "\xFF\xFF\xFF\xFF" );
 			$Buffer = $this->Socket->Read( );
 			
 			$Type = $Buffer->GetByte( );

Tests/Tests.php

@@ -38,11 +38,6 @@
 			return true;
 		}
 		
-		public function WritePadded( int $Header, string $String = '' ) : bool
-		{
-			return true;
-		}
-		
 		public function Read( int $Length = 1400 ) : Buffer
 		{
 			$Buffer = new Buffer( );

composer.json

@@ -23,8 +23,7 @@
 	"require-dev":
 	{
 		"phpunit/phpunit": "9.2",
-		"vimeo/psalm": "^3.12",
-		"php-coveralls/php-coveralls": "^2.2"
+		"vimeo/psalm": "^3.12"
 	},
 	"autoload":
 	{