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

Restructure application routing #1141

Open
wants to merge 8 commits into
base: develop
Choose a base branch
from
Open

Conversation

michplunkett
Copy link
Collaborator

@michplunkett michplunkett commented Dec 20, 2024

Fixes issue

Sets the groundwork for this issue: #385

Description of Changes

Simplifying the import logic for the auth and main routes of the application. This also sets up the work for this PR: #1138

Tests and Linting

  • This branch is up-to-date with the develop branch.
  • pytest passes on my local development environment.
  • pre-commit passes on my local development environment.

@michplunkett michplunkett self-assigned this Dec 20, 2024
from OpenOversight.app.utils.forms import set_dynamic_default
from OpenOversight.app.utils.general import validate_redirect_url


auth = Blueprint("auth", __name__, url_prefix="/auth")
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I feel like the url_prefix belongs with the creation of the Blueprint, but I'd be happy to shift it back to the application's __init__.py file if everyone else would prefer that.

Comment on lines -127 to -128
# Ensure the file is read/write by the creator only
SAVED_UMASK = os.umask(0o077)
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This isn't used anywhere in views.py.

@michplunkett michplunkett marked this pull request as ready for review December 20, 2024 23:40
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.

1 participant