fix(sdk): add more caching

[bodinsamuel]

Describe your changes

Fixes NAN-783

Not a fan of caching, however when running a sync it should be deterministic and not spam our own backend for no reason. I believe we should fetch all the data beforehand but in the meantime we can cache more stuff.

• Cache metadata It's only a part of connection, there is no reason for those to be unsync. nb: cache is cleared after updating the metadata

• Cache integration It's only used to get webhook_url however we have spike of usage (we have no way to know if they come from us or outside the backend)

[linear[bot]]

NAN-783 sdk: getMetadata should reuse cached connection

[bodinsamuel]

Tested with

import type { NangoSync } from '../../models';


export default async function fetchData(nango: NangoSync): Promise<void> {
  await nango.log('started');
  const conn = await nango.getConnection();
  await nango.log(conn);

  const meta1 = await nango.getMetadata();
  await nango.log(meta1);

  await nango.setMetadata({ foo: new Date().toISOString() });

  const meta2 = await nango.getMetadata();
  await nango.log(meta2);

  const conn2 = await nango.getConnection();
  await nango.log(conn2);
}

Works as expected and even fix a bug where connection was actually staled (metadata inside the object was not up to date because of the cache)