Skip to content

Commit b1cffbc

Browse files
committed
fix(rabbitmq): using __routeArguments__ to allow pipe injection
1 parent c927cb1 commit b1cffbc

File tree

3 files changed

+6
-10
lines changed

3 files changed

+6
-10
lines changed
Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,5 @@
11
export const RABBIT_HANDLER = Symbol('RABBIT_HANDLER');
22
export const RABBIT_CONFIG_TOKEN = Symbol('RABBIT_CONFIG');
3-
export const RABBIT_ARGS_METADATA = 'RABBIT_ARGS_METADATA';
43
export const RABBIT_PARAM_TYPE = 3;
54
export const RABBIT_HEADER_TYPE = 4;
65
export const RABBIT_REQUEST_TYPE = 5;

packages/rabbitmq/src/rabbitmq.decorators.ts

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -7,9 +7,9 @@ import {
77
Type,
88
assignMetadata,
99
} from '@nestjs/common';
10+
import { ROUTE_ARGS_METADATA } from '@nestjs/common/constants';
1011
import { isString } from 'lodash';
1112
import {
12-
RABBIT_ARGS_METADATA,
1313
RABBIT_CONFIG_TOKEN,
1414
RABBIT_HANDLER,
1515
RABBIT_HEADER_TYPE,
@@ -47,14 +47,14 @@ export const createPipesRpcParamDecorator =
4747
): ParameterDecorator =>
4848
(target, key, index) => {
4949
const args =
50-
Reflect.getMetadata(RABBIT_ARGS_METADATA, target.constructor, key) || {};
50+
Reflect.getMetadata(ROUTE_ARGS_METADATA, target.constructor, key) || {};
5151

5252
const hasParamData = isString(data);
5353
const paramData = hasParamData ? data : undefined;
5454
const paramPipes = hasParamData ? pipes : [data, ...pipes];
5555

5656
Reflect.defineMetadata(
57-
RABBIT_ARGS_METADATA,
57+
ROUTE_ARGS_METADATA,
5858
assignMetadata(args, type, index, paramData, ...paramPipes),
5959
target.constructor,
6060
key

packages/rabbitmq/src/rabbitmq.module.ts

Lines changed: 3 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -10,15 +10,12 @@ import {
1010
OnApplicationBootstrap,
1111
OnApplicationShutdown,
1212
} from '@nestjs/common';
13+
import { ROUTE_ARGS_METADATA } from '@nestjs/common/constants';
1314
import { ExternalContextCreator } from '@nestjs/core/helpers/external-context-creator';
1415
import { groupBy } from 'lodash';
1516
import { AmqpConnection } from './amqp/connection';
1617
import { AmqpConnectionManager } from './amqp/connectionManager';
17-
import {
18-
RABBIT_ARGS_METADATA,
19-
RABBIT_CONFIG_TOKEN,
20-
RABBIT_HANDLER,
21-
} from './rabbitmq.constants';
18+
import { RABBIT_CONFIG_TOKEN, RABBIT_HANDLER } from './rabbitmq.constants';
2219
import { RabbitRpcParamsFactory } from './rabbitmq.factory';
2320
import { RabbitHandlerConfig, RabbitMQConfig } from './rabbitmq.interfaces';
2421

@@ -190,7 +187,7 @@ export class RabbitMQModule
190187
discoveredMethod.parentClass.instance,
191188
discoveredMethod.handler,
192189
discoveredMethod.methodName,
193-
RABBIT_ARGS_METADATA,
190+
ROUTE_ARGS_METADATA,
194191
this.rpcParamsFactory,
195192
undefined, // contextId
196193
undefined, // inquirerId

0 commit comments

Comments
 (0)