Un message à la fermeture de la fenêtre

Version imprimable

Bonjour à tous,

Dans une interface administration privée, je souhaite, pour des mesures de sécurité, forcer à cliquer un lien "sortie" qui tuerait la session PHP plutôt que la croix de fermeture du navigateur. J'ai écrit la petite fonction suivante :
Code:

1 2 3 4 5 6 7 8 9 10 11 function avert_fermeture() { if(confirm('Avez-vous pensé à cliquer sur le bouton \"Sortie\" pour vous déconnecter de votre interface Administration ? Le fait de laisser votre session active constitue une faille de sécurité pour d\'éventuels hackers.')){ window.close(); }else{ return false; } } window.onbeforeunload = function (){avert_fermeture()};
Le message s'affiche bien, mais le navigateur se ferme quelque soit le choix que l'on clique. Mais est ce que c'est possible ?
Où le choix du user est prioritaire quoi que l'on fasse ?

Merci d'avance de vos lumières.

Précision : je vois cette fonction sur le web :
Code:

1 2 3 4 <script type="text/javascript"> window.onbeforeunload = function(e){ return 'Avez-vous pensé à cliquer sur le bouton \"Sortie\" pour vous déconnecter de votre interface Administration ? Le fait de laisser votre session active constitue une faille de sécurité pour d\'éventuels hackers...'; } </script>
Mais elle se déclenche à chaque clic sur un lien interne ce qui n'est pas le but. Je voudrais que ce soit vraiment à le fermeture du navigateur...

23/01/2010, 16h05
Bovino

Citation:

Dans une interface administration privée

Je comprends bien pourquoi tu apportes cette précision, cependant il n'existe pas de version de JavaScript spécifique aux interfaces d'administration donc d'une part, il semble évident qu'il ne doit pas être possible d'empêcher de quitter une page et encore moins de d'empêcher de fermer le navigateur, d'autre part, il est malgré tout possible de demander à l'utilisateur s'il veut vraiment quitter la page (un simple onbeforeunload = return false suffit) mais JavaScript est incapable de savoir si tu continues à naviguer sur le même site ou non.
23/01/2010, 16h15
renaud26

Merci de ta précision.
Effectivement, il ne s'agit en aucun cas de forcer qui que ce soit à rester sur une page. C'est privé.

J'ai essayé le unbeforeunload, mais il se produit à chaque clic sur un lien interne au cours de la navigation au sein de l'interface admin, ce qui est un peu pénible... Il n'est pas possible de le déclencher uniquement à la fermeture du navigateur (la vraie "sortie") ? car le but est de faire en sorte qu'il clique sur un lien de sortie pour détruire la session.

Une solution pourrait être de spécifier un nom de classe spécifique pour les liens autorisés (par exemple "lienOK") puis de faire :

Code:

1
2
3
4
5
6
7
8
9
window.onload = function(){
    window.onbeforeunload = function(){return false};
    var tempLiens = document.getElementsByTagName('a'), nbLiens = tempLiens.length;
    for(var i=0; i<nbLiens; i++){
        if(templiens[i].className == 'lienOK'){
            tempLiens[i].onclick = function(){window.onbeforeunload = function{return true;}}
        }
    }
}

Oui...je pense que cette idée est bonne !
Il y avait quelques bugs de syntaxe dans la console FF, j'ai mis :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
 
<script type="text/javascript">
window.onload = function(){
    window.onbeforeunload = function(){return 'Avez-vous pensé à cliquer sur le bouton \"Sortie\" pour vous déconnecter de votre interface Administration ? Le fait de laisser votre session active constitue une faille de sécurité pour d\'éventuels hackers...'};
    var tempLiens = document.getElementsByTagName('a'), nbLiens = tempLiens.length;
    for(var i=0; i<nbLiens; i++){
        if(tempLiens[i].className == 'lienMenu'){
            tempLiens[i].onclick = function(){window.onbeforeunload = function(){return true;}}
        }
    }
}
</script>

