[release/7.0] Put back setting of field in DirectorySearcher.FindAll #79192
Conversation
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
This was removed accidentally due to erroneously thinking it was a local rather than a field.
|
Tagging subscribers to this area: @dotnet/area-system-directoryservices, @jay98014 Issue DetailsBackport of #79093 to release/7.0 /cc @stephentoub Customer ImpactTestingRiskIMPORTANT: Is this backport for a servicing release? If so and this change touches code that ships in a NuGet package, please make certain that you have added any necessary package authoring and gotten it explicitly reviewed.
|
stephentoub
added
Servicing-consider
|
@stephentoub since this is an OOB package, I added the missing change to the src csproj. |
|
Approved by Tactics for 7.0.3. |
|
CI failure is unrelated (networking infra problem). Reported the failures here: #75611 |
ghost
locked as resolved and limited conversation to collaborators
Backport of #79093 to release/7.0
/cc @stephentoub
Customer Impact
Various aspects of LDAP directory searching using DirectorySearcher / DirectorySynchronization are broken, e.g. number of results reported might be 0 even when it's not, search results that should have been cached aren't, etc. We inadvertently removed a few lines of code that stored the result of a search for later use.
Testing
Existing tests.
Risk
Minimal. We're just reverting a few lines that were removed erroneously.