Nemcio/NodeBB
1
0
Fork 0
mirror of https://github.com/NodeBB/NodeBB.git synced 2025-10-26 16:46:12 +01:00
Code Issues Packages Projects Releases Wiki Activity

breaking: remove deprecated uploads.delete

Browse Source
This commit is contained in:
Barış Soner Uşaklı
2021-11-23 15:16:41 -05:00
parent eecd02fbee
commit c93d7fdbdd
3 changed files with 2 additions and 23 deletions
Download Patch File Download Diff File Expand all files Collapse all files

1
src/socket.io/admin.js
View File

@@ -30,7 +30,6 @@ SocketAdmin.email = require('./admin/email');
SocketAdmin.analytics = require('./admin/analytics'); SocketAdmin.analytics = require('./admin/analytics');
SocketAdmin.logs = require('./admin/logs'); SocketAdmin.logs = require('./admin/logs');
SocketAdmin.errors = require('./admin/errors'); SocketAdmin.errors = require('./admin/errors');
SocketAdmin.uploads = require('./admin/uploads');
SocketAdmin.digest = require('./admin/digest'); SocketAdmin.digest = require('./admin/digest');
SocketAdmin.cache = require('./admin/cache'); SocketAdmin.cache = require('./admin/cache');

20
src/socket.io/admin/uploads.js
View File

@@ -1,20 +0,0 @@
'use strict';
const fs = require('fs');
const path = require('path');
const nconf = require('nconf');
const sockets = require('..');
const Uploads = module.exports;
Uploads.delete = function (socket, pathToFile, callback) {
sockets.warnDeprecated(socket, 'DELETE /api/v3/files');
pathToFile = path.join(nconf.get('upload_path'), pathToFile);
if (!pathToFile.startsWith(nconf.get('upload_path'))) {
return callback(new Error('[[error:invalid-path]]'));
}
fs.unlink(pathToFile, callback);
};

4
test/authentication.js
View File

@@ -29,7 +29,7 @@ describe('authentication', () => {
it('should allow login with email for uid 1', async () => { it('should allow login with email for uid 1', async () => {
const oldValue = meta.config.allowLoginWith; const oldValue = meta.config.allowLoginWith;
meta.config.allowLoginWith = 'email'; meta.config.allowLoginWith = 'username-email';
const { res } = await helpers.loginUser('regular@nodebb.org', 'regularpwd'); const { res } = await helpers.loginUser('regular@nodebb.org', 'regularpwd');
assert.strictEqual(res.statusCode, 200); assert.strictEqual(res.statusCode, 200);
meta.config.allowLoginWith = oldValue; meta.config.allowLoginWith = oldValue;
@@ -37,7 +37,7 @@ describe('authentication', () => {
it('second user should fail to login with email since email is not confirmed', async () => { it('second user should fail to login with email since email is not confirmed', async () => {
const oldValue = meta.config.allowLoginWith; const oldValue = meta.config.allowLoginWith;
meta.config.allowLoginWith = 'email'; meta.config.allowLoginWith = 'username-email';
const uid = await user.create({ username: '2nduser', password: '2ndpassword', email: '2nduser@nodebb.org' }); const uid = await user.create({ username: '2nduser', password: '2ndpassword', email: '2nduser@nodebb.org' });
const { res, body } = await helpers.loginUser('2nduser@nodebb.org', '2ndpassword'); const { res, body } = await helpers.loginUser('2nduser@nodebb.org', '2ndpassword');
assert.strictEqual(res.statusCode, 403); assert.strictEqual(res.statusCode, 403);