Skip to content

Commit cf1b702

Browse files
committed
Remove salt from app config since it was only there for encryption anyway and isn't used anywhere.
1 parent 633121d commit cf1b702

3 files changed

Lines changed: 0 additions & 28 deletions

File tree

app/Config/App.php

Lines changed: 0 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -264,19 +264,4 @@ class App extends BaseConfig
264264
*/
265265
public $CSPEnabled = false;
266266

267-
/*
268-
|--------------------------------------------------------------------------
269-
| Application Salt
270-
|--------------------------------------------------------------------------
271-
|
272-
| The $salt can be used anywhere within the application that you need
273-
| to provide secure data. It should be different for every application
274-
| and can be of any length, though the more random the characters
275-
| the better.
276-
|
277-
*/
278-
public $salt = '';
279-
280-
//--------------------------------------------------------------------
281-
282267
}

system/Debug/Toolbar/Collectors/Config.php

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -60,7 +60,6 @@ public static function display(): array
6060
'timezone' => app_timezone(),
6161
'locale' => Services::request()->getLocale(),
6262
'cspEnabled' => $config->CSPEnabled,
63-
'salt' => $config->salt,
6463
];
6564
}
6665
}

system/Debug/Toolbar/Views/_config.tpl.php

Lines changed: 0 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -44,17 +44,5 @@
4444
<td>Content Security Policy Enabled:</td>
4545
<td>{ if $cspEnabled } Yes { else } No { endif }</td>
4646
</tr>
47-
<tr>
48-
<td>Salt Set?:</td>
49-
<td>
50-
{ if $salt == '' }
51-
<div class="warning">
52-
You have not defined an application-wide $salt. This could lead to a less secure site.
53-
</div>
54-
{ else }
55-
Set
56-
{ endif }
57-
</td>
58-
</tr>
5947
</tbody>
6048
</table>

0 commit comments

Comments
 (0)