Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Bugfix/error reading password env 35 #36

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

sikatikenmogne
Copy link

@sikatikenmogne sikatikenmogne commented Nov 9, 2024

Description

There was an error in the file apps/pages/results/results.py when attempting to read from the password.env file. The specific line of code causing the issue was:

with open(os.getcwd() + "\\apps\\pages\\results\\password.env") as f:
    password = f.read().split('=')[1]

This fix resolves the hardcoded file path and potential issues with file accessibility or format.

Changes

  1. Fixed the error in apps/pages/results/results.py by updating the file path handling for password.env.

Desktop

OS: Windows
Version: 11

Related Issues

Closes #35

Additional context

Consider adding a .gitignore file to exclude the ./scripts/results/ folder from being tracked. Additionally, update the documentation to instruct developers to add this folder to their local .gitignore to prevent accidental tracking of output files.

…ts/results.py

- Fixed the issue where the application was unable to read from the `password.env` file.
- Updated the file reading logic in `apps/pages/results/results.py` to handle file paths correctly.
@sikatikenmogne sikatikenmogne force-pushed the bugfix/error-reading-password-env-35 branch from fcf8581 to 380cdd5 Compare November 9, 2024 14:00
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.

Bug Report: Error reading from password.env file in apps/pages/results/results.py
1 participant