Skip to content

Commit

Permalink
Iets schonere fix voor check production server
Browse files Browse the repository at this point in the history
  • Loading branch information
M2WZ authored Nov 23, 2023
1 parent e8ff10c commit 251ed02
Show file tree
Hide file tree
Showing 3 changed files with 11 additions and 9 deletions.
8 changes: 3 additions & 5 deletions lib/common/Mail.php
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@
* @author C.S.R. Delft <pubcie@csrdelft.nl>
* @author P.W.G. Brussee <brussee@live.nl>
*
* Alle mailadressen in to of bcc zullen als de host niet syrinx is
* Alle mailadressen in to of bcc zullen als de host niet production is
* worden aangepast naar pubcie@csrdelft.nl
*/
class Mail
Expand Down Expand Up @@ -59,7 +59,7 @@ public function addTo(array $to)
}

/**
* Mails uit testomgevingen moet en niet naar andere dingen dan naar
* Mails uit testomgevingen moeten niet naar andere dingen dan naar
* het pubcie-mailadres.
* @param string $email
* @return string
Expand All @@ -75,9 +75,7 @@ private function productionSafe(string $email): string

public function inDebugMode(): bool
{
// Dirty fix voor veiligheidslek... Welke feut heeft dit geprogrammeerd ;)
// return !HostUtil::isSyrinx();
return false;
return !HostUtil::isProduction();
}

public function addBcc(array $bcc)
Expand Down
10 changes: 7 additions & 3 deletions lib/common/Util/HostUtil.php
Original file line number Diff line number Diff line change
Expand Up @@ -7,12 +7,16 @@
final class HostUtil
{
/**
* Is de huidige host genaamd 'syrinx'?
* Is de huidige host de production server?
* @return boolean
*/
public static function isSyrinx()
public static function isProduction()
{
return 'syrinx' === php_uname('n');
// Controleer voor alle namen die de production server had/heeft/zal hebben
return (php_uname('n') === 'syrinx'
|| php_uname('n') === 'tzdturbo'
|| php_uname('n') === 'albatros'
));
}

public static function isCLI()
Expand Down
2 changes: 1 addition & 1 deletion lib/configuratie.include.php
Original file line number Diff line number Diff line change
Expand Up @@ -94,7 +94,7 @@
}
}

if (HostUtil::isCI() && HostUtil::isSyrinx()) {
if (HostUtil::isCI() && HostUtil::isProduction()) {
die('Syrinx is geen Travis!');
}

Expand Down

0 comments on commit 251ed02

Please sign in to comment.