Infrastructure/pingvin-share
0
Fork 0
mirror of https://github.com/stonith404/pingvin-share.git synced 2025-01-15 01:14:27 -05:00
Code Issues Activity

chore: sanitize appUrl to remove trailing slash in updateConfigVariable function (#496)

Browse source
This commit is contained in:
Cabeza 2024-06-10 05:01:08 -04:00 • committed by GitHub
parent 9268e35141
commit 962ec27df4
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
1 changed files with 8 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
frontend/src/pages/admin/config/[category].tsx
View file

@ -72,6 +72,10 @@ export default function AppShellDemo() {
}; };
const updateConfigVariable = (configVariable: UpdateConfig) => { const updateConfigVariable = (configVariable: UpdateConfig) => {
if (configVariable.key === 'general.appUrl') {
configVariable.value = sanitizeUrl(configVariable.value);
}
const index = updatedConfigVariables.findIndex( const index = updatedConfigVariables.findIndex(
(item) => item.key === configVariable.key, (item) => item.key === configVariable.key,
); );
@ -86,6 +90,10 @@ export default function AppShellDemo() {
} }
}; };
const sanitizeUrl = (url: string): string => {
return url.endsWith('/') ? url.slice(0, -1) : url;
};
useEffect(() => { useEffect(() => {
configService.getByCategory(categoryId).then((configVariables) => { configService.getByCategory(categoryId).then((configVariables) => {
setConfigVariables(configVariables); setConfigVariables(configVariables);