banquise/pelican-panel-mirror
3
0
Fork 1
mirror of https://github.com/pelican-dev/panel.git synced 2025-05-19 23:24:46 +02:00
Code Issues Packages Projects Releases Wiki Activity

Fix escaping for EnvironmentWriterTrait (#610)

Browse Source 
* fix escaping for EnvironmentWriterTrait

* remove alphaNum from app name field

* add test for `'` escaping
This commit is contained in:
Boy132 2024-10-08 23:46:06 +02:00 committed by GitHub
parent a946669dc8
commit e23a4a667a
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
3 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
app/Filament/Pages/Settings.php
View File

@ -92,7 +92,6 @@ class Settings extends Page implements HasForms
TextInput::make('APP_NAME')
->label('App Name')
->required()
->alphaNum()
->default(env('APP_NAME', 'Pelican')),
TextInput::make('APP_FAVICON')
->label('App Favicon')

2
app/Traits/EnvironmentWriterTrait.php
View File

@ -14,7 +14,7 @@ trait EnvironmentWriterTrait
public function escapeEnvironmentValue(string $value): string
{
if (!preg_match('/^\"(.*)\"$/', $value) && preg_match('/([^\w.\-+\/])+/', $value)) {
return sprintf('"%s"', addslashes($value));
return sprintf('"%s"', addcslashes($value, '\\"'));
}
return $value;

1
tests/Unit/Helpers/EnvironmentWriterTraitTest.php
View File

@ -23,6 +23,7 @@ class EnvironmentWriterTraitTest extends TestCase
['foo', 'foo'],
['abc123', 'abc123'],
['val"ue', '"val\"ue"'],
['val\'ue', '"val\'ue"'],
['my test value', '"my test value"'],
['mysql_p@assword', '"mysql_p@assword"'],
['mysql_p#assword', '"mysql_p#assword"'],