Skip to content

Commit

Permalink
Fix header warnings from log flushing
Browse files Browse the repository at this point in the history
Fixes #1168

Co-authored-by: Jaime Pérez Crespo <jaime.perez@uninett.no>
  • Loading branch information
toth-dev and jaimeperez committed Sep 2, 2019
1 parent 9d5bd99 commit 55c9ba7
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 2 deletions.
4 changes: 3 additions & 1 deletion lib/SimpleSAML/Logger.php
Original file line number Diff line number Diff line change
Expand Up @@ -386,7 +386,9 @@ public static function popErrorMask()
private static function defer($level, $message, $stats)
{
// save the message for later
self::$earlyLog[] = ['level' => $level, 'string' => $message, 'statsLog' => $stats];
self::$earlyLog[] = [
'level' => $level, 'string' => $message, 'statsLog' => $stats
];

// register a shutdown handler if needed
if (!self::$shutdownRegistered) {
Expand Down
2 changes: 1 addition & 1 deletion lib/SimpleSAML/SessionHandlerPHP.php
Original file line number Diff line number Diff line change
Expand Up @@ -68,7 +68,7 @@ protected function __construct()
}

if (!empty($this->cookie_name)) {
session_name($this->cookie_name);
@session_name($this->cookie_name);
} else {
$this->cookie_name = session_name();
}
Expand Down

0 comments on commit 55c9ba7

Please sign in to comment.