Produits Entr'ouvert » w.c.s.

s/is_idp_manage_user_attributes/is_idp_managing_user_attributes/ (ou does_idp_manage_user_attributes mais j'aime moins).

Même quand c'est le cas, on ne perdrait rien à quand même exécuter le code local.

Et peut-être même alors taper les appels HTTP en afterjob, pour ne pas perdre de temps.

Frédéric Péters a écrit :

Même quand c'est le cas, on ne perdrait rien à quand même exécuter le code local.

On gagnerait beaucoup (c'est instantané, si la présence du rôle est nécessaire aussitôt). En fait, il faut absolument le faire.

A priori tu as posté l'ancien patch. Sur le nouveau, il faut penser à laisser l'action locale sur le Remove aussi. Avec ça, c'est ack pour moi. (je laisse la main à Fred pour insister sur le s/is_idp_manage_user_attributes/is_idp_managing_user_attributes/ le cas échéant)

Dans 0003, c'est resté is_idp_manage_user_attributes.

Fallait pas prendre de congés.

Et donc, Ack pour moi.

Exception:
  type = '<type 'exceptions.AttributeError'>', value = ''module' object has no attribute 'urlquote''

Stack trace (most recent call first):
  File "/home/fred/src/eo/wcs/wcs/wf/roles.py", line 33, in roles_ws_url
    31     entity_id = idps.values()[0]['metadata']
    32     base_url = entity_id.split('idp/saml2/metadata')[0]
>   33     return urlparse.urljoin(base_url, '/api/roles/%s/members/%s/' % (urllib.urlquote(role_uuid),
    34                                                                      urllib.urlquote(user_uuid)))
    35 

Et après ça il yaura le fait que "idps.values()[0]['metadata']" donne le nom du fichier avec les métadata, et pas les métadata.

Peut-être dès lors, pour ne pas creuser trop dans le SAML, simplement avoir un "Role Webservice URL", à l'instar du Registration URL (dans /settings/identification/idp/identities). (?)

Frédéric Péters a écrit :

[...]

C'est corrigé depuis le patch du 14 décembre.

Et après ça il yaura le fait que "idps.values()[0]['metadata']" donne le nom du fichier avec les métadata, et pas les métadata.

Idem.

Je pense que tu as testé un vieux patch, le code actuel ressemble à cela:

    entity_id = idps.values()[0]['metadata_url']
    base_url = entity_id.split('idp/saml2/metadata')[0]
    url = urlparse.urljoin(base_url, '/api/roles/%s/members/%s/' % (urllib.quote(role_uuid),
                                                                     urllib.quote(user_uuid)))

En effet j'avais appliqué par mégarde un vieux patch.

Cela étant :

Exception:
  type = '<type 'exceptions.AttributeError'>', value = ''NoneType' object has no attribute 'get_server''

Stack trace (most recent call first):
  File "/home/fred/src/eo/wcs/wcs/wf/roles.py", line 43, in roles_ws_url
    41                                                                      urllib.quote(user_uuid)))
    42     secret = get_secret(url)
>   43     orig = get_request().get_server().split(':')[0]
    44     url += '?orig=%s' % orig
    45     return sign_url(url, secret)

Et je n'aime guère le entity_id.split('idp/saml2/metadata')[0] (le "Registration URL" avait été créé pour ne pas dépendre des URL d'Authentic 2); mais je laisserai passer.

J'imagine maintenant une dépendance à #8896, non ?

[04/Jan/2016 18:48:06] "POST /api/roles/d100b3f981ef409f8a56d410e8573aeb/members/fred/?orig=auquo&algo=sha256&timestamp=2016-01-04T17%3A48%3A06Z&nonce=3e2603f2cc9e4cd44fe765b722ff438c&signature=6UNsPjsJtOrkKvLm/TyT1kOTcxgCxlyAeVK3UTqgfqU%3D HTTP/1.1" 401 58

Oui, j'avais le secret espoir que #8896 soit intégré bien avant celui-ci :)

Pour le moment ça signe la requête vers authentic avec une clé tirée de [api-secrets]; afin que les choses soient claires, [api-secrets] devrait uniquement être utilisé pour la vérification des requêtes s'adressant à l'API de w.c.s.

Pour ça je propose d'intégrer le patch suivant dans le 0003; et puis derrière il faudra adapter le hobo_deploy pour créer et remplir la section [wscall-secrets].

Manque le patch.

Ça marche pour moi.

(cela étant je préfère qu'on attende que l'authentic avec l'authent rest framework publik se trouve en prod)

Voilà l'authentification REST_FRAMEWORK est en prod:

In [1]: from django.conf import settings

In [2]: settings.REST_FRAMEWORK
Out[2]: 
{'DEFAULT_AUTHENTICATION_CLASSES': ('rest_framework.authentication.BasicAuthentication',
  'rest_framework.authentication.SessionAuthentication',
  'hobo.rest_authentication.PublikAuthentication'),
 'DEFAULT_FILTER_BACKENDS': ('rest_framework.filters.DjangoFilterBackend',
  'rest_framework.filters.OrderingFilter'),
 'DEFAULT_PAGINATION_CLASS': 'rest_framework.pagination.LimitOffsetPagination',
 'DEFAULT_PERMISSION_CLASSES': ('rest_framework.permissions.IsAuthenticated',),
 'NON_FIELD_ERRORS_KEY': '__all__',
 'PAGE_SIZE': 10}

Je pousse ?

Yep, c'est ok pour moi.