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

Centralize and Secure Server Configuration Using Environment Variables #1087

Open
saifrk opened this issue Nov 13, 2024 · 0 comments · May be fixed by #1091
Open

Centralize and Secure Server Configuration Using Environment Variables #1087

saifrk opened this issue Nov 13, 2024 · 0 comments · May be fixed by #1091
Assignees

Comments

@saifrk
Copy link
Collaborator

saifrk commented Nov 13, 2024

Description

This task aims to refactor the server configurations by moving them into a separate configuration file and fetching base_url from environment variables. Currently, the base_url for different servers (dev, test, production, LRM, XLI, etc.) is hardcoded within the server configurations, posing a security risk and lacking flexibility. The proposed change will improve security, maintainability, and flexibility by fetching the base_url from environment variables.

Implementation Considerations

  • Extract all server configuration details into a standalone configuration file (server_configs.py) in the Django project.
    -Define environment variable for the base_urls and fetch base_url from these environment variables rather than relying on hardcoded values

Deliverable

-A server_configs.py that centralizes all server configurations and imports them as needed across the project
-Refactor the existing codebase to use the centralized server configurations, ensuring that base_url is fetched from environment variables

Dependencies

No response

@saifrk saifrk self-assigned this Nov 13, 2024
saifrk pushed a commit that referenced this issue Nov 15, 2024
saifrk pushed a commit that referenced this issue Nov 15, 2024
saifrk pushed a commit that referenced this issue Nov 15, 2024
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 a pull request may close this issue.

1 participant