Merge pull request #40 from notjuliet/main

update db with updateHandle

src/millipds/service.py

@@ -379,9 +379,13 @@ async def identity_update_handle(request: web.Request):
 	handle = req_json.get("handle")
 	if handle is None:
 		raise web.HTTPBadRequest(text="missing or invalid handle")
-	# TODO: actually validate it, and update the db!!!
+	# TODO: actually validate it
 	# (I'm writing this half-baked version just so I can send firehose #identity events)
 	with get_db(request).new_con() as con:
+		con.execute(
+			"UPDATE user SET handle = ? WHERE did = ?",
+			(handle, request["authed_did"]),
+		)
 		# TODO: refactor to avoid duplicated logic between here and apply_writes
 		firehose_seq = con.execute(
 			"SELECT IFNULL(MAX(seq), 0) + 1 FROM firehose"

tests/integration_test.py

@@ -493,3 +493,19 @@ async def test_deleteSession(s, pds_host):
 		headers={"Authorization": "Bearer " + refresh_token},
 	) as r:
 		assert r.status != 200
+
+async def test_updateHandle(s, pds_host, auth_headers):
+	async with s.post(
+		pds_host + "/xrpc/com.atproto.identity.updateHandle",
+		headers=auth_headers,
+		json={ "handle": "juliet.test" },
+	) as r:
+		assert r.status == 200
+
+	async with s.get(
+		pds_host + "/xrpc/com.atproto.repo.describeRepo",
+		params={ "repo":  TEST_DID },
+	) as r:
+		assert r.status == 200
+		r = await r.json()
+		assert r["handle"] == "juliet.test"