-
Notifications
You must be signed in to change notification settings - Fork 267
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
feat(liveness): add credentialProvider prop #3790
Conversation
|
packages/react-liveness/src/components/FaceLivenessDetector/service/types/liveness.ts
Outdated
Show resolved
Hide resolved
...-liveness/src/components/FaceLivenessDetector/service/utils/__tests__/streamProvider.test.ts
Outdated
Show resolved
Hide resolved
packages/react-liveness/src/components/FaceLivenessDetector/service/utils/streamProvider.ts
Show resolved
Hide resolved
examples/next/pages/ui/components/liveness/with-credential-provider/index.page.tsx
Outdated
Show resolved
Hide resolved
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.
Looks good to me overall. Thanks for taking this up!
* feat(liveness): add credentialProvier prop * chore: update sdk types * chore: remove extra api definition * update yarn.lock * chore: update credential provider API based on internal review * chore: add unit tests to fix coverage * chore: address feedback * chore: add comment on why credentials are copied, fix unit tests * chore: add size limit check for react-liveness * fix: reusable unit test size check * chore: update types to better match AWS SDK types, export them as well * Update packages/react-liveness/src/components/FaceLivenessDetector/service/types/liveness.ts Co-authored-by: Caleb Pollman <cpollman@amazon.com> * Update packages/react-liveness/src/components/FaceLivenessDetector/service/types/liveness.ts Co-authored-by: Caleb Pollman <cpollman@amazon.com> * chore: switch to using credential-provider@3.310.0 * address feedback * Update packages/e2e/features/ui/components/liveness/with-credential-provider.feature * update to use more recent aws-sdk deps * try fixing deps * update aws-sdk dep to 3.350.0 * maybe this fixes it * another try * fix yarn.lock * fix e2e test --------- Co-authored-by: Caleb Pollman <cpollman@amazon.com> Co-authored-by: William Lee <43682783+wlee221@users.noreply.github.com>
* feat(liveness): add credentialProvider prop (#3790) * feat(liveness): add credentialProvier prop * chore: update sdk types * chore: remove extra api definition * update yarn.lock * chore: update credential provider API based on internal review * chore: add unit tests to fix coverage * chore: address feedback * chore: add comment on why credentials are copied, fix unit tests * chore: add size limit check for react-liveness * fix: reusable unit test size check * chore: update types to better match AWS SDK types, export them as well * Update packages/react-liveness/src/components/FaceLivenessDetector/service/types/liveness.ts Co-authored-by: Caleb Pollman <cpollman@amazon.com> * Update packages/react-liveness/src/components/FaceLivenessDetector/service/types/liveness.ts Co-authored-by: Caleb Pollman <cpollman@amazon.com> * chore: switch to using credential-provider@3.310.0 * address feedback * Update packages/e2e/features/ui/components/liveness/with-credential-provider.feature * update to use more recent aws-sdk deps * try fixing deps * update aws-sdk dep to 3.350.0 * maybe this fixes it * another try * fix yarn.lock * fix e2e test --------- Co-authored-by: Caleb Pollman <cpollman@amazon.com> Co-authored-by: William Lee <43682783+wlee221@users.noreply.github.com> * update yarn.lock * Create pretty-laws-obey.md * update ignored links * fix yarn.lock --------- Co-authored-by: Caleb Pollman <cpollman@amazon.com> Co-authored-by: William Lee <43682783+wlee221@users.noreply.github.com>
* feat(liveness): add credentialProvider prop (#3790) * feat(liveness): add credentialProvier prop * chore: update sdk types * chore: remove extra api definition * update yarn.lock * chore: update credential provider API based on internal review * chore: add unit tests to fix coverage * chore: address feedback * chore: add comment on why credentials are copied, fix unit tests * chore: add size limit check for react-liveness * fix: reusable unit test size check * chore: update types to better match AWS SDK types, export them as well * Update packages/react-liveness/src/components/FaceLivenessDetector/service/types/liveness.ts Co-authored-by: Caleb Pollman <cpollman@amazon.com> * Update packages/react-liveness/src/components/FaceLivenessDetector/service/types/liveness.ts Co-authored-by: Caleb Pollman <cpollman@amazon.com> * chore: switch to using credential-provider@3.310.0 * address feedback * Update packages/e2e/features/ui/components/liveness/with-credential-provider.feature * update to use more recent aws-sdk deps * try fixing deps * update aws-sdk dep to 3.350.0 * maybe this fixes it * another try * fix yarn.lock * fix e2e test --------- Co-authored-by: Caleb Pollman <cpollman@amazon.com> Co-authored-by: William Lee <43682783+wlee221@users.noreply.github.com> * update yarn.lock * Create pretty-laws-obey.md * update ignored links * fix yarn.lock * chore(docs): add liveness credential provider docs --------- Co-authored-by: Caleb Pollman <cpollman@amazon.com> Co-authored-by: William Lee <43682783+wlee221@users.noreply.github.com>
Description of changes
Issue #, if available
Description of how you validated changes
Checklist
yarn test
passes and tests are updated/addedsideEffects
field updatedBy submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.