#49249 cos_cache is erroneously logging schema checking failure
Closed: wontfix 3 years ago Opened 3 years ago by tbordaz.

Issue Description

Run the attached test case

Package Version and Platform

in 1.3.6

Steps to reproduce

  1. run the attach test case
  2. Check error logs for messages like

    ERR - cos-plugin - cos_cache_query_attr - cos attribute mailAlternateAddress failed schema check on dn: ou=no_cos_scope,dc=example,dc=com

3.

Actual results

The logs appears on all entries evaluated even if they are out of the scope of the cos

Expected results

This log should only be logged in plug-in debugging are enabled (nsslapd-errorlog-level: 65536)


This bug is a side effect of https://pagure.io/389-ds-base/issue/48978. The following commit
https://pagure.io/389-ds-base/c/fb9b9ef0a74f64abd89e52c5c90fe7dfcea6ba71 change the schema check control from 'SLAPI_LOG_PLUGIN' to 'SLAPI_LOG_ERR'

The code of cos cache is quite complex and IMHO should be rewritten.
A cos applies on the subtree of cos definition parent. So it is surprising to see the error message that is also logged on entries OUTSIDE of the scope of the cos. The code being complexe, schema is checked before the scope is tested. So at the end of the day, the cos attribute will not be added to an entry outside of the scope of the cos EVEN if the schema was checked on that entry.

Metadata Update from @tbordaz:
- Custom field type adjusted to defect

3 years ago

Metadata Update from @tbordaz:
- Issue assigned to tbordaz

3 years ago

Metadata Update from @tbordaz:
- Custom field origin adjusted to IPA
- Custom field reviewstatus adjusted to review
- Custom field rhbz adjusted to https://bugzilla.redhat.com/show_bug.cgi?id=1437492

3 years ago

Metadata Update from @mreynolds:
- Custom field reviewstatus adjusted to ack (was: review)

3 years ago

Metadata Update from @mreynolds:
- Issue set to the milestone: 1.3.6.0

3 years ago

git push origin master

Counting objects: 11, done.
Delta compression using up to 8 threads.
Compressing objects: 100% (11/11), done.
Writing objects: 100% (11/11), 2.59 KiB | 0 bytes/s, done.
Total 11 (delta 7), reused 0 (delta 0)
To ssh://git@pagure.io/389-ds-base.git
    063b367..04635e4  master -> master

Metadata Update from @tbordaz:
- Issue close_status updated to: fixed
- Issue status updated to: Closed (was: Open)

3 years ago

git push origin 389-ds-base-1.3.6

Counting objects: 11, done.
Delta compression using up to 8 threads.
Compressing objects: 100% (11/11), done.
Writing objects: 100% (11/11), 2.59 KiB | 0 bytes/s, done.
Total 11 (delta 7), reused 0 (delta 0)
To ssh://git@pagure.io/389-ds-base.git
   2d07ca4..71a98aa  389-ds-base-1.3.6 -> 389-ds-base-1.3.6

389-ds-base is moving from Pagure to Github. This means that new issues and pull requests
will be accepted only in 389-ds-base's github repository.

This issue has been cloned to Github and is available here:
- https://github.com/389ds/389-ds-base/issues/2308

If you want to receive further updates on the issue, please navigate to the github issue
and click on subscribe button.

Thank you for understanding. We apologize for all inconvenience.

Metadata Update from @spichugi:
- Issue close_status updated to: wontfix (was: fixed)

8 months ago

Login to comment on this ticket.

Metadata