search

optimizely / express-middleware

FullStack & Rollouts Express Middleware
Apache License 2.0
2 stars 3 forks source link

fix (client): Update instantiation based on feedback #1

Closed asaschachar closed 4 years ago

asaschachar commented 5 years ago

cc: @jordangarcia

asaschachar commented 5 years ago

@jordangarcia Travis Beck just happened to describe the problem with load balancers as well which I hadn't thought about. Thanks for bringing this up.

One recommendation he had was that the server side SDK could take as some configuration parameter some redis instance which it could connect to and sync the datafile there across server machines. Then the SDK could get synchronized updates across all the clients and be built as a kind of out-of-the-box thing for server-side SDKs. But perhaps this is a bit overkill if the datafile poller gets 80% of the value with much less complexity.

I'm down with reducing the functionality that's exposed via this express wrapper and just focusing on the ergonomic interface as a first experiment to see if this is a type of integration people prefer.