[WIP] Add multi token SDK setup support by pratikpatel351 · Pull Request #15 · gofynd/fdk-extension-javascript

pratikpatel351 · 2021-10-25T12:15:53Z

Users should be able to set up multiple extensions on a single server for the Background worker.
Removed singleton extension object.

Zissan · 2023-11-29T09:36:05Z

express/api_routes.js

        try {
+            const clusterId = req.params.cluster_id;
+            if (clusterId) {
+                extension = ExtensionFactory.getExtension(clusterId)


ExtensionFactory import is missing

express/api_routes.js

Zissan · 2023-11-29T09:38:27Z

express/routes.js

        try {
+            const cluster_id = req.params.cluster_id;
+            if (cluster_id) {
+                ext = ExtensionFactory.getExtension(cluster_id)


ExtensionFactory import is missing

- Specific routes will be matched first based on routing logic of Express so moving cluster id based route as first

Zissan · 2023-11-30T05:06:50Z

express/api_routes.js

+            const clusterId = req.params.cluster_id;
+            if (clusterId) {
+                extension = ExtensionFactory.getExtension(clusterId)
+            }


req.extension = extension;

Injection of extension instance in the request object will be helpful to get the cluster_id

0.1.14

…shot

pratikpatel351 requested a review from brijeshgajjarfynd October 25, 2021 12:15

pratikpatel351 added 3 commits October 28, 2021 12:57

Add multi token sdksetup support

6d53b63

fix

44b45f1

fix.

f773b90

pratikpatel351 force-pushed the multi-extension-support branch from 2437caa to f773b90 Compare October 28, 2021 08:04

Merge branch 'main' into multi-extension-support

959d012

brijeshgajjarfynd changed the base branch from pre-release to main October 8, 2023 13:49

Add cluster id support on extensions

bd68dcc