Dockery/Homarr
1
0
Fork 0
mirror of https://github.com/ajnart/homarr.git synced 2025-11-11 16:05:47 +01:00
Code Issues Packages Projects Releases Wiki Activity

✨ Add logging for config deletion

Browse Source
This commit is contained in:
Manuel
2023-01-14 23:28:59 +01:00
parent e05278c618
commit 1aac83d33f
2 changed files with 3 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
src/components/Config/ConfigChanger.tsx
View File

@@ -16,9 +16,6 @@ export default function ConfigChanger() {
const [isRefreshing, toggle] = useToggle(); const [isRefreshing, toggle] = useToggle();
const onConfigChange = (value: string) => { const onConfigChange = (value: string) => {
// TODO: check what should happen here with @manuel-rw
// Wheter it should check for the current url and then load the new config only on index
// Or it should always load the selected config and open index or ? --> change url to page
setCookie('config-name', value ?? 'default', { setCookie('config-name', value ?? 'default', {
maxAge: 60 * 60 * 24 * 30, maxAge: 60 * 60 * 24 * 30,
sameSite: 'strict', sameSite: 'strict',

3
src/pages/api/configs/[slug].ts
View File

@@ -130,6 +130,7 @@ function Delete(req: NextApiRequest, res: NextApiResponse<any>) {
// Get the slug of the request // Get the slug of the request
const { slug } = req.query as { slug: string }; const { slug } = req.query as { slug: string };
if (!slug) { if (!slug) {
Consola.error('Rejected config deletion request because config slug was not present');
return res.status(400).json({ return res.status(400).json({
message: 'Wrong request', message: 'Wrong request',
}); });
@@ -145,6 +146,7 @@ function Delete(req: NextApiRequest, res: NextApiResponse<any>) {
// If the target is not in the list of files, return an error // If the target is not in the list of files, return an error
if (!configs.includes(slug)) { if (!configs.includes(slug)) {
Consola.error(`Rejected config deletion request because config name '${slug}' was not included in present configurations`);
return res.status(404).json({ return res.status(404).json({
message: 'Target not found', message: 'Target not found',
}); });
@@ -152,6 +154,7 @@ function Delete(req: NextApiRequest, res: NextApiResponse<any>) {
// Delete the file // Delete the file
fs.unlinkSync(path.join('data/configs', `${slug}.json`)); fs.unlinkSync(path.join('data/configs', `${slug}.json`));
Consola.info(`Successfully deleted configuration '${slug}' from your file system`);
return res.status(200).json({ return res.status(200).json({
message: 'Configuration deleted with success', message: 'Configuration deleted with success',
}); });