The 'overwriteRemotePolicyUrl' configuration is not working

[f1ames]

I just took a quick peek into how Api URL overwriting is implemented here, in context of admission controller, and based on the code I see it won't work.

There is const with default URL defined (pointing to dev instance):

https://github.com/kubeshop/vscode-monokle/blob/924c8b68e9b6d31a51cda2924e3509f5ed7d7eb7/src/constants.ts#L9

And then there is global getter using it and overwrite config:

https://github.com/kubeshop/vscode-monokle/blob/924c8b68e9b6d31a51cda2924e3509f5ed7d7eb7/src/utils/globals.ts#L30-L32

The problem is (and the fact that it works now well, querying production Monokle Cloud) is that it is not propagated, and extension uses default synchronizer:

https://github.com/kubeshop/vscode-monokle/blob/924c8b68e9b6d31a51cda2924e3509f5ed7d7eb7/src/utils/synchronization.ts#L24-L25

Which in turn has it's own defaults (pointing to production instance), through ApiHandler util:

• https://github.com/kubeshop/monokle-core/blob/main/packages/synchronizer/src/createDefaultMonokleSynchronizer.ts
• https://github.com/kubeshop/monokle-core/blob/1df8a27699344b9c2ea12a81461019e8ec6369f0/packages/synchronizer/src/handlers/apiHandler.ts#L159
• https://github.com/kubeshop/monokle-core/blob/1df8a27699344b9c2ea12a81461019e8ec6369f0/packages/synchronizer/src/constants.ts#L4

---

To fix this, remoteUrl getter should just return overwrites:

get remotePolicyUrl() { 
  return process.env.MONOKLE_TEST_SERVER_URL ?? this.overwriteRemotePolicyUrl; 
} 

and pass a result to synchronizer initialization (to ApiHandler instance) - if empty string is passed it will still use default production API, if overwrite (or test env var) is used it will use this new URL.

[f1ames]

This needs to be reworked to origin similarly as done in CLI in https://github.com/kubeshop/monokle-cli/issues/23.