Tmp ldapauth2

[sunflowerbofh]

Description

This is just a slight enhancement of "Fixed #9063: Ask LDAP for user DN, don't concatenate username+baseDN." ->
#11544

Fixes #9063

Type of change

• Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?

The same way as in #11544

Test Configuration:

• PHP version: 8.1
• MySQL version: mariadb-server-10.6
• Webserver version: apache 2.4.54
• OS version: Debian bookworm

Checklist:

• I have read the Contributing documentation available here: https://snipe-it.readme.io/docs/contributing-overview
• I have formatted this PR according to the project guidelines: https://snipe-it.readme.io/docs/contributing-overview#pull-request-guidelines
• My code follows the style guidelines of this project
• I have performed a self-review of my own code
• I have commented my code, particularly in hard-to-understand areas
• I have made corresponding changes to the documentation
• My changes generate no new warnings
• I have added tests that prove my fix is effective or that my feature works
• New and existing unit tests pass locally with my changes (see comments in Fixed #9063: Ask LDAP for user DN, don't concatenate username+baseDN. #11544)

[uberbrady]

Exquisite. Let me test a little then I’d love to accept. Thank you!

[snipe]

@uberbrady any luck testing this? I'd love to get this merged or closed.

[snipe]

Ping @uberbrady :)

[snipe]

@uberbrady is this PR still necessary given all of the other changes we've made since it was opened?

[uberbrady]

@snipe I'm afraid so; I do still think it could be useful. Unfortunately my test rig still isn't quite working yet. We might need to ask @Godmartinz to test?

[snipe]

@uberbrady re-ping on this for v7. I know the test rig is up. Can you test this?

[uberbrady]

Does this #17832 PR do the same thing as yours?