mudhorn/n8n
1
0
Fork 0
mirror of https://github.com/n8n-io/n8n.git synced 2025-03-05 20:50:17 -08:00
Code Issues Actions Packages Projects Releases Wiki Activity

fix(core): Fix lint on SAML controller (no-changelog) (#5967)

Browse source 
* 👕 Fix lint on SAML controller

* 📘 Type request properly

* Update requests.ts

* 🚚 Move type to module

Assuming ACS falls under configuration.

---------

Co-authored-by: कारतोफ्फेलस्क्रिप्ट™ <netroy@users.noreply.github.com>
This commit is contained in:
Iván Ovejero 2023-04-13 14:14:55 +02:00 committed by GitHub
parent f8f8374506
commit 1a8a9f8ddb
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
2 changed files with 18 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

10
packages/cli/src/sso/saml/routes/saml.controller.ee.ts
View file

@ -83,7 +83,7 @@ export class SamlController {
* Assertion Consumer Service endpoint * Assertion Consumer Service endpoint
*/ */
@Get(SamlUrls.acs, { middlewares: [samlLicensedMiddleware] }) @Get(SamlUrls.acs, { middlewares: [samlLicensedMiddleware] })
async acsGet(req: express.Request, res: express.Response) { async acsGet(req: SamlConfiguration.AcsRequest, res: express.Response) {
return this.acsHandler(req, res, 'redirect'); return this.acsHandler(req, res, 'redirect');
} }
@ -92,7 +92,7 @@ export class SamlController {
* Assertion Consumer Service endpoint * Assertion Consumer Service endpoint
*/ */
@Post(SamlUrls.acs, { middlewares: [samlLicensedMiddleware] }) @Post(SamlUrls.acs, { middlewares: [samlLicensedMiddleware] })
async acsPost(req: express.Request, res: express.Response) { async acsPost(req: SamlConfiguration.AcsRequest, res: express.Response) {
return this.acsHandler(req, res, 'post'); return this.acsHandler(req, res, 'post');
} }
@ -101,7 +101,11 @@ export class SamlController {
* Available if SAML is licensed, even if not enabled to run connection tests * Available if SAML is licensed, even if not enabled to run connection tests
* For test connections, returns status 202 if SAML is not enabled * For test connections, returns status 202 if SAML is not enabled
*/ */
private async acsHandler(req: express.Request, res: express.Response, binding: SamlLoginBinding) { private async acsHandler(
req: SamlConfiguration.AcsRequest,
res: express.Response,
binding: SamlLoginBinding,
) {
const loginResult = await this.samlService.handleSamlLogin(req, binding); const loginResult = await this.samlService.handleSamlLogin(req, binding);
if (loginResult) { if (loginResult) {
// return attributes if this is a test connection // return attributes if this is a test connection

12
packages/cli/src/sso/saml/types/requests.ts
View file

@ -1,7 +1,17 @@
import type { AuthenticatedRequest } from '@/requests'; import type { AuthenticatedRequest, AuthlessRequest } from '@/requests';
import type { SamlPreferences } from './samlPreferences'; import type { SamlPreferences } from './samlPreferences';
export declare namespace SamlConfiguration { export declare namespace SamlConfiguration {
type Update = AuthenticatedRequest<{}, {}, SamlPreferences, {}>; type Update = AuthenticatedRequest<{}, {}, SamlPreferences, {}>;
type Toggle = AuthenticatedRequest<{}, {}, { loginEnabled: boolean }, {}>; type Toggle = AuthenticatedRequest<{}, {}, { loginEnabled: boolean }, {}>;
type AcsRequest = AuthlessRequest<
{},
{},
{
// eslint-disable-next-line @typescript-eslint/naming-convention
RelayState?: string;
},
{}
>;
} }