Skip to content

Commit 422ded2

Browse files
authored
Merge pull request #229 from lovesmilesmall-hue/issue-225-fix-request-workaround
Replace temporary Request workaround with proper @Req() decorator
2 parents bc539a0 + a23baf9 commit 422ded2

1 file changed

Lines changed: 6 additions & 3 deletions

File tree

backend/src/puzzle-review/puzzle-review/controllers/puzzle-review.controller.ts

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,7 @@ import {
66
Delete,
77
Param,
88
Query,
9+
Req,
910
Logger,
1011
ParseUUIDPipe,
1112
ParseIntPipe,
@@ -78,15 +79,17 @@ export class PuzzleReviewController {
7879
status: 201,
7980
description: 'Review submitted successfully',
8081
})
81-
async createReview(reviewData: CreateReviewDto): Promise<{
82+
async createReview(
83+
@Req() request: Request,
84+
reviewData: CreateReviewDto,
85+
): Promise<{
8286
success: boolean;
8387
message: string;
8488
data: ReviewResponse;
8589
}> {
86-
const request: Request = {} as Request; // Temporary workaround for @Req() decorator issue
8790
this.logger.log(`Creating review for puzzle: ${reviewData.puzzleId}`);
8891

89-
const ipAddress = request.ip || request.connection.remoteAddress;
92+
const ipAddress = request.ip || (request as any).connection?.remoteAddress;
9093

9194
const review = await this.reviewService.createReview(reviewData, ipAddress);
9295

0 commit comments

Comments
 (0)