0001-api-add-basic-auth-support-to-users-xxx-and-users-xx.patch

Lauréline Guérin, 14 septembre 2021 11:11

Voir les différences: en ligne côte à côte

Subject: [PATCH] api: add basic auth support to users/xxx and users/xxx/forms
 (#56908)

 tests/api/test_user.py | 151 ++++++++++++++++++++++++++---------------
 wcs/api.py             |   9 ++-
 2 files changed, 100 insertions(+), 60 deletions(-)

     import datetime
     import os
     from functools import partial
     import pytest
     from quixote import get_publisher
-...
         return user
     def _get_url(url, app, auth, access, user, **kwargs):
         if auth == 'http-basic':
             app.set_authorization(('Basic', ('test', '12345')))
             return app.get(url, **kwargs)
         else:
             return app.get(
                 sign_uri(url, user=user, orig=access.access_identifier, key=access.access_key), **kwargs
+            )
     @pytest.fixture
     def access(pub):
         ApiAccess.wipe()
         access = ApiAccess()
         access.name = 'test'
         access.access_identifier = 'test'
         access.access_key = '12345'
         access.store()
         return access
     def test_roles(pub, local_user):
         pub.role_class.wipe()
         role = pub.role_class(name='Hello World')
-...
         assert resp.json['data'][0]['details'] == 'kouign amann'
     def test_user_api_with_restricted_access(pub, access):
         role = pub.role_class(name='test')
         role.store()
         access.roles = [role]
         access.store()
         resp = get_app(pub).get(sign_uri('/api/user/', orig='test', key='12345'), status=403)
         assert resp.json['err'] == 1
         assert resp.json['err_desc'] == 'restricted API access'
     def test_users_api_with_restricted_access(pub, local_user, access):
         role = pub.role_class(name='test')
         role.store()
         access.roles = [role]
         access.store()
         resp = get_app(pub).get(sign_uri('/api/users/', orig='test', key='12345'), status=403)
         assert resp.json['err'] == 1
         assert resp.json['err_desc'] == 'restricted API access'
     def test_users(pub, local_user):
         resp = get_app(pub).get('/api/users/', status=403)
-...
+        )
     def test_user_by_nameid(pub, local_user):
         resp = get_app(pub).get(sign_uri('/api/users/xyz/', user=local_user), status=404)
     @pytest.mark.parametrize('auth', ['signature', 'http-basic'])
     def test_user_by_nameid(pub, local_user, access, auth):
         app = get_app(pub)
         get_url = partial(_get_url, app=app, auth=auth, access=access, user=local_user)
         resp = get_url('/api/users/xyz/', status=404)
         local_user.name_identifiers = ['xyz']
         local_user.store()
         resp = get_app(pub).get(sign_uri('/api/users/xyz/', user=local_user))
         resp = get_url('/api/users/xyz/')
         assert str(resp.json['id']) == str(local_user.id)
     def test_user_roles(pub, local_user):
         local_user.name_identifiers = ['xyz']
         local_user.store()
     @pytest.mark.parametrize('auth', ['signature', 'http-basic'])
     def test_user_roles(pub, local_user, access, auth):
         role = pub.role_class(name='Foo bar')
         role.store()
         local_user.name_identifiers = ['xyz']
         local_user.roles = [role.id]
         local_user.store()
         resp = get_app(pub).get(sign_uri('/api/users/xyz/', user=local_user))
         app = get_app(pub)
         get_url = partial(_get_url, app=app, auth=auth, access=access, user=local_user)
         resp = get_url('/api/users/xyz/')
         assert len(resp.json['user_roles']) == 1
         assert resp.json['user_roles'][0]['name'] == 'Foo bar'
     def test_user_forms(pub, local_user):
     def test_user_forms(pub, local_user, access):
         Workflow.wipe()
         workflow = Workflow.get_default_workflow()
         workflow.id = '2'
-...
         role = pub.role_class(name='test')
         role.store()
         access = ApiAccess()
         access.name = 'test'
         access.access_identifier = 'test'
         access.access_key = '12345'
         access.roles = [role]
         access.store()
-...
         assert resp.json['err_desc'] == 'restricted API access'
     def test_user_api_with_restricted_access(pub):
         role = pub.role_class(name='test')
         role.store()
         access = ApiAccess()
         access.name = 'test'
         access.access_identifier = 'test'
         access.access_key = '12345'
         access.roles = [role]
         access.store()
         resp = get_app(pub).get(sign_uri('/api/user/', orig='test', key='12345'), status=403)
         assert resp.json['err'] == 1
         assert resp.json['err_desc'] == 'restricted API access'
     def test_users_api_with_restricted_access(pub, local_user):
         role = pub.role_class(name='test')
         role.store()
         access = ApiAccess()
         access.name = 'test'
         access.access_identifier = 'test'
         access.access_key = '12345'
         access.roles = [role]
         access.store()
         resp = get_app(pub).get(sign_uri('/api/users/', orig='test', key='12345'), status=403)
         assert resp.json['err'] == 1
         assert resp.json['err_desc'] == 'restricted API access'
         resp = get_app(pub).get(sign_uri('/api/users/%s/' % local_user.id, orig='test', key='12345'), status=403)
         assert resp.json['err'] == 1
         assert resp.json['err_desc'] == 'restricted API access'
     def test_user_forms_limit_offset(pub, local_user):
         if not pub.is_using_postgresql():
             pytest.skip('this requires SQL')