Mais là, problème : non seulement mon message d'alerte ne s'affiche pas, mais il ne tient pas compte de l'exception et se déclenche sur les liens même si leur classe est "lienMenu".

23/01/2010, 19h01
Bovino

Attention !
Si tu fais :

Code:

window.onbeforeunload = function(){return 'Avez-vous pensé à cliquer sur le bouton \"Sortie\" pour vous déconnecter de votre interface Administration ? Le fait de laisser votre session active constitue une faille de sécurité pour d\'éventuels hackers...'};

Cela n'a aucun effet.
Le fait de renvoyer false sur onbeforeunload génère un message de demande de confirmation. Ce message ne peut pas être modifié et c'est le seul moyen d'empêcher de sortir de la page.
Sinon, pour annuler le onbeforeunload, ça m'étonne un peu, mais essaye peut-être comme ça :

Code:

tempLiens[i].onclick = function(){window.onbeforeunload = null;}
23/01/2010, 19h22
renaud26

Non, effectivement... c'est pareil : chaque lien cliqué provoque le msg, même si sa classe est définie dans la fonction js.
23/01/2010, 19h54
supersnail

Bonjour,

à mon avis,le passage à l'autre site doit se faire avant le onclick...
Il faudrait donc faire une navigation par le biais de javascript et non "classique",ce qui nous laisserait le temps de désactiver le message ;)
23/01/2010, 20h01
renaud26

Citation:

Envoyé par supersnail

Bonjour,

à mon avis,le passage à l'autre site doit se faire avant le onclick...
Il faudrait donc faire une navigation par le biais de javascript et non "classique",ce qui nous laisserait le temps de désactiver le message ;)

Bonsoir,

Humm...ça c'est pas gagné et me ferait restructurer beaucoup de choses...
Mais je pense à un truc (peut-être débile, m'enfin...) :

Existe-t-il un objet qui considère la fenêtre entière du navigateur, croix de fermeture incluse ? Si oui, ne serait-il pas possible, avec les offsets, de créer un genre de "zone réactive" qui pourrait générer un confirm() au passage souris ?

8O

Ca m'étonne, car

Code:

1
2
3
4
5
6
7
window.onload = function(){
    window.onbeforeunload = function(){return 'Avez-vous pensé à cliquer sur le bouton \"Sortie\" pour vous déconnecter de votre interface Administration ? Le fait de laisser votre session active constitue une faille de sécurité pour d\'éventuels hackers...'};
    var tempLiens = document.getElementsByTagName('a'), nbLiens = tempLiens.length;
    for(var i=0; i<nbLiens; i++){
        tempLiens[i].onclick = function(){window.onbeforeunload = null;}
    }
}

fonctionne bien sur FF :?

Concernant la solution que tu proposais, ce n'est pas possible, l'interface du navigateur est un programme exécuté par l'OS du poste client, heureusement que JavaScript n'y a pas accès !

23/01/2010, 20h20
renaud26

:D

Non seulement ça fonctionne impec sur FF, mais également sur IE !
Alors là, bravo, Monsieur l'Administrateur / Rédacteur / Responsable et chapeau bas !

Même pas besoin de spécifier de classe de lien. C'est top.
Je te remercie beaucoup et bonne soirée.
23/01/2010, 20h29
Bovino

Citation:

Envoyé par renaud26

Monsieur l'Administrateur / Rédacteur / Responsable et chapeau bas !

Non, pas Administrateur :)
Mais sinon merci ;)

Citation:

Même pas besoin de spécifier de classe de lien. C'est top.
Je te remercie beaucoup et bonne soirée.

En fait, j'avais prévu cette option au cas où tu aies des liens dans ton admin qui fassent sortir de l'interface et donc qui devraient être traités comme une fermeture de la page.
24/01/2010, 11h42
renaud26
Bonjour à tous,

je me permets de rouvrir ce post, car je voudrais ajouter d'autres exceptions à la balise <a> pour que le warning ne se déclenche pas :

