Skip to content

Auth system (JWT)#75

Open
darkgl06 wants to merge 5 commits into
masterfrom
feature/auth_system
Open

Auth system (JWT)#75
darkgl06 wants to merge 5 commits into
masterfrom
feature/auth_system

Conversation

@darkgl06

Copy link
Copy Markdown
Collaborator

No description provided.

@github-actions

github-actions Bot commented Jul 25, 2025

Copy link
Copy Markdown

Coverage

Coverage Report
FileStmtsMissCoverMissing
queryzen
   __init__.py33876%60, 64, 69–85
   backend.py89199%49
   queryzen.py1961294%60–61, 75–77, 83–88, 351, 554, 560
   types.py33391%62–63, 76
TOTAL4272494% 

Tests Skipped Failures Errors Time
92 0 💤 0 ❌ 0 🔥 28.987s ⏱️

@darkgl06 darkgl06 requested a review from surister July 25, 2025 16:05
@darkgl06

Copy link
Copy Markdown
Collaborator Author

Test performance drops a bit after adding test user creation on queryzen-api/apps/testing/views.py

@surister surister left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

I'll check this again later

import httpx


class HttpxWrapper:

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

I'm unsure if wee need a wrapper over httpx just to add a header token

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

I think it'll be useful for future features such as who run the query (python-client, through http directly...)

@surister

Copy link
Copy Markdown
Owner

Test performance drops a bit after adding test user creation on queryzen-api/apps/testing/views.py

any idea why?

@darkgl06

Copy link
Copy Markdown
Collaborator Author

Test performance drops a bit after adding test user creation on queryzen-api/apps/testing/views.py

any idea why?

Nop, I suppose django needs to setup everytime this is called:

# Create a new testing user for auth
User.objects.get_or_create(email="test@test.com", password=make_password('test'))

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