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

Port register, login and update services to the new ECS style #923

Merged
merged 25 commits into from
Jan 22, 2025
Merged
Changes from 1 commit
Commits
Show all changes
25 commits
Select commit Hold shift + click to select a range
54ce170
Move VPN routes to its own file
LDiazN Jan 15, 2025
f079cea
Refactor VPN Routes into its own file
LDiazN Jan 15, 2025
38f3b8a
Add probe_login_post endpoint copied and pasted from the probe_servic…
LDiazN Jan 15, 2025
481f9af
Removed useless example model
LDiazN Jan 15, 2025
61ae79e
Add register service
LDiazN Jan 15, 2025
769a0e7
Added informational comment; Formatting endpoint docs string
LDiazN Jan 16, 2025
d1a8137
Fix broken tests
LDiazN Jan 16, 2025
f3785be
Changed the path to match the same path as the old version
LDiazN Jan 16, 2025
c52c208
Fixing wrong schema in ProbeRegister model for /register
LDiazN Jan 16, 2025
be3fb93
Porting probe_services tests related to probe login
LDiazN Jan 16, 2025
6cd9426
Fixing broken test; handling case with empty username and password
LDiazN Jan 17, 2025
c69b37d
Removing unnecessary import
LDiazN Jan 17, 2025
ddc93da
Set username and password optional to deliver a 401 error when they'r…
LDiazN Jan 17, 2025
eb4cf8d
Add simple test for fake /update endpoint
LDiazN Jan 17, 2025
dc48d33
Fix spacing
LDiazN Jan 17, 2025
c5a05ea
Adding logging to update function
LDiazN Jan 17, 2025
b493e1f
Add client_id path argument to update to make it consistent with the …
LDiazN Jan 17, 2025
884cb70
Add usage of actual token into the test
LDiazN Jan 17, 2025
a76a17c
Add login token to update testing
LDiazN Jan 17, 2025
21a166c
Remove useless global log statement
LDiazN Jan 22, 2025
dcf48d5
Black reformat
LDiazN Jan 22, 2025
cc0342d
Add comment with warning about encryption key
LDiazN Jan 22, 2025
c164783
improve comment writing
LDiazN Jan 22, 2025
3679d4e
Add warning about the encryption key to ooniauth
LDiazN Jan 22, 2025
b7bb39f
Add warning about the encryption key to ooniauth v1
LDiazN Jan 22, 2025
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Removing unnecessary import
LDiazN committed Jan 17, 2025
commit c69b37db1ae339b61b69e48593d13a96eef887b2
3 changes: 1 addition & 2 deletions ooniapi/services/ooniprobe/tests/test_probe_auth.py
Original file line number Diff line number Diff line change
@@ -1,5 +1,4 @@
from ooniprobe.common import auth
from httpx import Response


def test_register(client):
@@ -35,7 +34,7 @@ def test_register_then_login(client, jwt_encryption_key):
assert resp.status_code == 401

def postj(client, url, **kw):
response : Response = client.post(url, json=kw)
response = client.post(url, json=kw)
assert response.status_code == 200, f"Error: {response.content}"
assert response.headers.get('content-type') == 'application/json'
return response.json()