[GH-ISSUE #48] Create a usable production build #19

Closed
opened 2026-09-04 00:21:37 +02:00 by Stone_Red · 4 comments
Owner

Originally created by @stephan418 on GitHub (May 15, 2022).
Original GitHub issue: https://github.com/detleph/server/issues/48

Originally assigned to: @Stefan-5422 on GitHub.

And merge it into the main branch for deployment

Please make sure that:

  • Everything is working as expected (Docker, FS and so on)
  • Logging code is turned to production mode (With the NODE_ENV variable)
  • [PR #94] [MERGED] Feature/default admin (#97)
  • The default admin is removed from app.ts (!!)
  • The minimum required features are working (Tho CRUD should be working for all of them except if something went terribly wrong)
  • [GH-ISSUE #97] Check the server for unsafe code (#49)
  • There is no confidential information in the build (As the whole code could be made public)
  • Make sure that everything but reads is currently only available to admins
  • Review the authorization code and the media code again (As those are likely the most vulnerable)
  • Add CORS

(The prod server should be able to be deployed with this release tomorrow)

Minium feature requirement for the release (C for Create, R for Read, U for Update, D for Delete):

  • Auth (C)
  • Events (R)
  • Disciplines (R)
  • RoleSchemas (R)
  • Organisation (R)
  • Group (R)
Originally created by @stephan418 on GitHub (May 15, 2022). Original GitHub issue: https://github.com/detleph/server/issues/48 Originally assigned to: @Stefan-5422 on GitHub. And merge it into the main branch for deployment Please make sure that: - [x] Everything is working as expected (Docker, FS and so on) - [x] Logging code is turned to production mode (With the NODE_ENV variable) - [ ] #97 - [x] The default admin is removed from app.ts (!!) - [x] The minimum required features are working (Tho CRUD should be working for all of them except if something went terribly wrong) - [x] #49 - [x] There is no confidential information in the build (As the whole code could be made public) - [x] Make sure that everything but reads is currently only available to admins - [x] Review the authorization code and the media code again (As those are likely the most vulnerable) - [x] Add CORS (The prod server should be able to be deployed with this release tomorrow) Minium feature requirement for the release (C for Create, R for Read, U for Update, D for Delete): - Auth (C) - Events (R) - Disciplines (R) - RoleSchemas (R) - Organisation (R) - Group (R)
Author
Owner

@stephan418 commented on GitHub (May 16, 2022):

Media is now working

<!-- gh-comment-id:1127621322 --> @stephan418 commented on GitHub (May 16, 2022): Media is now working
Author
Owner

@Stefan-5422 commented on GitHub (May 17, 2022):

Everything is checked for unsafe code

I looked over most of the code we are using and have not found any obvious security issues. Maybe I'm blind tho

<!-- gh-comment-id:1129258823 --> @Stefan-5422 commented on GitHub (May 17, 2022): > Everything is checked for unsafe code I looked over most of the code we are using and have not found any obvious security issues. Maybe I'm blind tho
Author
Owner

@Stefan-5422 commented on GitHub (May 17, 2022):

Logging code is turned to production mode (With the NODE_ENV variable)

This will need to be done on the server and shall be remembered

<!-- gh-comment-id:1129261057 --> @Stefan-5422 commented on GitHub (May 17, 2022): > Logging code is turned to production mode (With the NODE_ENV variable) This will need to be done on the server and shall be remembered
Author
Owner

@stephan418 commented on GitHub (Dec 8, 2022):

Everything resolved or moved;

<!-- gh-comment-id:1342638384 --> @stephan418 commented on GitHub (Dec 8, 2022): Everything resolved or moved;
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: detleph/server#19