Projet

Général

Profil

Development #42123

Affichage de l'information comme quoi un accès 'public' est configuré sur un connecteur

Ajouté par Emmanuel Cazenave il y a presque 4 ans. Mis à jour il y a presque 4 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Version cible:
-
Début:
27 avril 2020
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

Accès public = utilisateur sans clé.

Affichage = un icône dans l'encart du connecteur sur /manage ?


Fichiers

Screenshot_2020-04-27 Passerelle.png (14,1 ko) Screenshot_2020-04-27 Passerelle.png Frédéric Péters, 27 avril 2020 11:48
Screenshot_2020-04-27 Passerelle(1).png (15,5 ko) Screenshot_2020-04-27 Passerelle(1).png Frédéric Péters, 27 avril 2020 11:50
0002-views-flag-connectors-with-open-access-on-homepage-4.patch (3,75 ko) 0002-views-flag-connectors-with-open-access-on-homepage-4.patch Valentin Deniaud, 09 juin 2020 17:17
0001-base-add-confirmation-when-adding-open-access-right-.patch (3,82 ko) 0001-base-add-confirmation-when-adding-open-access-right-.patch Valentin Deniaud, 09 juin 2020 17:17
1591717871.png (105 ko) 1591717871.png Valentin Deniaud, 09 juin 2020 17:51
1592234276.png (3,24 ko) 1592234276.png Valentin Deniaud, 15 juin 2020 17:18
1592234352.png (3,15 ko) 1592234352.png Valentin Deniaud, 15 juin 2020 17:19
Screenshot_2020-06-15 Passerelle.png (27,3 ko) Screenshot_2020-06-15 Passerelle.png Frédéric Péters, 15 juin 2020 17:33
open-over.png (6,65 ko) open-over.png Frédéric Péters, 15 juin 2020 21:06
0001-style.patch (2,42 ko) 0001-style.patch Frédéric Péters, 15 juin 2020 21:07
Screenshot_2020-06-15 Passerelle(1).png (25,9 ko) Screenshot_2020-06-15 Passerelle(1).png Frédéric Péters, 15 juin 2020 21:08
0002-views-flag-connectors-with-open-access-on-homepage-4.patch (4,2 ko) 0002-views-flag-connectors-with-open-access-on-homepage-4.patch Valentin Deniaud, 16 juin 2020 10:39
0001-base-add-confirmation-when-adding-open-access-right-.patch (3,82 ko) 0001-base-add-confirmation-when-adding-open-access-right-.patch Valentin Deniaud, 16 juin 2020 10:39
0002-views-flag-connectors-with-open-access-on-homepage-4.patch (4,2 ko) 0002-views-flag-connectors-with-open-access-on-homepage-4.patch Valentin Deniaud, 16 juin 2020 10:56
0001-base-add-confirmation-when-adding-open-access-right-.patch (3,82 ko) 0001-base-add-confirmation-when-adding-open-access-right-.patch Valentin Deniaud, 16 juin 2020 10:56

Révisions associées

Révision 8ade392b (diff)
Ajouté par Valentin Deniaud il y a presque 4 ans

