-
Notifications
You must be signed in to change notification settings - Fork 1
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
chore!(sdk): remove v1 support #393
Conversation
- Removes axios and XHR support - all requests now use fetch - Removes `v1` path support for rewrap and public key - Removes `entityObject` - Removes AppIdAuthProvider - Removes `upsert`
If these changes look good, signoff on them with:
If they aren't any good, please remove them with:
|
This reverts commit b78cd0f.
71f3b00
to
ebf24bc
Compare
|
@@ -128,8 +128,23 @@ export default class Client { | |||
ephemeralKeyPair?: CryptoKeyPair, | |||
dpopEnabled = false | |||
) { | |||
const enwrapAuthProvider = (a: AuthProvider): AuthProvider => { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
enwrap - is this a crypto term?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I guess I should have called it proxied or delegated/delegating. I must just have wrapping on the mind
🎁
- Removes axios and XHR support - all requests now use fetch - Removes `v1` path support for rewrap and public key - Removes `entityObject` - Removes AppIdAuthProvider - Removes `upsert` Signed-off-by: Elizabeth Healy <[email protected]>
v1
path support for rewrap and public keyentityObject
upsert