Fix challenge preview for admins (#978)

* Allow admins to hit `/api/v1/challenges` and `/api/v1/challenges/[id]` without having a team
* Fixes regression from 2.0.6

CTFd/api/v1/challenges.py

@@ -65,6 +65,9 @@ class ChallengeList(Resource):
             solve_ids = set([value for value, in solve_ids])
 
             # TODO: Convert this into a re-useable decorator
+            if is_admin():
+                pass
+            else:
                 if config.is_teams_mode() and get_current_team() is None:
                     abort(403)
         else:
@@ -211,6 +214,9 @@ class Challenge(Resource):
             team = get_current_team()
 
             # TODO: Convert this into a re-useable decorator
+            if is_admin():
+                pass
+            else:
                 if config.is_teams_mode() and team is None:
                     abort(403)
 

tests/api/v1/test_challenges.py

@@ -99,6 +99,22 @@ def test_api_challenges_post_non_admin():
     destroy_ctfd(app)
 
 
+def test_api_challenges_get_admin():
+    """Can a user GET /api/v1/challenges if admin without team"""
+    app = create_ctfd(user_mode="teams")
+    with app.app_context():
+        gen_challenge(app.db)
+        # Admin does not have a team but should still be able to see challenges
+        user = Users.query.filter_by(id=1).first()
+        assert user.team_id is None
+        with login_as_user(app, 'admin') as admin:
+            r = admin.get('/api/v1/challenges', json="")
+            assert r.status_code == 200
+            r = admin.get('/api/v1/challenges/1', json="")
+            assert r.status_code == 200
+    destroy_ctfd(app)
+
+
 def test_api_challenges_post_admin():
     """Can a user post /api/v1/challenges if admin"""
     app = create_ctfd()