teleport-lib-js
teleport-lib-js copied to clipboard
`use` method is error prone in its current format
The use of use
method is error prone: if you forget to call with wrapped with async/await
the libraries are not loaded properly.
Proposal: remove readPluginDefinitionFromUrl
from it and make it sync. Keep readPluginDefinitionFromUrl
as a standalone method.
Another option would be to have both use
and useAsync
methods for sync and async operations.
i agree with removing the possibility of using use
with urls, but keep it for useAsync
@paulbrie, are the discussed APIs already published?
Not yet. If we all agree on the API updates, we could move forward. @nicutimofte, do you want to take the lead on this?