fix(auth): fix ldap auth cache by creating authClient on construct in…

[kopax]

Inspired from:

• fix(pack): now ignoring file useless for dist #40 (comment)
• Plugin crashes verdaccio #39 (comment)

Test: OK

How to test

Use the test release version on npm:

$ npm install -g [email protected]

You will need at least to rename your ldap config in your config.yaml:

auth:
  # Note: to people who want to test it
  # this MUST be the suffix of the module so here it is ldap-node-cache instead of ldap 
 verdaccio-ldap-fixed-cache:
    type: ldap
    client_options:
      cache: true

It is also possible to test with Docker:

FROM verdaccio/verdaccio:3
RUN yarn add [email protected]

Related resources

• Verdaccio generates lots of authentication requests to LDAP #37

[kopax]

I myself try to test it but unfortunately, I could not log in at all. Can anyone test this? It's been a long time since we are not using this plugin because of its unusable state due to caching absence.

Again, I am now disabling LDAP entirely and I hope contributors here can take a hand.
The code should be working the same for both fixed (#40 and #41), it needs tester.

[Alexandre-io]

Thank you for your help, #42 seems to correct the issue.