New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Issue 4701 - RFE - Exclude attributes from retro changelog #4723
Conversation
Nice, a few minor issues, but it looks good to me. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Joining to Mark's comments.
And if the run with ASAN is clean - looks good :)
Oh, and put the test to the testing tier, please. So it will run in the GitHub CI. You can do this similar to this test suite: Just by adding this line:
|
ASAN run is clean |
ac927d1
to
d56115d
Compare
d56115d
to
550703d
Compare
Description: When the retro changelog plugin is enabled it writes the added/modified values to the "cn-changelog" suffix. In some cases an entries attribute values can be of a sensitive nature and should be excluded. This RFE adds functionality that will allow an admin exclude certain attributes from the retro changelog DB. Relates: 389ds#4701 Reviewed by: mreynolds389, droideck (Thanks folks)
550703d
to
49803a9
Compare
log.fatal("Changelog search failed, error: " +str(e)) | ||
assert False | ||
|
||
def test_retrocl_exclude_attr_mod(topology_st): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Modify operation test added
@@ -424,6 +430,15 @@ entry2reple(Slapi_Entry *e, Slapi_Entry *oe, int optype) | |||
} | |||
slapi_entry_add_values(e, retrocl_changetype, vals); | |||
|
|||
/* Does this entry contain any excluded attributes */ | |||
for (attrs = oe->e_attrs; attrs != NULL; attrs = attrs->a_next) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Line 434 -> 440 are the bug fix changes
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great!
Looks good to me!
Description: When the retro changelog plugin is enabled it writes the added/modified values to the "cn-changelog" suffix. In some cases an entries attribute values can be of a sensitive nature and should be excluded. This RFE adds functionality that will allow an admin exclude certain attributes from the retro changelog DB. Relates: 389ds#4701 Reviewed by: mreynolds389, droideck (Thanks folks)
…4746) Description: When the retro changelog plugin is enabled it writes the added/modified values to the "cn-changelog" suffix. In some cases an entries attribute values can be of a sensitive nature and should be excluded. This RFE adds functionality that will allow an admin exclude certain attributes from the retro changelog DB. Relates: #4701 Reviewed by: mreynolds389, droideck (Thanks folks)
Description: When the retro changelog plugin is enabled it writes the added/modified values to the "cn-changelog" suffix. In some cases an entries attribute values can be of a sensitive nature and should be excluded. This RFE adds functionality that will allow an admin exclude certain attributes from the retro changelog DB. Relates: #4701 Reviewed by: mreynolds389, droideck (Thanks folks)
Description: The RetroCL exclude attribute RFE is dependent on functionality of the EntryUUID bug fix, that didn't make into the latest build. This breaks the RetroCL exclude attr feature so we need to provide a workaround. Fixes: 389ds#4791 Relates: 389ds#4723 Relates: 389ds#4224 Reviewed by: tbordaz, droideck (Thank you)
Description: The RetroCL exclude attribute RFE is dependent on functionality of the EntryUUID bug fix, that didn't make into the latest build. This breaks the RetroCL exclude attr feature so we need to provide a workaround. Fixes: #4791 Relates: #4723 Relates: #4224 Reviewed by: tbordaz, droideck (Thank you)
Description: The RetroCL exclude attribute RFE is dependent on functionality of the EntryUUID bug fix, that didn't make into the latest build. This breaks the RetroCL exclude attr feature so we need to provide a workaround. Fixes: #4791 Relates: #4723 Relates: #4224 Reviewed by: tbordaz, droideck (Thank you)
Description: When the retro changelog plugin is enabled it writes the
added/modified values to the "cn-changelog" suffix. In
some cases an entries attribute values can be of a
sensitive nature and should be excluded. This RFE adds
functionality that will allow an admin exclude certain
attributes from the retro changelog DB.
Relates: #4701
Reviewed by: ??