Discussion :

[ludojojo]

Bonjour,

Je cherche à récupérer la liste de tous les utilisateurs de mon AD qui ne se sont pas connecter depuis plus de "90" jours.
Pour se faire, je parcours l'ensemble de l'annuaire à la recherche de la propriété "LastLogon". Cela fonctionne bien sauf pour certains utilisateurs...
En effet, mon AD est très vieux et à vécu plusieurs migrations.

Mon problème c'est que avec certains utilisateurs, j'ai un message d'erreur :

Impossible d'exécuter la requête : L'index était hors limites. Il ne doit pas être négatif et doit être inférieur à la taille de la collection. Nom du paramètre : index

Hors cela fonctionne très bien pour les autres...

Voici mon code :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
[...]
DirectoryEntry ldap = new DirectoryEntry(ConfigurationSettings.AppSettings["LdapUrl"], ConfigurationSettings.AppSettings["LdapLogin"], ConfigurationSettings.AppSettings["LdapPwd"]);
DirectorySearcher searcher = new DirectorySearcher(ldap);
searcher.Filter = "(&(objectClass=user)(employeeID=*)(mail=*))";
foreach (string str in adProperties)
   searcher.PropertiesToLoad.Add(str);
SearchResultCollection AdUsers = searcher.FindAll();
foreach (SearchResult result in AdUsers)
{
   int flag = int.Parse(result.Properties["userAccountControl"][0].ToString());
   if (!Convert.ToBoolean(flag & 0x0002))
   {
      if (result.Properties["lastLogon"][0] != null && long.Parse(result.Properties["lastLogon"][0].ToString()) > 0)
      {
         TimeSpan ts = DateTime.Now - new DateTime(1601, 01, 01).AddTicks((long)result.Properties["lastLogon"][0]);
         if (ts.Days >= 90)
         {[...]

Quelqu'un aurait-il une idée ?

[Max]

Salut.

Tu as pu identifier précisément les entrées/valeurs qui posaient problème ?

Si oui, peux-tu tester le code suivant (enfin quelque chose qui s'en approche), manipuler plutôt ce type d'objets et dire ce que tu obtiens :

Code :

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

1
2
3
IADsLargeInteger lastLogonAsIADs = (IADsLargeInteger)result.Properties["lastLogon"][0];
long lastLogonAsLong = (lastLogonAsIADs.HighPart * 0x100000000) + lastLogonAsIADs.LowPart; 
DateTime lastLogonAsDateTime = DateTime.FromFileTimeUtc(lastLogonAsLong);

Je n'ai jamais manipulé ce champ mais j'ai déjà eu des soucis avec d'autres du même type (Large Integer/Interval) .

[ludojojo]

Tu as pu identifier précisément les entrées/valeurs qui posaient problème ?

Oui... Pour la valeur "129507190229911135" par exemple !

Si oui, peux-tu tester le code suivant (enfin quelque chose qui s'en approche), manipuler plutôt ce type d'objets et dire ce que tu obtiens :

Code :

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

1
2
3
IADsLargeInteger lastLogonAsIADs = (IADsLargeInteger)result.Properties["lastLogon"][0];
long lastLogonAsLong = (lastLogonAsIADs.HighPart * 0x100000000) + lastLogonAsIADs.LowPart; 
DateTime lastLogonAsDateTime = DateTime.FromFileTimeUtc(lastLogonAsLong);

Je n'ai jamais manipulé ce champ mais j'ai déjà eu des soucis avec d'autres du même type (Large Integer/Interval) .

C'est fait, cela règle le premier problème, mais une autre erreur apparait :

Impossible d'effectuer un cast d'un objet de type 'System.Int64' en type 'ActiveDs.IADsLargeIntegrer'. [...] the value must be a number less than infinity

[Max]

Stp tu peux dire à quelle ligne exactement tu as la nouvelle exception ? Poster ton nouveau code ?

[ludojojo]

Stp tu peux dire à quelle ligne exactement tu as la nouvelle exception ? Poster ton nouveau code ?

Voici le code :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
try
{
    IADsLargeInteger lastLogonAsIADs = (IADsLargeInteger)result.Properties["lastLogon"][0];
    long lastLogonAsLong = (lastLogonAsIADs.HighPart * 0x100000000) + lastLogonAsIADs.LowPart;
    DateTime lastLogonAsDateTime = DateTime.FromFileTimeUtc(lastLogonAsLong);
    if (lastLogonAsDateTime.Day > 90)
    {
        ADUser u = new ADUser();
        u.expires = lastLogonAsDateTime.Day.ToString();
        u.completeName = result.Properties["cn"][0].ToString();
        if (result.Properties["employeeID"].Count != 0)
            u.employeeID = result.Properties["employeeID"][0].ToString();
        users.Add(u);
    }
}
catch (Exception e)
{
    users.Add(new ADUser(result.Properties["cn"][0].ToString(), e.Message));
}

Le résultat est :

<cn>User Name</cn>
<messageException>Impossible d'effectuer un cast d'un objet de type 'System.Int64' en type 'ActiveDs.IADsLargeInteger'.</messageException>

Et ce pour l'ensemble des enregistrements...
L'exception est levée à la première ligne :

Code :

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

IADsLargeInteger lastLogonAsIADs = (IADsLargeInteger)result.Properties["lastLogon"][0];

[Max]

Si oui, peux-tu tester le code suivant (enfin quelque chose qui s'en approche)

J'ai bien fait de prendre des pincettes .

Je pense avoir pigé le problème : je t'ai passé un bout de code dans lequel j'utilisais des PropertyCollection. Or, toi, tu utilises des ResultPropertyCollection... et donc tu as déjà un long à disposition !

Ton souci concerne donc uniquement la conversion du file time UTC en DateTime :

Code :

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

DateTime lastLogonAsDateTime = DateTime.FromFileTimeUtc((long)result.Properties["lastLogon"][0]);

Ça devrait rouler maintenant .

[ludojojo]

Ton souci concerne donc uniquement la conversion du file time UTC en DateTime :

Code :

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

DateTime lastLogonAsDateTime = DateTime.FromFileTimeUtc((long)result.Properties["lastLogon"][0]);

Ça devrait rouler maintenant .

En effet.
Cela dit, mon ancien code fonctionne de nouveau...
Allez savoir pourquoi cela ne fonctionnait pas...

Code :

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

1
2
TimeSpan ts = DateTime.Now - new DateTime(1601, 01, 01).AddTicks((long)result.Properties["lastLogon"][0]);
                                                        if (ts.Days > JobsConstantes.NB_DAYS_BEFORE_UNUSED_STATE)

Merci pour ton aide.