Throw more when using RCON

SourceQuery/GoldSourceRcon.php

@@ -71,7 +71,7 @@
 			
 			if( $Buffer->GetByte( ) !== SourceQuery::S2A_RCON )
 			{
-				return false;
+				throw new InvalidPacketException( 'Invalid rcon response.', InvalidPacketException::PACKET_HEADER_MISMATCH );
 			}
 			
 			$Buffer  = $Buffer->Get( );
@@ -114,7 +114,7 @@
 		{
 			if( !$this->RconChallenge )
 			{
-				return false;
+				throw new AuthenticationException( 'Tried to execute a RCON command before successful authorization.', AuthenticationException::BAD_PASSWORD );
 			}
 			
 			$this->Write( 0, 'rcon ' . $this->RconChallenge . ' "' . $this->RconPassword . '" ' . $Command . "\0" );
@@ -132,11 +132,9 @@
 			
 			if( $Buffer->Get( 14 ) !== 'challenge rcon' )
 			{
-				return false;
+				throw new AuthenticationException( 'Failed to get RCON challenge.', AuthenticationException::BAD_PASSWORD );
 			}
 			
 			$this->RconChallenge = Trim( $Buffer->Get( ) );
-			
-			return true;
 		}
 	}

SourceQuery/SourceQuery.php

@@ -503,8 +503,6 @@
 		 * @throws AuthenticationException
 		 * @throws InvalidPacketException
 		 * @throws SocketException
-		 *
-		 * @return bool True if authentication succeeded, false on failure
 		 */
 		public function SetRconPassword( $Password )
 		{
@@ -530,8 +528,7 @@
 			}
 			
 			$this->Rcon->Open( );
-			
+			$this->Rcon->Authorize( $Password );
-			return $this->Rcon->Authorize( $Password );
 		}
 		
 		/**

SourceQuery/SourceRcon.php

@@ -137,7 +137,7 @@
 			}
 			else if( $Type !== SourceQuery::SERVERDATA_RESPONSE_VALUE )
 			{
-				return false;
+				throw new InvalidPacketException( 'Invalid rcon response.', InvalidPacketException::PACKET_HEADER_MISMATCH );
 			}
 			
 			$Data = $Buffer->Get( );
@@ -197,7 +197,5 @@
 			{
 				throw new AuthenticationException( 'RCON authorization failed.', AuthenticationException::BAD_PASSWORD );
 			}
-			
-			return true;
 		}
 	}