-...
         assert len(resp.json['data']) == 5
     def test_user_forms_from_agent(pub, local_user):
     @pytest.mark.parametrize('auth', ['signature', 'http-basic'])
     def test_user_forms_from_agent(pub, local_user, access, auth):
         pub.role_class.wipe()
         role = pub.role_class(name='Foo bar')
         role.store()
-...
         formdata.jump_status('new')
         formdata.store()
         resp = get_app(pub).get(sign_uri('/api/users/%s/forms' % local_user.id, user=agent_user))
         if auth == 'http-basic':
             access.roles = [role]
             access.store()
         app = get_app(pub)
         get_url = partial(_get_url, app=app, auth=auth, access=access, user=agent_user)
         resp = get_url('/api/users/%s/forms' % local_user.id)
         assert resp.json['err'] == 0
         assert len(resp.json['data']) == 1
         assert resp.json['data'][0]['form_name'] == 'test'
-...
         formdef.skip_from_360_view = True
         formdef.store()
         resp = get_app(pub).get(sign_uri('/api/users/%s/forms' % local_user.id, user=agent_user))
         resp = get_url('/api/users/%s/forms' % local_user.id)
         assert len(resp.json['data']) == 0
         formdef.workflow_roles = {'_receiver': str(role.id)}
         formdef.store()
         formdef.data_class().rebuild_security()
         resp = get_app(pub).get(sign_uri('/api/users/%s/forms' % local_user.id, user=agent_user))
         resp = get_url('/api/users/%s/forms' % local_user.id)
         assert len(resp.json['data']) == 1
         agent_user.roles = []
         agent_user.store()
         get_app(pub).get(sign_uri('/api/users/%s/forms' % local_user.id, user=agent_user), status=403)
         if auth == 'http-basic':
             access.roles = []
             access.store()
             resp = get_url('/api/users/%s/forms' % local_user.id)
             assert len(resp.json['data']) == 0
         else:
             get_url('/api/users/%s/forms' % local_user.id, status=403)
     def test_user_forms_include_accessible(pub, local_user):
     def test_user_forms_include_accessible(pub, local_user, access):
         if not pub.is_using_postgresql():
             pytest.skip('this requires SQL')
             return
-...
         formdata4.jump_status('new')
         formdata4.store()
         app = get_app(pub)
         def get_ids(url):
             resp = get_app(pub).get(url)
             resp = app.get(url)
             return {int(x['form_number_raw']) for x in resp.json['data']}
         resp = get_ids(sign_uri('/api/user/forms', user=local_user))
-...
         resp = get_ids(sign_uri('/api/users/%s/forms?include-accessible=on' % local_user.id, user=agent_user))
         assert resp == {formdata1.id, formdata3.id}
         # an api access gets the same results
         access.roles = [role]
         access.store()
         app.set_authorization(('Basic', ('test', '12345')))
         resp = get_ids('/api/users/%s/forms' % local_user.id)
         assert resp == {formdata1.id}
         resp = get_ids('/api/users/%s/forms?include-accessible=on' % local_user.id)
         assert resp == {formdata1.id, formdata3.id}
     def test_user_drafts(pub, local_user):
         FormDef.wipe()

                 # (from query string or session).
                 query_user = get_user_from_api_query_string() or get_request().user
                 if query_user and query_user.id != self.user.id:
                     if not query_user.can_go_in_backoffice():
                     if not query_user.is_api_user and not query_user.can_go_in_backoffice():
                         raise AccessForbiddenError('user not allowed to query data from others')
                     # mark forms that are readable by querying user
                     user_roles = set(query_user.get_roles())
-...
             return json.dumps({'data': data, 'err': 0}, cls=misc.JSONEncoder)
         def _q_lookup(self, component):
             if not (is_url_signed() or (get_request().user and get_request().user.can_go_in_admin())):
                 raise AccessForbiddenError('unsigned request or user has no access to backoffice')
             api_user = get_user_from_api_query_string()
             if api_user and api_user.is_api_user:
                 raise AccessForbiddenError('restricted API access')
                 pass  # API users are ok
             elif not (is_url_signed() or (get_request().user and get_request().user.can_go_in_admin())):
                 raise AccessForbiddenError('unsigned request or user has no access to backoffice')
             user_class = get_publisher().user_class
             try:
+    -

Projet

Général

Profil

Produits Entr'ouvert » w.c.s.

0001-api-add-basic-auth-support-to-users-xxx-and-users-xx.patch