Merge pull request #13 from jagerman/fix-room-list

jagerman · web-flow · commit 2befd45ec4d7 · 2021-11-26T17:03:55.000-04:00
Fix static room list
diff --git a/sogs/legacy_routes.py b/sogs/legacy_routes.py
@@ -102,16 +102,13 @@ def legacy_check_user_room(
 
 @app.get("/legacy/rooms")
 def get_rooms():
-    """serve room list for user"""
-    pubkey = get_pubkey_from_token(request.headers.get("Authorization"))
-    if not pubkey:
-        abort(http.BAD_REQUEST)
+    """serve public room list for user"""
 
     return jsonify(
         {
             'status_code': 200,
             # Legacy Session only wants token (returned as 'id') and name:
-            'rooms': [{'id': r.token, 'name': r.name} for r in model.get_readable_rooms(pubkey)],
+            'rooms': [{'id': r.token, 'name': r.name} for r in model.get_readable_rooms()],
         }
     )
 

Original file line number	Diff line number	Diff line change
`@@ -102,16 +102,13 @@ def legacy_check_user_room(`
`102`	`102`
`103`	`103`	`@app.get("/legacy/rooms")`
`104`	`104`	`def get_rooms():`
`105`		`- """serve room list for user"""`
`106`		`- pubkey = get_pubkey_from_token(request.headers.get("Authorization"))`
`107`		`- if not pubkey:`
`108`		`- abort(http.BAD_REQUEST)`
	`105`	`+ """serve public room list for user"""`
`109`	`106`
`110`	`107`	`return jsonify(`
`111`	`108`	`{`
`112`	`109`	`'status_code': 200,`
`113`	`110`	`# Legacy Session only wants token (returned as 'id') and name:`
`114`		`- 'rooms': [{'id': r.token, 'name': r.name} for r in model.get_readable_rooms(pubkey)],`
	`111`	`+ 'rooms': [{'id': r.token, 'name': r.name} for r in model.get_readable_rooms()],`
`115`	`112`	`}`
`116`	`113`	`)`
`117`	`114`