Compare commits

..

2 Commits

Author SHA1 Message Date
2316663dac Show unsent echomails in cyan not red
All checks were successful
Create Docker Image / Build Docker Image (x86_64) (push) Successful in 1m12s
Create Docker Image / Build Docker Image (arm64) (push) Successful in 1m38s
Create Docker Image / Final Docker Image Manifest (push) Successful in 9s
2024-11-17 21:39:14 +11:00
59af68dca8 Validation to make sure netmails have an INTL kludge, needed to determine address details. Update Packet View to not trigger validation netmails 2024-11-17 21:38:59 +11:00
5 changed files with 16 additions and 7 deletions

View File

@ -874,6 +874,9 @@ class Message extends FTNBase
); );
$validator->after(function($validator) { $validator->after(function($validator) {
if (($this->mo instanceof Netmail) && (! $this->mo->kludges->has('INTL')))
$validator->errors()->add('no-intl','Netmail message is missing INTL KLUDGE.');
if ($this->zone->domain->flatten) { if ($this->zone->domain->flatten) {
if (! $this->zone->domain->zones->pluck('zone_id')->contains($this->fz)) if (! $this->zone->domain->zones->pluck('zone_id')->contains($this->fz))
$validator->errors()->add('invalid-zone',sprintf('Message from zone [%d] doesnt match any zone in domain for packet zone [%d].',$this->fz,$this->zone->zone_id)); $validator->errors()->add('invalid-zone',sprintf('Message from zone [%d] doesnt match any zone in domain for packet zone [%d].',$this->fz,$this->zone->zone_id));

View File

@ -89,10 +89,11 @@ abstract class Packet extends FTNBase implements \Iterator, \Countable
* @param string $name * @param string $name
* @param int $size * @param int $size
* @param System|null $so - The system that sent us the packet, used to figure out domains if the packet is for a different zone * @param System|null $so - The system that sent us the packet, used to figure out domains if the packet is for a different zone
* @param bool $process
* @return Packet * @return Packet
* @throws InvalidPacketException * @throws InvalidPacketException
*/ */
public static function process(mixed $f,string $name,int $size,System $so=NULL): self public static function process(mixed $f,string $name,int $size,System $so=NULL,bool $process=TRUE): self
{ {
Log::debug(sprintf('%s:+ Opening Packet [%s] with size [%d]',self::LOGKEY,$name,$size)); Log::debug(sprintf('%s:+ Opening Packet [%s] with size [%d]',self::LOGKEY,$name,$size));
@ -183,7 +184,7 @@ abstract class Packet extends FTNBase implements \Iterator, \Countable
{ {
// Parse our message // Parse our message
Log::debug(sprintf('%s:- Message at offset [%d] in [%s]',self::LOGKEY,$read_ptr-strlen($readbuf),$name)); Log::debug(sprintf('%s:- Message at offset [%d] in [%s]',self::LOGKEY,$read_ptr-strlen($readbuf),$name));
$o->parseMessage(substr($msgbuf,0,$end)); $o->parseMessage(substr($msgbuf,0,$end),$process);
$msgbuf = substr($msgbuf,$end+3); $msgbuf = substr($msgbuf,$end+3);
continue; continue;
@ -380,16 +381,17 @@ abstract class Packet extends FTNBase implements \Iterator, \Countable
* Parse a message in a mail packet * Parse a message in a mail packet
* *
* @param string $message * @param string $message
* @throws InvalidPacketException|\Exception * @param bool $process
* @throws \Exception
*/ */
private function parseMessage(string $message): void private function parseMessage(string $message,bool $process): void
{ {
Log::info(sprintf('%s:+ Processing packet message [%d] bytes',self::LOGKEY,strlen($message))); Log::info(sprintf('%s:+ Processing packet message [%d] bytes',self::LOGKEY,strlen($message)));
$msg = Message::parseMessage($message,$this->zone); $msg = Message::parseMessage($message,$this->zone);
// If the message is invalid, we'll ignore it // If the message is invalid, we'll ignore it
if ($msg->errors->count()) { if ($process && $msg->errors->count()) {
Log::info(sprintf('%s:- Message [%s] has [%d] errors',self::LOGKEY,$msg->msgid ?: 'No ID',$msg->errors->count())); Log::info(sprintf('%s:- Message [%s] has [%d] errors',self::LOGKEY,$msg->msgid ?: 'No ID',$msg->errors->count()));
// If the messages is not for the right zone, we'll ignore it // If the messages is not for the right zone, we'll ignore it

View File

@ -104,7 +104,7 @@ class HomeController extends Controller
$f = new File($file); $f = new File($file);
foreach ($f as $packet) foreach ($f as $packet)
$pkt->push([$f->itemName()=>Packet::process($packet,$f->itemName(),$f->itemSize())]); $pkt->push([$f->itemName()=>Packet::process($packet,$f->itemName(),$f->itemSize(),NULL,FALSE)]);
} catch (\Exception $e) { } catch (\Exception $e) {
return redirect()->back()->withErrors(sprintf('%s (%s:%d)',$e->getMessage(),$e->getFile(),$e->getLine())); return redirect()->back()->withErrors(sprintf('%s (%s:%d)',$e->getMessage(),$e->getFile(),$e->getLine()));

View File

@ -82,6 +82,10 @@ strong.success {
color: #0e0; color: #0e0;
} }
strong.info {
color: #7cc;
}
strong.gray { strong.gray {
color: #565656; color: #565656;
} }

View File

@ -70,7 +70,7 @@ use App\Models\{Echomail,Netmail};
if ($item->pivot->sent_at) if ($item->pivot->sent_at)
return sprintf('<strong class="success" title="Sent: %s Packet: %s">%s</strong>',$item->pivot->sent_at,$item->pivot->sent_pkt,$item->ftn2d); return sprintf('<strong class="success" title="Sent: %s Packet: %s">%s</strong>',$item->pivot->sent_at,$item->pivot->sent_pkt,$item->ftn2d);
else else
return sprintf('<strong class="danger">%s</strong>',$item->ftn2d); return sprintf('<strong class="info">%s</strong>',$item->ftn2d);
} else { } else {
return sprintf('<strong class="highlight">%s</strong>',$item->ftn2d); return sprintf('<strong class="highlight">%s</strong>',$item->ftn2d);