base: add confirmation when adding open access right (#42123)

Révision 3ba4e0e6 (diff)
Ajouté par Valentin Deniaud il y a presque 4 ans

views: flag connectors with open access on homepage (#42123)

Historique

#1

Mis à jour par Frédéric Péters il y a presque 4 ans

Accès public = utilisateur sans clé.

Ni restriction IP.

Reprendre également l'info dans le panneau "sécurité" (ou même en haut de page?).

Et sur la page d'accueil, encadrer de vert les accès publics ? Ou juste taper ça sous forme textuelle ? Ou autre idée ?

#2

Mis à jour par Emmanuel Cazenave il y a presque 4 ans

J'aime bien l'encadré vert simple et efficace.

En ce qui me concerne je passe toujours par /manage avant d'aller sur la page d'un connecteur donc je dirai que sur cette dernière pas besoin d'attirer a nouveau l'attention.

#3

Mis à jour par Thomas Noël il y a presque 4 ans

Emmanuel Cazenave a écrit :

J'aime bien l'encadré vert simple et efficace.

En ce qui me concerne je passe toujours par /manage avant d'aller sur la page d'un connecteur donc je dirai que sur cette dernière pas besoin d'attirer a nouveau l'attention.

De mon côté je vais rarement sur le /manage donc reprendre l'info me semble utile.

Est-ce qu'on met aussi en avant les endpoint qui ont un accès via apikey (ie dont le contenu est également diffusable en clair par simple clic sur une URL dès qu'elle contient apikey=truc) ?

Sinon, perso je suis pas très chaud pour le cadre vert : ça donne plutôt l'impression qu'un "truc qui marche" alors que c'est presque l'idée inverse qu'on veut donner.

#4

Mis à jour par Serghei Mihai il y a presque 4 ans

Thomas Noël a écrit :

Sinon, perso je suis pas très chaud pour le cadre vert : ça donne plutôt l'impression qu'un "truc qui marche" alors que c'est presque l'idée inverse qu'on veut donner.

Pareil.
Je verrais un triangle avec un point d'éxclamation dans le coin haut à droite. Il pourrait être un lien vers une fênetre popup informant que l'accès au connecteur est public.

#5

Mis à jour par Frédéric Péters il y a presque 4 ans

Ouaip, le vert ou n'importe quelle couleur je pense ça sera jamais bien clair.

#6

Mis à jour par Frédéric Péters il y a presque 4 ans

Le point d'exclamation non plus, surtout qu'on a déjà une croix rouge quand le connecteur est down, ça devient compliqué; ma préférence va vraiment à l'ajout d'un explicite "accès ouvert", qui indique quelque chose, sans donner l'impression que c'est à corriger.

Alternativement, diviser en deux l'accueil, une section "services web internes" puis une section "services web publics" ?

(ça ne répond pas à la question "apikey" de Thomas)

#7

Mis à jour par Benjamin Dauvergne il y a presque 4 ans

Thomas Noël a écrit :

Est-ce qu'on met aussi en avant les endpoint qui ont un accès via apikey (ie dont le contenu est également diffusable en clair par simple clic sur une URL dès qu'elle contient apikey=truc) ?

Je ne sais pas, la DINSIC distribue les données de la terre entière via des apikeys et ça n'a pas l'air de les choquer plus que ça, apikey + HTTPS = secure !!

#8

Mis à jour par Thomas Noël il y a presque 4 ans

Benjamin Dauvergne a écrit :

Thomas Noël a écrit :

Est-ce qu'on met aussi en avant les endpoint qui ont un accès via apikey (ie dont le contenu est également diffusable en clair par simple clic sur une URL dès qu'elle contient apikey=truc) ?

Je ne sais pas, la DINSIC distribue les données de la terre entière via des apikeys et ça n'a pas l'air de les choquer plus que ça, apikey + HTTPS = secure !!

On est moins cons que la DINSIC, non ? :) En vrai c'est pas tellement le sujet ici, faisons cet affichage pour les connecteurs ouverts "à tout vent", c'est le cas le plus important.

Sur « l'explicite "accès ouvert" sans donner l'impression que c'est à corriger » : pour moi si un connecteur a au moins un endpoint programmé avec des permissions et que celles-ci sont retirées par configuration, je suis pour mettre une petite alerte. Bon, éventuellement peut être juste « ⚠️ Accès ouvert» (avec le ⚠️ en blink bien sûr)

#9

Mis à jour par Frédéric Péters il y a presque 4 ans

Mais à qui/quoi ça sert ? À nous passant par hasard sur une plateforme et notant ainsi que peut-être un connecteur a été ouvert par mégarde ? (si c'est ça je pense qu'il y a plus adapté).

Et de là je serais plutôt, ou d'abord, pour ajouter un message clair, explicite, avec peut-être une case à cocher, au moment du paramétrage qui ajoute cet accès public.

#10

Mis à jour par Thomas Noël il y a presque 4 ans

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

