Skip to content

Commit c1346c4

Browse files
Merge pull request #3 from andresWeitzel/security-02-fix-function-inside-loop-code-scanning
security-02-fix-function-inside-loop-code-scanning
2 parents 4662fe8 + 3f9c1c4 commit c1346c4

File tree

1 file changed

+13
-11
lines changed

1 file changed

+13
-11
lines changed

src/api-integration/helpers/request/get-data-from-address.js

Lines changed: 13 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -4,14 +4,15 @@ const axios = require("axios");
44
const URL = process.env.WHOIS_BASE_URL;
55
let ip;
66
let ipsLength = 100;
7+
let axiosResponse;
78
let responseData;
89
let arrayDataAddress;
910

1011
const getDataFromSpecificAddress = async (ip) => {
1112
try {
12-
await axios.get(URL + ip).then(function (response) {
13-
responseData = response?.data;
14-
});
13+
axiosResponse = await axios.get(URL + ip);
14+
15+
responseData = axiosResponse?.data || null;
1516
} catch (error) {
1617
console.log(error);
1718
responseData = null;
@@ -24,13 +25,14 @@ const getDataFromRandomAddress = async () => {
2425
arrayDataAddress = [];
2526
for (let i = 0; i < ipsLength; i++) {
2627
ip = `8.8.4.${i}`;
27-
await axios.get(URL + ip).then(function (response) {
28-
responseData = response?.data;
29-
if (responseData != undefined || null) {
30-
arrayDataAddress.push(responseData);
31-
//console.log(responseData);
32-
}
33-
});
28+
axiosResponse = await axios.get(URL + ip);
29+
30+
responseData = axiosResponse?.data || null;
31+
32+
if (responseData != undefined || null) {
33+
arrayDataAddress.push(responseData);
34+
//console.log(responseData);
35+
}
3436
}
3537
} catch (error) {
3638
console.log(error);
@@ -42,5 +44,5 @@ const getDataFromRandomAddress = async () => {
4244

4345
module.exports = {
4446
getDataFromSpecificAddress,
45-
getDataFromRandomAddress
47+
getDataFromRandomAddress,
4648
};

0 commit comments

Comments
 (0)