Skip to content

Unexport LdapImpl.Config and expose via getter method#84

Merged
bosvos merged 1 commit intoldap-tests-asg-5028from
copilot/sub-pr-83
Mar 7, 2026
Merged

Unexport LdapImpl.Config and expose via getter method#84
bosvos merged 1 commit intoldap-tests-asg-5028from
copilot/sub-pr-83

Conversation

Copy link

Copilot AI commented Mar 7, 2026

LdapImpl.Config was exported, unintentionally widening the public API and allowing direct struct mutation. This replaces the exported field with an unexported config and a GetConfig() *ConfigLDAP getter.

Changes

  • ldap.go: Config *ConfigLDAPconfig *ConfigLDAP with GetConfig() *ConfigLDAP getter; all internal references updated
  • cmd/ldapfetch/main.go: Replaced struct literal initialization with NewAuthenticator_LDAP(ldapConf) constructor
// Before
ldapImpl := authaus.LdapImpl{Config: ldapConf}

// After
ldapImpl := authaus.NewAuthenticator_LDAP(ldapConf)

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Copilot AI changed the title [WIP] Fix LDAP debug logging and command line tool changes Unexport LdapImpl.Config and expose via getter method Mar 7, 2026
@bosvos bosvos marked this pull request as ready for review March 7, 2026 08:57
… tool

Co-authored-by: bosvos <2437699+bosvos@users.noreply.github.com>
@bosvos bosvos force-pushed the copilot/sub-pr-83 branch from 7abdf47 to de361f3 Compare March 7, 2026 09:05
@bosvos bosvos merged commit d9d51ca into ldap-tests-asg-5028 Mar 7, 2026
1 check passed
@bosvos bosvos deleted the copilot/sub-pr-83 branch March 7, 2026 09:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants