Await safety uncertain in telemetry/src/ingest-slog-entrypoint.js

[erights]

agoric-sdk/packages/telemetry/src/ingest-slog-entrypoint.js

Line 96 in a1dedea

await stats(update);

The triage at #6219 currently classifies this as

  // TODO FIXME This code should be refactored to make this
  // await checkably safe, or to remove it, or to record here
  // why it is actually safe.
  //
  // It is nested in a conditional with no
  // balancing await on the other branch. Likely this is low risk
  // because it is only in telemetry. The integrity of critical computation
  // should not be endangered by a bug in telemetry in any case.

Git blame shows @michaelfig as the one who should probably investigate this, so I'm assigning to them. Feel free to reassign as appropriate of course.

As the comment above indicates, this is likely low risk so I'm omitting the "security" label.