Skip to content

fix: rename TOKEN_SECRET to JWT_SECRET across backend, small fixes#2111

Merged
rteas merged 4 commits intohackforla:developmentfrom
rteas:refactor-permission-management
Mar 31, 2026
Merged

fix: rename TOKEN_SECRET to JWT_SECRET across backend, small fixes#2111
rteas merged 4 commits intohackforla:developmentfrom
rteas:refactor-permission-management

Conversation

@rteas
Copy link
Copy Markdown
Member

@rteas rteas commented Mar 31, 2026

Fixes #2090

What changes did you make and why did you make them ?

  • Fixed issue where code was depending on SECRET instead of JWT_SECRET
  • async data call in user controller
  • add support for new method call in projects route

Screenshots of Proposed Changes Of The Website (if any, please do not screen shot code changes)

Visuals before changes are applied

image

Visuals after changes are applied

image

@rteas rteas requested review from kkchu791 and trillium March 31, 2026 01:18
// router.use(Auth.authUser, Auth.requireMinimumRole(ROLES.PROJECT_MANAGER));
// The base is /api/projects

import { AuthUtil } from '../middleware/auth.middleware';
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I just saw an error. Cannot find module....
Can you change this to this?

const AuthUtil = require('../middleware/auth.middleware');

It will work.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Corrected, thanks

@rteas rteas merged commit 8143a0a into hackforla:development Mar 31, 2026
2 of 5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants