CSI-KJSCE/Travel-policy-
2
1
Fork 5
Code Issues 2 Pull Requests 2 Actions Packages Projects Releases Wiki Activity

Implemented Oauth #2

Open
Soundcreates wants to merge 1 commits from Soundcreates/Travel-policy-:GOAuth into main
pull from: Soundcreates/Travel-policy-:GOAuth
merge into: CSI-KJSCE:main
Conversation 1 Commits 1 Files Changed 22 +960 -257
Soundcreates commented 2026-01-02 00:53:15 +05:30
First-time contributor
Copy Link

Issue

Google OAuth authentication was not implemented in the application.

Changes Made

  • Implemented Google authentication in the backend using Passport.js by adding a dedicated passport_service.js.
  • Added OAuth initialization and callback controllers in authController.js.
  • Integrated Google OAuth with the frontend authentication flow.
  • Ensured user roles are assigned correctly during OAuth login:
  • Users logging in via the applicant login page are assigned the applicant designation.
  • All other OAuth logins are assigned the faculty designation.

Testing

  • Verified the complete OAuth flow end-to-end.

  • Confirmed correct role assignment based on the login entry point.

  • Attached a video demonstrating the successful authentication flow.

Please let me know if there are any issues, improvements, or edge cases that need to be addressed.
Thank you!

**Issue** Google OAuth authentication was not implemented in the application. **Changes Made** - Implemented Google authentication in the backend using Passport.js by adding a dedicated passport_service.js. - - Added OAuth initialization and callback controllers in authController.js. - - Integrated Google OAuth with the frontend authentication flow. - - Ensured user roles are assigned correctly during OAuth login: - - Users logging in via the applicant login page are assigned the applicant designation. - - All other OAuth logins are assigned the faculty designation. **Testing** - Verified the complete OAuth flow end-to-end. - Confirmed correct role assignment based on the login entry point. - Attached a video demonstrating the successful authentication flow. Please let me know if there are any issues, improvements, or edge cases that need to be addressed. Thank you!
Vite + React — Zen Browser 2026-01-02 00-45-19.mp4
8.3 MiB
Soundcreates added 1 commit 2026-01-02 00:53:15 +05:30
Implemented Oauth 0130f746b0
Soundcreates commented 2026-01-02 01:32:53 +05:30
Author
First-time contributor
Copy Link

Fixes Issue #1

Fixes Issue #1
This pull request can be merged automatically.
You are not authorized to merge this pull request.
View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u GOAuth:Soundcreates-GOAuth
git checkout Soundcreates-GOAuth
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
Clear labels
bug
Something isn't working
 documentation
Improvements or additions to documentation
 duplicate
This issue or pull request already exists
 enhancement
New feature or request
 good first issue
Good for newcomers
 help wanted
Extra attention is needed
 invalid
This doesn't seem right
 question
Further information is requested
 wontfix
This will not be worked on
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
CSI-KJSCE
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: CSI-KJSCE/Travel-policy-#2
Delete Branch "Soundcreates/Travel-policy-:GOAuth"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?