freeCodeCamp / chapter

A self-hosted event management tool for nonprofits
BSD 3-Clause "New" or "Revised" License
1.92k stars 360 forks source link

feat: only warn about missing keys #2423

Closed ojeytonwilliams closed 1 year ago

ojeytonwilliams commented 1 year ago

The main thing is we can't rely on oauth2.keys.json existing. Notably it won't exist in railway, so we need an alternative approach.

For now, I just want to downgrade the error to a warning. Longer term we could make use of environment variables, but I'm focusing on getting the MVP deployable for now.

ghost commented 1 year ago
👇 Click on the image for a new way to code review #### [![Review these changes using an interactive CodeSee Map](https://s3.us-east-2.amazonaws.com/maps.codesee.io/images/github/freeCodeCamp/chapter/2423/3c02bf4d/2f8d5a9bfff5306c5dbc560d9e728d601c643c3e.svg)](https://app.codesee.io/r/reviews?pr=2423&src=https%3A%2F%2Fgithub.com%2FfreeCodeCamp%2Fchapter) #### Legend CodeSee Map legend
ojeytonwilliams commented 1 year ago

I added the colours to (hopefully) make it easier to read the logs that concurrently produces. If anyone hates them, feel free to revert that change.