Merge pull request #1768 from automatisch/rest-create-access-token
feat: Implement create access token API endpoint
This commit is contained in:
@@ -0,0 +1,13 @@
|
|||||||
|
import User from '../../../../models/user.js';
|
||||||
|
import { renderObject, renderError } from '../../../../helpers/renderer.js';
|
||||||
|
|
||||||
|
export default async (request, response) => {
|
||||||
|
const { email, password } = request.body;
|
||||||
|
const token = await User.authenticate(email, password);
|
||||||
|
|
||||||
|
if (token) {
|
||||||
|
return renderObject(response, { token });
|
||||||
|
}
|
||||||
|
|
||||||
|
renderError(response, [{ general: ['Incorrect email or password.'] }]);
|
||||||
|
};
|
@@ -0,0 +1,39 @@
|
|||||||
|
import { describe, it, expect, beforeEach } from 'vitest';
|
||||||
|
import request from 'supertest';
|
||||||
|
import app from '../../../../app.js';
|
||||||
|
import { createUser } from '../../../../../test/factories/user';
|
||||||
|
|
||||||
|
describe('POST /api/v1/access-tokens', () => {
|
||||||
|
beforeEach(async () => {
|
||||||
|
await createUser({
|
||||||
|
email: 'user@automatisch.io',
|
||||||
|
password: 'password',
|
||||||
|
});
|
||||||
|
});
|
||||||
|
|
||||||
|
it('should return the token data with correct credentials', async () => {
|
||||||
|
const response = await request(app)
|
||||||
|
.post('/api/v1/access-tokens')
|
||||||
|
.send({
|
||||||
|
email: 'user@automatisch.io',
|
||||||
|
password: 'password',
|
||||||
|
})
|
||||||
|
.expect(200);
|
||||||
|
|
||||||
|
expect(response.body.data.token.length).toBeGreaterThan(0);
|
||||||
|
});
|
||||||
|
|
||||||
|
it('should return error with incorrect credentials', async () => {
|
||||||
|
const response = await request(app)
|
||||||
|
.post('/api/v1/access-tokens')
|
||||||
|
.send({
|
||||||
|
email: 'incorrect@email.com',
|
||||||
|
password: 'incorrectpassword',
|
||||||
|
})
|
||||||
|
.expect(422);
|
||||||
|
|
||||||
|
expect(response.body.errors.general).toEqual([
|
||||||
|
'Incorrect email or password.',
|
||||||
|
]);
|
||||||
|
});
|
||||||
|
});
|
@@ -44,4 +44,22 @@ const renderObject = (response, object, options) => {
|
|||||||
return response.json(computedPayload);
|
return response.json(computedPayload);
|
||||||
};
|
};
|
||||||
|
|
||||||
export { renderObject };
|
const renderError = (response, errors, status, type) => {
|
||||||
|
const errorStatus = status || 422;
|
||||||
|
const errorType = type || 'ValidationError';
|
||||||
|
|
||||||
|
const payload = {
|
||||||
|
errors: errors.reduce((acc, error) => {
|
||||||
|
const key = Object.keys(error)[0];
|
||||||
|
acc[key] = error[key];
|
||||||
|
return acc;
|
||||||
|
}, {}),
|
||||||
|
meta: {
|
||||||
|
type: errorType,
|
||||||
|
},
|
||||||
|
};
|
||||||
|
|
||||||
|
return response.status(errorStatus).send(payload);
|
||||||
|
};
|
||||||
|
|
||||||
|
export { renderObject, renderError };
|
||||||
|
@@ -5,6 +5,7 @@ import crypto from 'node:crypto';
|
|||||||
import appConfig from '../config/app.js';
|
import appConfig from '../config/app.js';
|
||||||
import { hasValidLicense } from '../helpers/license.ee.js';
|
import { hasValidLicense } from '../helpers/license.ee.js';
|
||||||
import userAbility from '../helpers/user-ability.js';
|
import userAbility from '../helpers/user-ability.js';
|
||||||
|
import createAuthTokenByUserId from '../helpers/create-auth-token-by-user-id.js';
|
||||||
import Base from './base.js';
|
import Base from './base.js';
|
||||||
import Connection from './connection.js';
|
import Connection from './connection.js';
|
||||||
import Execution from './execution.js';
|
import Execution from './execution.js';
|
||||||
@@ -161,6 +162,17 @@ class User extends Base {
|
|||||||
: Execution.query();
|
: Execution.query();
|
||||||
}
|
}
|
||||||
|
|
||||||
|
static async authenticate(email, password) {
|
||||||
|
const user = await User.query().findOne({
|
||||||
|
email: email?.toLowerCase() || null,
|
||||||
|
});
|
||||||
|
|
||||||
|
if (user && (await user.login(password))) {
|
||||||
|
const token = createAuthTokenByUserId(user.id);
|
||||||
|
return token;
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
login(password) {
|
login(password) {
|
||||||
return bcrypt.compare(password, this.password);
|
return bcrypt.compare(password, this.password);
|
||||||
}
|
}
|
||||||
|
9
packages/backend/src/routes/api/v1/access-tokens.js
Normal file
9
packages/backend/src/routes/api/v1/access-tokens.js
Normal file
@@ -0,0 +1,9 @@
|
|||||||
|
import { Router } from 'express';
|
||||||
|
import asyncHandler from 'express-async-handler';
|
||||||
|
import createAccessTokenAction from '../../../controllers/api/v1/access-tokens/create-access-token.js';
|
||||||
|
|
||||||
|
const router = Router();
|
||||||
|
|
||||||
|
router.post('/', asyncHandler(createAccessTokenAction));
|
||||||
|
|
||||||
|
export default router;
|
@@ -4,6 +4,7 @@ import webhooksRouter from './webhooks.js';
|
|||||||
import paddleRouter from './paddle.ee.js';
|
import paddleRouter from './paddle.ee.js';
|
||||||
import healthcheckRouter from './healthcheck.js';
|
import healthcheckRouter from './healthcheck.js';
|
||||||
import automatischRouter from './api/v1/automatisch.js';
|
import automatischRouter from './api/v1/automatisch.js';
|
||||||
|
import accessTokensRouter from './api/v1/access-tokens.js';
|
||||||
import usersRouter from './api/v1/users.js';
|
import usersRouter from './api/v1/users.js';
|
||||||
import paymentRouter from './api/v1/payment.ee.js';
|
import paymentRouter from './api/v1/payment.ee.js';
|
||||||
import appAuthClientsRouter from './api/v1/app-auth-clients.js';
|
import appAuthClientsRouter from './api/v1/app-auth-clients.js';
|
||||||
@@ -26,6 +27,7 @@ router.use('/webhooks', webhooksRouter);
|
|||||||
router.use('/paddle', paddleRouter);
|
router.use('/paddle', paddleRouter);
|
||||||
router.use('/healthcheck', healthcheckRouter);
|
router.use('/healthcheck', healthcheckRouter);
|
||||||
router.use('/api/v1/automatisch', automatischRouter);
|
router.use('/api/v1/automatisch', automatischRouter);
|
||||||
|
router.use('/api/v1/access-tokens', accessTokensRouter);
|
||||||
router.use('/api/v1/users', usersRouter);
|
router.use('/api/v1/users', usersRouter);
|
||||||
router.use('/api/v1/payment', paymentRouter);
|
router.use('/api/v1/payment', paymentRouter);
|
||||||
router.use('/api/v1/app-auth-clients', appAuthClientsRouter);
|
router.use('/api/v1/app-auth-clients', appAuthClientsRouter);
|
||||||
|
Reference in New Issue
Block a user