Skip to content

Commit 1c461c7

Browse files
authored
Merge pull request #690 from frknbasaran/sdk-response
[api] app_key check enabled for sdk write requests.
2 parents db7842d + fd80e6a commit 1c461c7

File tree

1 file changed

+5
-8
lines changed

1 file changed

+5
-8
lines changed

api/utils/requestProcessor.js

+5-8
Original file line numberDiff line numberDiff line change
@@ -1138,10 +1138,6 @@ const processRequest = (params) => {
11381138
});
11391139
});
11401140

1141-
if (!plugins.getConfig("api", params.app && params.app.plugins, true).safe && !params.res.finished) {
1142-
common.returnMessage(params, 200, 'Success');
1143-
}
1144-
11451141
break;
11461142
}
11471143
case '/o/users': {
@@ -1981,10 +1977,7 @@ const validateAppForWriteAPI = (params, done, try_times) => {
19811977
}
19821978
common.db.collection('apps').findOne({'key': params.qstring.app_key}, (err, app) => {
19831979
if (!app) {
1984-
if (plugins.getConfig("api", params.app && params.app.plugins, true).safe) {
1985-
common.returnMessage(params, 400, 'App does not exist');
1986-
}
1987-
1980+
common.returnMessage(params, 400, 'App does not exist');
19881981
return done ? done() : false;
19891982
}
19901983

@@ -2152,6 +2145,10 @@ const validateAppForWriteAPI = (params, done, try_times) => {
21522145
}
21532146
});
21542147
});
2148+
if (!plugins.getConfig("api", params.app && params.app.plugins, true).safe && !params.res.finished) {
2149+
common.returnMessage(params, 200, 'Success');
2150+
return;
2151+
}
21552152
});
21562153
};
21572154

0 commit comments

Comments
 (0)