Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[FIX] Corrigindo Vulnerabilidade #16

Merged
merged 1 commit into from
Sep 2, 2024
Merged
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
12 changes: 8 additions & 4 deletions src/Controllers/bankAccountController.js
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@
const createBankAccount = async (req, res) => {
try {
// Log dos dados recebidos para depuração
console.log("Dados recebidos:", req.body);

Check warning on line 8 in src/Controllers/bankAccountController.js

View workflow job for this annotation

GitHub Actions / lint

Unexpected console statement

// Acessando dados diretamente
const formData = req.body.formData || {}; // Fallback para objeto vazio se formData estiver ausente
Expand All @@ -16,21 +16,25 @@
}

// Verifica se já existe uma conta bancária com o mesmo nome
const existingName = await BankAccount.findOne({ name });
if (typeof name == "string") {
const existingName = await BankAccount.findOne({ name });

if (existingName) {
// Mensagem de erro se o nome já estiver repetido
return res.status(409).send({ error: "Nome já cadastrado" }); // Código 409 para conflito
if (existingName) {
// Mensagem de erro se o nome já estiver repetido
return res.status(409).send({ error: "Nome já cadastrado" }); // Código 409 para conflito
}
} else {
return res.status(500).send({ error: "Tipo de dado incorreto" });
}

// Criação de uma nova conta bancária
const bankAccount = new BankAccount(formData);
await bankAccount.save(); // Salvando a conta bancária no banco de dados
console.log("Conta bancária criada com sucesso:", bankAccount);

Check warning on line 33 in src/Controllers/bankAccountController.js

View workflow job for this annotation

GitHub Actions / lint

Unexpected console statement
res.status(201).send(bankAccount); // Enviando resposta de sucesso
} catch (error) {
// Log do erro para depuração
console.error("Erro ao criar conta bancária:", error.message);

Check warning on line 37 in src/Controllers/bankAccountController.js

View workflow job for this annotation

GitHub Actions / lint

Unexpected console statement
res.status(400).send({ error: error.message }); // Enviando erro caso ocorra
}
};
Expand All @@ -53,7 +57,7 @@
res.status(200).json(bankAccount); // Enviando conta bancária encontrada
} catch (error) {
// Log do erro para depuração
console.error("Erro ao buscar conta bancária:", error.message);

Check warning on line 60 in src/Controllers/bankAccountController.js

View workflow job for this annotation

GitHub Actions / lint

Unexpected console statement
res.status(500).send({ error: error.message }); // Enviando mensagem de erro
}
};
Expand All @@ -67,7 +71,7 @@
res.status(200).send({ message: "Conta deletada com sucesso" });
} catch (error) {
// Log do erro para depuração
console.error("Erro ao deletar conta bancária:", error.message);

Check warning on line 74 in src/Controllers/bankAccountController.js

View workflow job for this annotation

GitHub Actions / lint

Unexpected console statement
res.status(500).send({ error: error.message }); // Enviando mensagem de erro
}
};
Expand All @@ -80,7 +84,7 @@
}

// Log dos dados recebidos
console.log("Dados recebidos para atualização:", req.body);

Check warning on line 87 in src/Controllers/bankAccountController.js

View workflow job for this annotation

GitHub Actions / lint

Unexpected console statement

// Atualizar apenas os campos fornecidos
const bankAccount = await BankAccount.findByIdAndUpdate(id, req.body, {
Expand All @@ -93,7 +97,7 @@

res.status(200).send(bankAccount);
} catch (error) {
console.error("Erro ao atualizar conta bancária:", error.message);

Check warning on line 100 in src/Controllers/bankAccountController.js

View workflow job for this annotation

GitHub Actions / lint

Unexpected console statement
res.status(500).send({ error: error.message });
}
};
Expand Down
Loading