Skip to content

Move the session user loader to module level - #710

Open
harminius wants to merge 1 commit into
masterfrom
bye-bye-tooljet
Open

harminius wants to merge 1 commit into
masterfrom
bye-bye-tooljet

Conversation

@harminius

Copy link
Copy Markdown
Contributor

Moves the Flask-Login session loader out of create_app to mergin.auth.load_user, so other apps built on CE can register the same loader, auth_version session revocation included. No behaviour change.

@coveralls

coveralls commented Nov 4, 2026 •

Copy link
Copy Markdown

Coverage Report for CI Build 41989702172

Coverage remained the same at 93.061%

Details

  • Coverage remained the same as the base build.
  • Patch coverage: 9 of 9 lines across 3 files are fully covered (100%).
  • 1 coverage regression across 1 file.

Uncovered Changes

No uncovered changes found.

Coverage Regressions

1 previously-covered line in 1 file lost coverage.

File Lines Losing Coverage Coverage
server/mergin/tests/test_middleware.py 1 79.55%

Coverage Stats

Coverage Status
Relevant Lines: 11630
Covered Lines: 10823
Line Coverage: 93.06%
Coverage Strength: 0.93 hits per line

💛 - Coveralls

@harminius
harminius changed the base branch from develop to master November 5, 2026 07:47

This branch has not been deployed

No deployments
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.

2 participants