Produits Entr'ouvert » Combo

un début de patch avant que je rentre chez moi, pour percevoir l'idée (et me crier dessus si elle est trop mauvaise)

diff --git a/combo/utils.py b/combo/utils.py
index 8f22ece..700d059 100644
--- a/combo/utils.py
+++ b/combo/utils.py
@@ -87,6 +87,21 @@ class Requests(RequestsSession):
         raise_if_not_cached = kwargs.pop('raise_if_not_cached', False)
         log_errors = kwargs.pop('log_errors', True)

+        if remote_service == 'auto':
+            scheme, netloc, old_path, params, old_query, fragment = urlparse.urlparse(url)
+            for services in settings.KNOWN_SERVICES.values():
+                for service in services.values():
+                    remote_url = service.get('url')
+                    remote_netloc = urlparse.urlparse(remote_url).netloc.split(':')[0]
+                    if remote_netloc == netloc:
+                        remote_service = service
+                        break
+                else:
+                    continue
+                break
+            if remote_service:
+                url = urlparse.urlunparse(('', '', path, params, query, fragment))
+
         if remote_service:
             if isinstance(user, dict):
                 query_params = user.copy()

note: on pourrait décider de logger+planter si aucun service n'est trouvé

Oui je pense qu'on peut décider de logger une erreur, bien décrite.

Tu sépares bien le port du netloc venant de KNOWN_SERVICES mais pas celui venant de l'URL passée à requests, il faut le faire dans les deux cas ou ne pas le faire, mais il faut choisir, je pencherai pour ne pas le faire et j'irai jusqu'à comparer le scheme aussi, port != ou scheme != pour moi ce n'est pas le même service.

Sinon pour le reste ça me va. Mais en fait je ne comprends pas trop le code de la classe Requests ça fait des trucs bizarre au lieu de simplement ajouter orig et la signature à l'URL ça va mélanger service['url'] de dedans, je suppose qu'il y a une raison mais ce serait bien de la mettre dans le code à un jour.

        if remote_service:
            if isinstance(user, dict):
                query_params = user.copy()
            elif not user or not user.is_authenticated():
                if without_user:
                    query_params = {}
                else:
                    query_params = {'NameID': '', 'email': ''}
            else:
                query_params = {}
                if federation_key == 'nameid':
                    query_params['NameID'] = user.saml_identifiers.first().name_id
                elif federation_key == 'email':
                    query_params['email'] = user.email
                else: # 'auto'
                    if hasattr(user, 'saml_identifiers') and user.saml_identifiers.exists():
                        query_params['NameID'] = user.saml_identifiers.first().name_id
                    else:
                        query_params['email'] = user.email

            query_params['orig'] = remote_service.get('orig')

            remote_service_base_url = remote_service.get('url')
            scheme, netloc, old_path, params, old_query, fragment = urlparse.urlparse(
                    remote_service_base_url)

            query = urlencode(query_params)
            if '?' in url:
                path, old_query = url.split('?')
                query += '&' + old_query
            else:
                path = url

            url = urlparse.urlunparse((scheme, netloc, path, params, query, fragment))

Sinon pour le reste ça me va. Mais en fait je ne comprends pas trop le code de la classe Requests ça fait des trucs bizarre au lieu de simplement ajouter orig et la signature à l'URL ça va mélanger service['url'] de dedans, je suppose qu'il y a une raison mais ce serait bien de la mettre dans le code à un jour.

Je veux bien essayer d'expliquer des trucs mais je ne vois pas à quoi tu fais référence. (je ne vois pas service['url'])

            remote_service_base_url = remote_service.get('url')
,,,
scheme, netloc, old_path, params, old_query, fragment = urlparse.urlparse(
                    remote_service_base_url)
...
            path = url # je saute le cas où remote_url contient aussi une query string 
...
            url = urlparse.urlunparse((scheme, netloc, path, params, query, fragment))

Pourquoi n'utilise-t-on pas tout simplement url sans sa query string à laquelle on ajouterait orig et la signature ?

Parce qu'url peut en fait simplement être un chemin.

Dans les cellules wcs, le wcs concerné est sélectionné lors de la configuration de la cellule, et les appels aux webservices se font façon .get(wcs_site, '/api/...').

Ok je comprends mieux.

Ordoncques, le patch, il vous va ?

Pour moi c'est ok.