search

codegouvfr / keycloak-theme-dsfr

Generic Keycloak DSFR theme
MIT License
17 stars 6 forks source link

Question à propos des TOS #3

Closed hjonin closed 9 months ago

hjonin commented 1 year ago

Est-ce que les TOS ici (qui semblent correspondre à ceux du SILL) sont utilisés et sont utilisés tels quels (afin de savoir s'il faut mettre à jour l'adresse du SILL à l'intérieur) ?

hjonin commented 1 year ago

Même question pour https://github.com/codegouvfr/keycloak-theme-dsfr/blob/6c936e0c3131ea099e5af7f333bf5e9940bbbd2c/public/index.html

bzg commented 1 year ago

A mon avis ce sont simplement des informations qu'il faut nettoyer de keycloak-theme-dsfr.

@garronej tu confirmes?

garronej commented 1 year ago

@hjonin Yes, pour le title in public oui tu as raison il faut que je vire ça.

Pour les TOS du SILL, je ne crois pas y faire référence dans le code si ce n'est dans le readme, comme exampte de configuration:

image

Les gens fournissent la propre terms en environement variable (de manière optionel).

Ca te gène qu'on utilise les tos du sill comme example?

hjonin commented 1 year ago

@garronej pas de problème pour l'utiliser en exemple dans le README !

Ce sont bien les références dans public/ qu'il semble falloir nettoyer (public/tos_fr.md et public/index.html).

sbourdon13 commented 9 months ago

Hello @garronej @hjonin,

J'ai le même problème concernant public/index.html. Dans mon projet, j'ai uniquement récupéré l'archive jar du thème keycloak dsfr et configuré les variables d'environnement fournies en exemple. Sur ma page de login, j'ai beaucoup de balises du SILL : title, meta et script qui envoie du tracking à Matomo.

Donc quand je copie colle ma page de login dans Teams ou Outlook ça prête à confusion 😬 image

Et le RSSI nous demande pourquoi on envoie une requête vers une instance matomo/piwik de data.gouv.fr.

Je n'ai pas bien compris comment "nettoyer" ce fichier : l'idée serait de cloner ce repo, enlever les balises HTML qui ne correspondent pas à mon projet et remplacer le title, puis regénérer l'archive du thème grâce à Keycloakify ?

Merci !

garronej commented 9 months ago

@sbourdon13 @hjonin, Je viens de nétoyer les balises relative au SILL, j'ai aussi enlever le TOS du SILL du dossier publique.

Une nouvelle version est en train d'être publiée.

Merci d'avoir reporté!

bzg commented 9 months ago

Merci @garronej - est-ce que le suivi via data.gouv.fr a aussi été retiré ?

PS: Je rouvre l'issue tant dans l'attente.

garronej commented 9 months ago

@bzg Ah oui en effet non, je viens de le faire. Une nouvelle version est en cour de publication par le CI.

bzg commented 9 months ago

Top, merci !

SaraBourdon-Gouv commented 9 months ago

Hello je me rends compte que j'avais oublié de vous remercier pour le correctif super rapide alors merci 🙏