Skip to content

Add JWT auth module#3

Merged
Relin404 merged 13 commits into
mainfrom
side-17-create-jwt-authentication
Aug 7, 2025
Merged

Add JWT auth module#3
Relin404 merged 13 commits into
mainfrom
side-17-create-jwt-authentication

Conversation

@Relin404
Copy link
Copy Markdown
Owner

@Relin404 Relin404 commented Aug 7, 2025

No description provided.

@linear
Copy link
Copy Markdown

linear Bot commented Aug 7, 2025

@Relin404 Relin404 self-assigned this Aug 7, 2025
@Relin404 Relin404 added the enhancement New feature or request label Aug 7, 2025
@Relin404 Relin404 requested a review from Copilot August 7, 2025 12:08
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR adds JWT authentication functionality to the application, including login capability, token generation, and cookie-based authentication. The changes introduce a complete authentication module with JWT token handling and update configuration to use standardized environment variables.

  • Added JWT authentication module with login mutation and token generation
  • Created GraphQL context interface for request/response handling
  • Standardized environment variable names (PORT, DATABASE_URL)

Reviewed Changes

Copilot reviewed 13 out of 14 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
package.json Added JWT and Express dependencies
libs/nestjs/src/lib/graphql/gql-context.interface.ts Created GraphQL context interface for Express req/res
apps/trigger-auth/src/app/auth/* New authentication module with service, resolver, and DTOs
apps/trigger-auth/src/app/users/* Extended users service and exported it for auth module
apps/trigger-auth/src/app/app.module.ts Integrated auth module and GraphQL context
apps/trigger-auth/src/main.ts Updated to use standardized PORT environment variable
apps/trigger-auth/prisma/schema.prisma Updated to use standardized DATABASE_URL environment variable
Files not reviewed (1)
  • pnpm-lock.yaml: Language not supported

Comment thread apps/trigger-auth/src/app/auth/auth.module.ts Outdated
Comment thread apps/trigger-auth/src/app/auth/auth.service.ts Outdated
@Relin404 Relin404 merged commit 8707b9a into main Aug 7, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants