Skip to content

Fix: Issue #112 - Improve environment configuration and reset token handling#219

Open
Sakshi-shukla01 wants to merge 2 commits intoopenSUSE:mainfrom
Sakshi-shukla01:fix-issue-112
Open

Fix: Issue #112 - Improve environment configuration and reset token handling#219
Sakshi-shukla01 wants to merge 2 commits intoopenSUSE:mainfrom
Sakshi-shukla01:fix-issue-112

Conversation

@Sakshi-shukla01
Copy link
Copy Markdown
Contributor

Description

This pull request addresses Issue #112 by improving the handling of environment variables and reset token behavior.

Changes

  • Updated the .env.example file to provide clearer and more accurate configuration options
  • Ensured that RESET_TOKEN is used from the environment when provided
  • Added fallback logic to generate a readable token only when RESET_TOKEN is not set
  • Improved handling of STATS_FILE_PATH to prevent file path issues across environments
  • Aligned configuration behavior with Docker setup and README documentation

Fixes

  • Prevents RESET_TOKEN from being regenerated unnecessarily
  • Resolves ENOENT errors caused by invalid or non-existent stats file paths
  • Ensures consistent behavior in both local and containerized environments

Testing

  • Verified application runs correctly using npm on a custom port
  • Confirmed stats file is created and updated correctly
  • Tested reset functionality using a manually provided RESET_TOKEN
  • Ensured no runtime errors occur due to missing files or misconfiguration

Notes

  • No breaking changes introduced
  • Fully backward compatible

Closes #112

@netlify
Copy link
Copy Markdown

netlify bot commented Apr 17, 2026

👷 Deploy request for resonant-starlight-01802d pending review.

Visit the deploys page to approve it

Name Link
🔨 Latest commit dd6d8f0

@netlify
Copy link
Copy Markdown

netlify bot commented Apr 17, 2026

Deploy Preview for quiz-o-o ready!

Name Link
🔨 Latest commit dd6d8f0
🔍 Latest deploy log https://app.netlify.com/projects/quiz-o-o/deploys/69e266bc0939ac00089c2e9b
😎 Deploy Preview https://deploy-preview-219--quiz-o-o.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@lkocman
Copy link
Copy Markdown
Collaborator

lkocman commented Apr 20, 2026

Hello, I'll review and merge this after this week's SUSECON/ODS. I don't want to risk anything this week :-) Thanks @Sakshi-shukla01

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.

Generate random default reset token or prevent start

2 participants