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

React Face Liveness component in iOS 15.x all browser occurred websocket errror, component version 2.0.11 #4825

Closed
4 tasks done
yoruponder opened this issue Dec 8, 2023 · 9 comments
Assignees
Labels
bug Something isn't working Liveness Service Team An issue which requires a collaboration with a service team

Comments

@yoruponder
Copy link

yoruponder commented Dec 8, 2023

Before creating a new issue, please confirm:

On which framework/platform are you having an issue?

React

Which UI component?

Liveness

How is your app built?

Create React App

What browsers are you seeing the problem on?

Chrome, Safari

Which region are you seeing the problem in?

ap-northeast-1

Please describe your bug.

user can not connect aws websocket, and throw this error, via sentry we collection this information
image

in user side they show this dialog
image

here is sentry tag
image

User-Agent
Mozilla/5.0 (iPhone; CPU iPhone OS 15_1 like Mac OS X) AppleWebKit/605.1.15 (KHTML, like Gecko) Version/15.1 Mobile/15E148 Safari/604.1

image

What's the expected behaviour?

iOS user can use liveness service normally

Help us reproduce the bug!

when use the iOS 15.x device run liveness component in browser may reproduce the error

Code Snippet

<FaceLivenessDetector
          sessionId={createLivenessApiData ? createLivenessApiData.sessionId : ''}
          region="ap-northeast-1"
          onAnalysisComplete={handleAnalysisComplete}
          disableInstructionScreen={disableInit}
          onUserCancel={() => {
            setLoading(true);
            userCancel();
          }}
          onError={(error) => {
            // -------------------- I send component error to sentry write here --------------------
            console.log('<FaceLivenessDetector>', error)

            if(uid){
              Sentry.setTag("uid", uid);
            }
            
            Sentry.setTag("component", "FaceLivenessDetector");
            let sentry_context = {
              uid,
              type: 'component',
              client,
              access_token,
            }
            if(createLivenessApiData){
              sentry_context.session_id = createLivenessApiData ? createLivenessApiData.sessionId : '';
            }
            Sentry.setContext("Custom-error", sentry_context);
            Sentry.captureException(error);
            Sentry.captureMessage(`<FaceLivenessDetector>-${error.state}`);
          }}
       .....
     />

Additional information and screenshots

user devices
image

@github-actions github-actions bot added the pending-triage Issue is pending triage label Dec 8, 2023
@ioanabrooks ioanabrooks added being-investigated and removed pending-triage Issue is pending triage labels Dec 11, 2023
@thaddmt thaddmt added the bug Something isn't working label Jan 8, 2024
@esauerbo esauerbo self-assigned this Feb 7, 2024
@esauerbo
Copy link
Contributor

esauerbo commented Feb 8, 2024

Hi @yoruponder, are you also seeing this error with the latest version of @aws-amplify/ui-react-liveness?

@yoruponder
Copy link
Author

Hi @esauerbo , I found v3 version react-liveness UI had change, maybe some unknow api also been change, I couldn't update react-liveness to v3 by myself, need to contact and confirm with other team member, it's not easy things 😞, Is it v2 version fix this problem?

@yoruponder
Copy link
Author

v3 react-liveness components prop did not have customization Instructions, react-liveness docs also change, and could not find v2 docs, our product need this api to tell user be ready for face check😅😅

@esauerbo
Copy link
Contributor

Hi @yoruponder, thanks for your patience. So we can try to reproduce your error, which exact version of @aws-amplify/ui-react-liveness are you currently using?

Our migration guide has more information on upgrading to the latest version, and if you encounter any issues we're happy to help.

@yoruponder
Copy link
Author

Hi @esauerbo, thank you for response.
I saw sentry it still occur this error, our @aws-amplify/ui-react-liveness version is 2.0.11

image

Their devices totally are iPhone, and error from 861184 change to 1721344

image

Our team has no plan to Upgrading @aws-amplify/ui-react-liveness in the near future 😂

@yoruponder
Copy link
Author

react-liveness
├── @aws-amplify/[email protected] -----------------> 2.0.11
├── @babel/[email protected]
├── @sentry/[email protected]
├── @testing-library/[email protected]
├── @testing-library/[email protected]
├── @testing-library/[email protected]
├── [email protected] -------------------> 5.3.7
├── [email protected]
├── [email protected]
├── [email protected]
├── [email protected]
├── [email protected]
├── [email protected]
└── [email protected]

@reesscot reesscot added the Service Team An issue which requires a collaboration with a service team label Mar 8, 2024
@reesscot
Copy link
Contributor

Hi @yoruponder,
There was a change made on the service side that should have resolved this issue. Can you please let us know if you are still seeing this issue?

@yoruponder
Copy link
Author

Hi @reesscot , In sentry console, It seen like never occur this error again.
Thank you!!

@reesscot
Copy link
Contributor

reesscot commented Apr 3, 2024

@yoruponder So glad to hear that, and thank you for letting us know! I'll close this out now.

@reesscot reesscot closed this as completed Apr 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working Liveness Service Team An issue which requires a collaboration with a service team
Projects
None yet
Development

No branches or pull requests

5 participants