* Refactor logic for determining if we should use data plane RBAC to a
common function.
* Support RBAC for gremlin API.
* Refactor to use common function.
* Fix unit tests.
* Move test function inside test scope.
* Minor clean ups.
* Reinstate utf8ToB64 function in case this breaks a corner case.
* Increase Gremlin request timeout from 6min to 1hr
* Increase query MAX_RESULT_SIZE limit from 10k to 100k.
* Run npm format
---------
Co-authored-by: Laurent Nguyen <laurent.nguyen@microsoft.com>
- created `getErrorMessage` function which takes in an error string or any type of error object and returns the correct error message
- replaced `error.message` with `getErrorMessage` since `error` could be a string in some cases
- merged sendNotificationForError.ts with ErrorHandlingUtils.ts
- some minor refactoring
In part 2, I will make the following changes:
- Make `Logger.logError` function take an error message string instead of an error object. This will reduce some redundancy where the `getErrorMessage` function is being called twice (the error object passed by the caller is already an error message).
- Update every `TelemetryProcessor.traceFailure` call to make sure we pass in an error message instead of an error object since we stringify the data we send.
