Attempt to work out a packets domain from a nodes addresses for looking at a default

app/Classes/FTN/Packet.php

@@ -3,6 +3,7 @@
 namespace App\Classes\FTN;
 
 use Carbon\Carbon;
+use Illuminate\Database\Eloquent\ModelNotFoundException;
 use Illuminate\Support\Arr;
 use Illuminate\Support\Collection;
 use Illuminate\Support\Facades\Log;
@@ -11,7 +12,7 @@ use Symfony\Component\HttpFoundation\File\File;
 
 use App\Classes\FTN as FTNBase;
 use App\Exceptions\InvalidPacketException;
-use App\Models\{Address,Domain,Echomail,Netmail,Software,System,Zone};
+use App\Models\{Address,Echomail,Netmail,Software,System,Zone};
 use App\Notifications\Netmails\{EchomailBadAddress,NetmailBadAddress};
 
 /**
@@ -87,11 +88,11 @@ abstract class Packet extends FTNBase implements \Iterator, \Countable
 	 * @param mixed $f File handler returning packet data
 	 * @param string $name
 	 * @param int $size
-	 * @param Domain|null $domain
+	 * @param System|null $so - The system that sent us the packet, used to figure out domains if the packet is for a different zone
 	 * @return Packet
 	 * @throws InvalidPacketException
 	 */
-	public static function process(mixed $f,string $name,int $size,Domain $domain=NULL): self
+	public static function process(mixed $f,string $name,int $size,System $so=NULL): self
 	{
 		Log::debug(sprintf('%s:+ Opening Packet [%s] with size [%d]',self::LOGKEY,$name,$size));
 
@@ -140,22 +141,29 @@ abstract class Packet extends FTNBase implements \Iterator, \Countable
 
 		Log::info(sprintf('%s:- Packet [%s] is a [%s] packet',self::LOGKEY,$o->name,get_class($o)));
 
-		// Work out the packet zone
-		if ($o->fz && ($o->fd || $domain)) {
-			$o->zone = Zone::select('zones.*')
-				->join('domains',['domains.id'=>'zones.domain_id'])
-				->where('zone_id',$o->fz)
-				->where('name',$o->fd ?: $domain->name)
-				->single();
+		if ($o->fz && ($o->fd || $so)) {
+			Log::alert(sprintf('%s:! No domain in the packet, work it out from the system [%d] for zone [%d]',self::LOGKEY,$so->name,$o->fz));
+
+			if (($x=$so->zones->where('zone_id',$o->fz)->unique('domain_id'))->count() === 1) {
+				$o->zone = $x->pop();
+
+			} else {
+				Log::alert(sprintf('%s:! Node [%s] has two zones with [%d]',self::LOGKEY,$so->name,$o->fz));
+			}
 		}
 
 		// If zone is not set, then we need to use a default zone - the messages may not be from this zone.
 		if (empty($o->zone)) {
 			Log::alert(sprintf('%s:! We couldnt work out the packet zone, so we have fallen back to the default for [%d]',self::LOGKEY,$o->fz));
 
-			$o->zone = Zone::where('zone_id',$o->fz)
-				->where('default',TRUE)
-				->singleOrFail();
+			try {
+				$o->zone = Zone::where('zone_id',$o->fz)
+					->where('default',TRUE)
+					->singleOrFail();
+
+			} catch (ModelNotFoundException $e) {
+				throw new InvalidPacketException(sprintf('%s:! We couldnt work out the packet zone, and there isnt a default for[%d]',self::LOGKEY,$o->fz));
+			}
 		}
 
 		Log::info(sprintf('%s:- Packet Dated [%s] from [%s] to [%s]',self::LOGKEY,$o->date,$o->fftn_t,$o->tftn_t));
@@ -186,7 +194,7 @@ abstract class Packet extends FTNBase implements \Iterator, \Countable
 			}
 
 			// If we get here
-			throw new InvalidPacketException(sprintf('Cannot determine END of message/packet: %s|%s',get_class($o),hex_dump($message)));;
+			throw new InvalidPacketException(sprintf('Cannot determine END of message/packet: %s|%s',get_class($o),hex_dump($message)));
 		}
 
 		if ($msgbuf)

app/Classes/File/Receive.php

@@ -128,9 +128,9 @@ class Receive extends Base
 							// If packet is greater than a size, lets queue it
 							if ($this->queue || ($this->receiving->size > config('fido.queue_size',0))) {
 								Log::info(sprintf('%s:- Packet [%s] will be sent to the queue for processing because its [%d] size, or queue forced',self::LOGKEY,$this->receiving->full_name,$this->receiving->size));
-								PacketProcess::dispatch($this->receiving->rel_name,$this->ao->zone->domain,FALSE,$rcvd_time);
+								PacketProcess::dispatch($this->receiving->rel_name,$this->ao->system,FALSE,$rcvd_time);
 							} else
-								PacketProcess::dispatchSync($this->receiving->rel_name,$this->ao->zone->domain,TRUE,$rcvd_time);
+								PacketProcess::dispatchSync($this->receiving->rel_name,$this->ao->system,TRUE,$rcvd_time);
 
 						} catch (\Exception $e) {
 							Log::error(sprintf('%s:! Got error dispatching packet [%s] (%d:%s-%s).',self::LOGKEY,$this->receiving->rel_name,$e->getLine(),$e->getFile(),$e->getMessage()));