LastLogon Active Directory

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 :

Citation:

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:

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 ?

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:

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) ;).

Citation:

Envoyé par _Max_

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

Oui... Pour la valeur "129507190229911135" par exemple !
Citation:
Envoyé par _Max_
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:

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 :

Citation:

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

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

Citation:

Envoyé par _Max_

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

Voici le code :
Code:

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 :

Citation:

<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:

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

Citation:

Envoyé par _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:

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

Ça devrait rouler maintenant ;).

Citation:

Envoyé par _Max_

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

Code:

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

Ça devrait rouler maintenant ;).

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

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.