Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

bugfix: TrueVault client/api version related errors #23

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

jcrosen
Copy link

@jcrosen jcrosen commented Jan 28, 2021

The usage of the truevault clients was not consistent with the way they
are intended to function, and the current API is incompatible with the
server's older client version. Updates:

  • Bumped the server's truevault client to 1.3.0 to match the react app
  • Updated the react app's usage to pull from the accessToken field as
    the apiKeyOrAccessToken getter is no longer available; this led to all
    requests routed through the local server to fail because it was not
    including the correct token in the header
  • Updated the react app's handling of document search results as it was
    referencing a key that no longer exists

The usage of the truevault clients was not consistent with the way they
are intended to function, and the current API is incompatible with the
server's older client version.  Updates:

* Bumped the server's truevault client to 1.3.0 to match the react app
* Updated the react app's usage to pull from the accessToken field as
  the apiKeyOrAccessToken getter is no longer available; this led to all
  requests routed through the local server to fail because it was not
  including the correct token in the header
* Updated the react app's handling of document search results as it was
  referencing a key that no longer exists
@jcrosen jcrosen added the bug label Jan 28, 2021
@jcrosen jcrosen linked an issue Jan 28, 2021 that may be closed by this pull request
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

can't load admin Dashboard. Invalid Response
1 participant