- La validation d'un formulaire.
- Un reload ou un window.location.href JS, sur un select, par exemple.

Est-ce possible ?
Faut-il que j'incrémente la var nbliens :
Code:

1 2 3 4 5 var tempLiens = document.getElementsByTagName('a'); var tempLiens += document.getElementsByTagName('Submit'); ...etc... nbLiens = tempLiens.length;
Merci de votre patience...
24/01/2010, 11h53
Bovino

Non, tu ne peux pas faire comme ça, déjà parce qu'il n'existe pas de dalise <submit> en HTML :aie: mais surtout parce que les méthodes getElementsBy renvoient des collections, c'est-à-dire au sens de JavaScript, des objets Object, or il n'existe pas de méthode native pour fusionner deux objets.
Le plus simple serait selon moi de récupérer les input et les filtrer en fonction de leur type.
24/01/2010, 11h58
renaud26

Hello, merci d'être encore là.
Oui, oup's par "Submit" je voulais dire "<form>"
:oops:

Citation:

Le plus simple serait selon moi de récupérer les input et les filtrer en fonction de leur type.

Humm... j'ai bien peur que "le plus simple pour toi" ne devienne une effroyable usine à gaz pour moi...
Peux-tu me donner un exemple de ce "filtrage" ?

Code:

1
2
3
4
5
6
7
8
9
 
    var textboxes = document.getElementsByTagName("input");
    for(i=0;i<textboxes.length;i++)
    {
            if(textboxes[i].type == 'submit')  
            {
                 //ce qu'il y a à faire
            }
    }

par exemple ;)

24/01/2010, 16h43
renaud26
Euh... une dernière question car un truc m'étonne :
:oops:

Comment se fait-il que :
Code:

1 2 3 4 5 6 for(var i=0; i<nbLiens; i++){ tempLiens[i].onclick = function(){window.onbeforeunload = null;} }
rende inopérant les liens d'ouverture de popup ?

Parce qu'affecter le onclick comme ça écrase les événements déjà présents... :aie:
Essaye avec des fonctions (fais une recherche dessus) du type addEvent ou éventuellement :
Code:

1 2 3 4 5 6 7 8 var tempEvent; for(var i=0; i<nbLiens; i++){ tempEvent = tempLiens[i].onclick; tempLiens[i].onclick = function(){ tempEvent(); window.onbeforeunload = null; } }

25/01/2010, 10h01
renaud26

Bonjour,

Je viens de regarder du côté de ces fonctions addEventListener et attachEvent, qui déjà n'ont pas le même comportement avec IE et FF, mais bon...ça dépasse mes compétences même si j'ai bien compris à quoi elles servent - assigner un évènement à un objet JS - je ne vois pas comment les adapter à ce cas précis...
Dommage.
;)

Citation:

Envoyé par Bovino

ou éventuellement :

Code:

1
2
3
4
5
6
7
8
var tempEvent;
for(var i=0; i<nbLiens; i++){
    tempEvent = tempLiens[i].onclick;
    tempLiens[i].onclick = function(){
        tempEvent();
        window.onbeforeunload = null;
    }
}

:koi:

25/01/2010, 10h35
renaud26

Mille excuses ! tellement absorbé dans la lecture de ces fonctions que j'ai oublié de te donner le résultat de ta soluce.
:oops:

- Le message s'affiche sur tous les liens.
- La console FF sort : "tempEvent is not a function"

Code:

1
2
3
4
5
6
7
8
var tempEvent;
for(var i=0; i<nbLiens; i++){
    tempEvent = tempLiens[i].onclick;
    tempLiens[i].onclick = function(){
        if(tempEvent){tempEvent();}
        window.onbeforeunload = null;
    }
}

:roll:

25/01/2010, 12h03
renaud26

Bon, encore une fois, merci de ta patience.

Cette fois, plus d'erreur FF, mais les liens window.open des popups sont toujours désactivés.

Fuseau horaire GMT +2. Il est actuellement 17h27.