Discussion :

[dharkan]

Bonjour,

j'ai un bouton dans un page html qui me permet de me connecter sur requête LDAP à notre domaine. Lors du clique, si la connexion est réussie mon alert javascript s'affiche bien
pour informer l'utilisateur qu'il est bien connecter. Par contre dans le cas où la connexion est un échec, le JavaScript ne s’exécute pas.

voici le code behind :

Code vb.net :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
Protected Sub Button1_Click(sender As Object, e As EventArgs) Handles btCnx.Click
        If Authenticate(txtNomUsr.Text, txtMdp.Text, System.Configuration.ConfigurationManager.AppSettings("domainName")) Then
            Response.Write("<script type='text/javascript' language=javascript>window.onload = function() {alert('Connexion réussie.');};</script>")
        Else
            Response.Redirect("connection.aspx")
            Response.Write("<script type='text/javascript' language=javascript>window.onload = function() {alert('Échec de l'authentification :" & ErrorMsg & "');};</script>")
        End If
    End Sub

dans mon aspx :

Code html :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
<div id ="LogonScreen">
            <asp:Label ID="Label1" runat="server" Text="Nom d'utilisateur"></asp:Label> &nbsp;&nbsp;
            <asp:TextBox ID="txtNomUsr" runat="server"></asp:TextBox><br />
            <asp:Label ID="Label2" runat="server" Text="Mot de passe"></asp:Label> &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;
            <asp:TextBox ID="txtMdp" runat="server" TextMode="Password"></asp:TextBox>
            <br />
            <asp:Button ID="btCnx" runat="server" Text="Valider" />
        </div>

Que dois-je faire pour que l'alert s affiche aussi en cas d'échec de connexion ?
D'avance merci.

[DotNetMatt]

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
        Else
            Response.Redirect("connection.aspx")
            Response.Write("<script type='text/javascript' language=javascript>window.onload = function() {alert('Échec de l'authentification :" & ErrorMsg & "');};</script>")
        End If

Le problem dans ton Else, c'est que quand tu appelles Response.Redirect, cela redirige directement l'utilisateur donc tout ce qui est apres (en l'occurence le Response.Write) ne sera pas execute. En interne, Response.Redirect appelle Thread.Abort qui lance une System.Threading.ThreadAbortException, ce qui a pour effet de stopper immediatement le thread en cours.

Ce que tu peux faire, c'est soit ajouter ton message d'erreur en Query String, par exemple :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
        Else
            Response.Redirect("connection.aspx?msg=" & HttpUtility.UrlEncode("Échec de l'authentification :" & ErrorMsg))
        End If

Puis recuperer le Query String pour afficher le message sur ta page. Bien sur cela n'est pas recommande, car l'utilisateur pourra facilement modified le message a la main, et si ton message est trop long ca peut causer quelques soucis...

Une meilleure solution est de passer ton message via la Session par exemple, puis de le recuperer dans ta page connection.aspx pour l'afficher :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
        Else
            Session("Message") = "Échec de l'authentification :" & ErrorMsg
            Response.Redirect("connection.aspx")
        End If

Et dans connection.aspx quelque chose du genre :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

tonLabel.Text = Session("Message")

Enfin, je t'invite a te documenter sur l'utilisation correcte de Response.Redirect : Correct use of System.Web.HttpResponse.Redirect.

[dharkan]

Bonjour DotNetMatt,

Merci pour ta réponse.
J'ai mis en place la solution que tu préconisais. et lire la doc sur le Response.redirect que tu m'a mis en lien.

Cordialement.

Une meilleure solution est de passer ton message via la Session par exemple, puis de le recuperer dans ta page connection.aspx pour l'afficher :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
        Else
            Session("Message") = "Échec de l'authentification :" & ErrorMsg
            Response.Redirect("connection.aspx")
        End If

Et dans connection.aspx quelque chose du genre :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

tonLabel.Text = Session("Message")

Enfin, je t'invite a te documenter sur l'utilisation correcte de Response.Redirect : Correct use of System.Web.HttpResponse.Redirect.