Skip to content
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

sdap: Log hint for ignore unreadable references #7604

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

justin-stephenson
Copy link
Contributor

in AD direct integration with ldap_id_mapping = false when adding a custom ldap_user_search_base I encountered this cryptic error when running getent group posixgroup

(2024-09-18 14:20:43): [be[ad.test]] [sdap_nested_group_single_step_process] (0x0040): [RID#3] Unknown entry type [CN=nonposixuser,CN=Users,DC=ad,DC=test]!
(2024-09-18 14:20:43): [be[ad.test]] [sdap_nested_group_single_step_done] (0x0020): [RID#3] Error processing direct membership [22]: Invalid argument
(2024-09-18 14:20:43): [be[ad.test]] [sdap_nested_done] (0x0020): [RID#3] Nested group processing failed: [22][Invalid argument]
(2024-09-18 14:20:43): [be[ad.test]] [sdap_id_op_destroy] (0x4000): [RID#3] releasing operation connection
(2024-09-18 14:20:43): [be[ad.test]] [sdap_id_op_done] (0x4000): [RID#3] releasing operation connection
(2024-09-18 14:20:43): [be[ad.test]] [sdap_id_conn_data_idle] (0x4000): [RID#3] Marking connection as idle
(2024-09-18 14:20:43): [be[ad.test]] [ad_account_info_done] (0x0040): [RID#3] ad_handle_acct_info_recv failed [22]: Invalid argument

It was only when I checked the code I realized that there is this ldap_ignore_unreadable_references boolean. Setting it to true (default is false) fixed this issue for me. I wanted to add a hint to help administrators.

    case SDAP_NESTED_GROUP_DN_UNKNOWN:
        if (state->ignore_unreadable_references) {
            DEBUG(SSSDBG_TRACE_FUNC, "Ignoring unreadable reference [%s]\n",
                  state->current_member->dn);
        } else {
            DEBUG(SSSDBG_OP_FAILURE, "Unknown entry type [%s]!\n",
                  state->current_member->dn);
            ret = EINVAL;
            goto done;
        }
        break;
    }

Copy link
Contributor

@sumit-bose sumit-bose left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi,

thanks for the fix, ACK.

bye,
Sumit

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Accepted backport-to-stable Targets also latest stable branch Trivial
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants