Produits Entr'ouvert » Passerelle

En django 2.1, ils ajoutent une auto permission view https://docs.djangoproject.com/en/3.1/releases/2.1/#model-view-permission
Or on a déjà une permission view_accessright sur le model AccessRight.

Si on la conserve sans la renommer, en 2.2 un user avec cette permissions pourrait accéder à l'admin
https://docs.djangoproject.com/en/3.1/releases/2.1/#new-default-view-permission-could-allow-unwanted-access-to-admin-views
Est-ce que c'est ok ?

pareil pour la permission view_resourcelog

Sûr on peut renommer mais ça m'irait aussi de virer le /admin/

Il vaut mieux renommer, c'est galère à gérer en fonction de la version de django

0005 : cosmétique et facultatif mais comme ça m'a perturbé au début je le mentionne, j'aurais fait if Permission.objects.filter(content_type=ct, codename='view_accessright').exists() plutôt que if not Permission.objects.filter(content_type=ct, codename='see_accessright').exists() , et même idée sur le deuxième if
0007 : je comprends pas, que se passe-t-il ?
0008 : je passerai aussi un django>=1.11 dans le setup.py qui me semble être un oubli de #36087

0005 : cosmétique et facultatif mais comme ça m'a perturbé au début je le mentionne, j'aurais fait if Permission.objects.filter(content_type=ct, codename='view_accessright').exists()

Je préfère comme j'ai fait: au cas où, très improbable, mais possible, on crée une DB from scratch avec un django 2.1, jusqu'à la migration 0027 de base, alors les 2 permissions existeront au run de la 0028.
Alors que si lors du run de la 0027 see_access_right n'existe pas, alors c'est qu'il faut la créer.
Note: les permissions sont ajoutée en post_migrate, comme les contenttypes, donc si on est en mode DB from scratch, cette migration ne fera rien (ContentType.DoesNotExist)

0007 : je comprends pas, que se passe-t-il ?

dans django 2.2, et probablement avant, ils ont ajouté un log automatique qu'on ne peut pas débrancher dans django/core/handlers/exception.py::response_for_exception:

    else:
        signals.got_request_exception.send(sender=None, request=request)
        response = handle_uncaught_exception(request, get_resolver(get_urlconf()), sys.exc_info())
        log_response(
            '%s: %s', response.reason_phrase, request.path,
            response=response,
            request=request,
            exc_info=sys.exc_info(),
        )

donc quand on lance les tests avec django 2.2, on a plus de logs qu'avant