Properties are also shown as available field mappings

Created on 7 August 2018, almost 6 years ago
Updated 12 April 2024, 2 months ago

The dropdown should only show fields as fields and not properties, e.g. Property: Username is duplicated as Field: Name.

The fields are duplicate.

[field.name]
[field.pass]
[field.mail]
[field.status]

[property.status]

πŸ› Bug report
Status

Fixed

Version

4.0

Component

Code

Created by

πŸ‡¨πŸ‡­Switzerland grahl ZΓΌrich

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Merge Requests

Comments & Activities

Not all content is available!

It's likely this issue predates Contrib.social: some issue and comment data are missing.

  • πŸ‡ΊπŸ‡ΈUnited States bluegeek9
  • Pipeline finished with Skipped
    6 months ago
    #70839
  • Status changed to Fixed 6 months ago
  • πŸ‡ΊπŸ‡ΈUnited States bluegeek9
  • Automatically closed - issue fixed for 2 weeks with no activity.

  • πŸ‡©πŸ‡ͺGermany alxn

    It might also help to warn if now excluded properties are found in ldap_user.settings config.
    We faced an issue with ldap 4.7 where the following old (ldap v4.6) field mappings (field-name, field-mail) from ldap_user.settings config where not visible in the UI under /admin/config/people/ldap/user/drupal
    Excerpt from ldap_user.settings

    ldapUserSyncMappings:
      drupal:
        field-name:
          ldap_attr: '[samaccountname]'
          user_attr: '[field.name]'
          convert: false
          user_tokens: ''
          config_module: ldap_user
          prov_module: ldap_user
          prov_events:
            - create_drupal_user
            - sync_to_drupal_user
        field-mail:
          ldap_attr: '[mail]'
          user_attr: '[field.mail]'
          convert: false
          user_tokens: ''
          config_module: ldap_user
          prov_module: ldap_user
          prov_events:
            - create_drupal_user
            - sync_to_drupal_user

    After saving the form under /admin/config/people/ldap/user/drupal these entries field-name, field-mail were removed from ldap_user.settings config and ultimately the mapping for ldap attribute "mail" was missing, because the "mail_attr" was not set in ldap_servers.server.xyz config. For us the fix was to set the mail_attr in the ldap_servers.server.xyz config.

  • Status changed to Active 3 months ago
  • πŸ‡ΊπŸ‡ΈUnited States bluegeek9
  • Merge request !96Resolve #2991181 "Bug" β†’ (Merged) created by bluegeek9
  • Pipeline finished with Skipped
    3 months ago
    #130925
  • Status changed to Fixed 3 months ago
  • πŸ‡ΊπŸ‡ΈUnited States bluegeek9
  • πŸ‡©πŸ‡ͺGermany alxn

    Thanks for the fixes so far in v4.8, but now in my case an error occurred when upgrading from 4.7 to 4.8 with empty user defined "drupal mappings".

    ...
    - Upgrading drupal/ldap (4.7.0 => 4.8.0): Extracting archive
      - Upgrading drupal/ldap_servers (4.7.0 => 4.8.0)
    ...
    Generating autoload files
    55 packages you are using are looking for funding.
    Use the `composer fund` command to find out more!
    No security vulnerability advisories found.
     ----------- ----------- --------------- ------------------------------------- 
      Module      Update ID   Type            Description                          
     ----------- ----------- --------------- ------------------------------------- 
      ldap_user   8408        hook_update_n   8408 - Fix account creation setting  
                                              spelling and Drupal mappings.        
     ----------- ----------- --------------- ------------------------------------- 
    
    
     // Do you wish to run the specified pending updates?: yes.                     
    
    >  [notice] Update started: ldap_user_update_8408
    >  [error]  array_key_exists(): Argument #2 ($array) must be of type array, null given 
    >  [error]  Update failed: ldap_user_update_8408 
     [error]  Update aborted by: ldap_user_update_8408 
     [error]  Finished performing updates. 
    
  • πŸ‡³πŸ‡±Netherlands cpdp

    Also, the checked value of config key 'acctCreation' has changed from 'ldap_behaviour' to 'ldap_behavior'. This prevented my automated install routine from setting the correct value for this config key.

  • πŸ‡ΊπŸ‡ΈUnited States bluegeek9

    @alex.n,

    I created a new issue to document the change for release notes better.

    I have merged the change to dev. Can you confirm the change fixes the issue?

    composer require 'drupal/ldap:4.x-dev@dev'
    

    πŸ› Upgrade bug: 8.x-4.7 to 8.x-4.8 Active

  • Automatically closed - issue fixed for 2 weeks with no activity.

Production build 0.69.0 2024