madhatter22 / LinqToLdap

C# LINQ provider built on top of System.DirectoryServices.Protocols for querying and updating LDAP servers.
MIT License
45 stars 23 forks source link

Use more reasonable exception and fix C# doc #40

Open rklec opened 1 month ago

rklec commented 1 month ago

Breaking change, potentially!

InvalidOperationException fits better than both other exceptions. Also, the C# doc did miss the immediate exception here.

IMHO the LDAP connection has nothing to do with mapping...

includes https://github.com/madhatter22/LinqToLdap/pull/39 (separated it just in case you did not want to merge it)