search

koopjs / koop-provider-marklogic

A Koop Provider that can be used to exposed data in MarkLogic via Esri feature services
https://koopjs.github.io/koop-provider-marklogic/
Other
6 stars 11 forks source link

DEVEXP-206: Now supporting auth via a basic header #208

Closed rjrudin closed 1 year ago

rjrudin commented 1 year ago

Did a fair amount of cleanup in dbClientManager to hopefully make it easier to understand and read.

One key rename is from "req.marklogicUsername" to "req.markLogicClientCacheKey" to make it clear that the purpose of this value is for retrieving a client instance from the cache maintained by dbClientManager.

Added two more configs - one for testing basic auth, and one for testing what happens with basic with when there's an invalid port.

BillFarber commented 1 year ago

This looks good. It's a significant improvement.