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

Fixing customSchemaFiles #129

Closed
wants to merge 1 commit into from

Conversation

joshuacox
Copy link

What this PR does / why we need it:

customSchemaFiles were not being placed in the /schemas directory as the bitnami env var LDAP_CUSTOM_SCHEMA_DIR defaults to. This is nearly identical to the behavior of LDAP_CUSTOM_LDIF_DIR so I altered customSchemaFiles to be similar to customLdifFiles.

Signed-off-by: Josh Cox <josh@webhosting.coop>
Copy link

@SuperQ SuperQ left a comment

Choose a reason for hiding this comment

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

Does this fix things so that you can include both the ldif and the schema file?

Something like this:

customSchemaFiles:
  openssh_lpk.ldif: |-
    dn: cn=openssh-lpk,cn=schema,cn=config
    objectClass: olcSchemaConfig
    cn: openssh-lpk
    olcAttributeTypes: ( 1.3.6.1.4.1.24552.500.1.1.1.13 NAME 'sshPublicKey'
      DESC 'MANDATORY: OpenSSH Public key'
      EQUALITY octetStringMatch
      SYNTAX 1.3.6.1.4.1.1466.115.121.1.40 )
    olcObjectClasses: ( 1.3.6.1.4.1.24552.500.1.1.2.0 NAME 'ldapPublicKey' SUP top AUXILIARY
      DESC 'MANDATORY: OpenSSH LPK objectclass'
      MAY ( sshPublicKey $ uid )
      )
  openssh_ldap.schema: |-
    # octetString SYNTAX
    attributetype ( 1.3.6.1.4.1.24552.500.1.1.1.13 NAME 'sshPublicKey'
      DESC 'MANDATORY: OpenSSH Public key'
      EQUALITY octetStringMatch
      SYNTAX 1.3.6.1.4.1.1466.115.121.1.40 )
    # printableString SYNTAX yes|no
    objectclass ( 1.3.6.1.4.1.24552.500.1.1.2.0 NAME 'ldapPublicKey' SUP top AUXILIARY
      DESC 'MANDATORY: OpenSSH LPK objectclass'
      MUST uid
      MAY sshPublicKey
      )

Currently the ltb-passwd supports sshPublicKey updating, but I haven't been able to get the additional attribute added to users.

Copy link

stale bot commented Jan 9, 2024

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the wontfix This will not be worked on label Jan 9, 2024
@SuperQ
Copy link

SuperQ commented Jan 9, 2024

Not stale.

@stale stale bot removed the wontfix This will not be worked on label Jan 9, 2024
Copy link

stale bot commented Mar 9, 2024

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the wontfix This will not be worked on label Mar 9, 2024
@SuperQ
Copy link

SuperQ commented Mar 10, 2024

Still not stale.

@stale stale bot removed the wontfix This will not be worked on label Mar 10, 2024
Copy link

stale bot commented May 9, 2024

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the wontfix This will not be worked on label May 9, 2024
@SuperQ
Copy link

SuperQ commented May 9, 2024

Still not stale, but, seems like this project may be abandoned.

@stale stale bot removed the wontfix This will not be worked on label May 9, 2024
@joshuacox
Copy link
Author

sorry to have missed your question so long ago. But, I believe you have the essence of what I have done here. I was trying to solve this problem

Copy link

stale bot commented Jul 13, 2024

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the wontfix This will not be worked on label Jul 13, 2024
@SuperQ
Copy link

SuperQ commented Jul 13, 2024

Still not stale.

@stale stale bot removed the wontfix This will not be worked on label Jul 13, 2024
Copy link

stale bot commented Sep 11, 2024

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the wontfix This will not be worked on label Sep 11, 2024
@stale stale bot closed this Sep 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
wontfix This will not be worked on
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants