Bug #60041
Scheduler Task aborts if LDAP returns entry without dn
Status: | Closed | Start date: | 2014-07-01 | |
---|---|---|---|---|
Priority: | Should have | Due date: | ||
Assigned To: | Armin Ruediger Vieweg | % Done: | 100% |
|
Category: | - | |||
Target version: | 1.3.0 | |||
TYPO3 Version: |
Description
- Get all entries by predefined settings
- Get details for each entry by dn
This may be such a result for step 1:
But as you see, somehow I get entries back, which has no dn. So the second step, which fetches the details by dn, fails.
The fix is very simple (I've attached a patch, and will push it to Gerrit as well), but now the scheduler task does not interrupt anymore.
Related issues
Associated revisions
[BUGFIX] Scheduler task aborts if LDAP returns entry without DN
Sometimes it happens that LDAP returns records with empty DN,
but this DN is required to synchronize record details from LDAP
to TYPO3.
This fix just checks if the requested record details are a valid
array, and skips this whole entry, if not.
Resolves: #60041
Change-Id: I2dc36374e8a82f174ac63907aaccc25789c0a324
Reviewed-on: https://review.typo3.org/31266
Reviewed-by: Xavier Perseguers
Tested-by: Xavier Perseguers
History
#1 Updated by Gerrit Code Review about 1 year ago
- Status changed from New to Under Review
Patch set 1 for branch master of project TYPO3CMS/Extensions/igLdapSsoAuth has been pushed to the review server.
It is available at https://review.typo3.org/31266
#2 Updated by Gerrit Code Review about 1 year ago
Patch set 2 for branch master of project TYPO3CMS/Extensions/igLdapSsoAuth has been pushed to the review server.
It is available at https://review.typo3.org/31266
#3 Updated by Armin Ruediger Vieweg about 1 year ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Applied in changeset c52d63b97c7ea5ed251f938792113df963254530.
#4 Updated by Xavier Perseguers about 1 year ago
- Assigned To set to Armin Ruediger Vieweg
- Target version set to 1.3.0
#5 Updated by Xavier Perseguers 11 months ago
- Status changed from Resolved to Closed