Et de là je serais plutôt, ou d'abord, pour ajouter un message clair, explicite, avec peut-être une case à cocher, au moment du paramétrage qui ajoute cet accès public.

Excellente idée. Et après cela, le message sur la page /manage et sur la page du connecteur pourra être minimal (un simple «(accès ouvert)» suffira)

#11

Mis à jour par Valentin Deniaud il y a presque 4 ans

  • Assigné à mis à Valentin Deniaud
#12

Mis à jour par Valentin Deniaud il y a presque 4 ans

On comprend quand on ajoute un accès ouvert que c'est grave, par contre ça risque d'être soulant à la longue, peut-être ne le faire que si DEBUG=False ?

Aussi, il faut revoir le style de « Accès ouvert » mais j'ai pas trop d'inspiration.
Je n'ai pas l'impression que les queries que ça ajoute aient un effet sensible sur le temps de chargement de la page mais mon publik local n'est pas très géchar, à voir donc.

#13

Mis à jour par Emmanuel Cazenave il y a presque 4 ans

Tu pourrais mettre des captures d'écran ?

#14

Mis à jour par Valentin Deniaud il y a presque 4 ans

Pour l'affichage sur la page d'accueil (l'alerte à l'ajout est plus dans le genre dynamique, il faut l'essayer).

#15

Mis à jour par Emmanuel Cazenave il y a presque 4 ans

Je suis pour changer la couleur du 'open access' en vert (parce que vert et 'open' ça va ensemble même si le open est un problème ici). Si les gens veulent pas du vert ça peut-être chose mais à mon sens il faut attirer l'attention, c'est l'objet initial du ticket, là ça passe assez inaperçu.

#16

Mis à jour par Valentin Deniaud il y a presque 4 ans

(vert ou orange je laisse le choix (dans la branche j'ai mis vert))

#17

Mis à jour par Frédéric Péters il y a presque 4 ans

Quid de peindre totalement en vert ? Ça les distinguerait nettement. (dans la capture, #0fb30f, plutôt tapé au hasard, et qui manque un peu de contraste (pour un truc qui rend moins bien mais avec un contraste AA, #088a08)).

#18

Mis à jour par Valentin Deniaud il y a presque 4 ans

Moi je n'aime pas le vert, c'est beaucoup trop connoté « tout est normal », le orange associé dans l'imaginaire collectif au warning me paraissait adapté.

Tout peindre c'est sympa. Je veux bien faire le patch mais avec l'assurance que j'aurais un ack et qu'on ne me redemandera pas un changement (sinon autant continuer la discussion tout de suite sans attendre de nouveau code).

#19

Mis à jour par Frédéric Péters il y a presque 4 ans

Vert, c'est ouVERT, et tout colorié ça les distingue suffisamment pour que la personne se pose au moins la question. (j'ai pas de patch j'ai juste torché ça dans l'inspecteur, je vais prendre ta branche et regarder pour te fournir ça demain).

#20

Mis à jour par Frédéric Péters il y a presque 4 ans

Le positionnement absolu proposé échoue quand il y a deux lignes (cf capture).

Patch attaché, qui place l'info "open access" à côté du type de connecteur, et qui colorie le tout en vert.

#23

Mis à jour par Frédéric Péters il y a presque 4 ans

  • Statut changé de Solution proposée à Résolu (à déployer)

Validé poussé,

commit 3ba4e0e6e39550d1050b82dd91e9547ae18242b3
Author: Valentin Deniaud <vdeniaud@entrouvert.com>
Date:   Tue Jun 9 16:26:42 2020 +0200

    views: flag connectors with open access on homepage (#42123)

commit 8ade392b04cba82ebcdc5b7641b9711af67d09a1
Author: Valentin Deniaud <vdeniaud@entrouvert.com>
Date:   Tue Jun 9 15:11:01 2020 +0200

    base: add confirmation when adding open access right (#42123)
#24

Mis à jour par Frédéric Péters il y a presque 4 ans

  • Statut changé de Résolu (à déployer) à Solution déployée

Formats disponibles : Atom PDF