Add Logging options to IPersistenceConfiguration.

[fredrikhr]

TL;DR

@azure/msal-node-extensions

Adds support for configuring LoggerOptions when creating an IPersistence using PersistenceCreator.createPersistence.

Description

All types implementing IPersistence in the @azure/msal-node-extensions package support a LoggerOptions object as an optional parameter to their static create methods. PersistenceCreator.createPersistence, the factory method creating instances of these types, has currently no way of passing such a LoggerOptions object in to these create methods.

In cases when a custom logger callback is used in MSAL, it is desirable that logging is done consistently across all components being used in MSAL. This PR adds a way for users to customize the logger, allowing for a consistent logging experience.

Implementation

Add a new optional property loggerOptions to the IPersistenceConfiguration interface type. Inside PersistenceCreator.createPersistence this property is simply passed along to all create methods that are called within the factory.

Breaking changes

None. The added property in IPersistenceConfiguration is the only noticeable change to the API surface. If that property is left undefined the previous behavior still applies.

[tnorling]

@fredrikhr Thanks for the PR! Can you please run npm run beachball:change from the root of the repo and commit the changefile it generates? Thanks!

[fredrikhr]

@tnorling Does 4f4cf9e look fine?

[derisen]

LGTM, thanks @fredrikhr -one suggestion

[ghost]

🎉@azure/msal-node-extensions@v1.0.0-alpha.30 has been released which incorporates this pull request.:tada:

Handy links:

• Release Notes
• Official NPM Release