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

fix(meeting): fetching captcha details within meeting object #337

Merged
merged 6 commits into from
Jun 28, 2024

Conversation

Kesari3008
Copy link
Contributor

@Kesari3008 Kesari3008 commented Jun 25, 2024

Storing the response for verifyPassword() and fetch details like invalidPassword, failureReason and requiredCaptcha and update the meeting object with these details.
Updated verifyPassword() function to accept captcha as a parameter.
Added a new function to refresh the captcha when needed

JIRA: https://jira-eng-gpk2.cisco.com/jira/browse/SPARK-521834

Tests Performed:

  1. Entered the wrong password several times. Captcha inbox appeared to enter the captcha.
  2. Entered incorrect password with the valid captcha code.
  3. Entered the correct password with the wrong captcha code.
  4. Entered the correct password with a valid captcha code.

Meeting Widget Testing
https://github.com/webex/sdk-component-adapter/assets/65543166/8b6b8ee4-7676-48b2-a122-32a487a7b1d1

Logs
Widgets_Testing_logs.log

Copy link

@Shreyas281299 Shreyas281299 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! Tested locally working fine. Captcha details were populated

Copy link
Contributor

@sreenara sreenara left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we have details of the JIRA that this is fixing as well as the reason why the change is required?
Please see the comments.

}),
}));
} else {
logger.info('MEETING', ID, 'joinMeeting()', 'Password succesfully verified');
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There is a typo in the log. Please fix it.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Addressed

src/MeetingsSDKAdapter.js Show resolved Hide resolved
Copy link
Contributor

@sreenara sreenara left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please add the JIRA link to the description for tracking.

@Kesari3008 Kesari3008 merged commit 34a8a18 into webex:master Jun 28, 2024
arun3528 pushed a commit that referenced this pull request Jun 28, 2024
## [1.112.5](v1.112.4...v1.112.5) (2024-06-28)

### Bug Fixes

* **meeting:** fetching captcha details within meeting object ([#337](#337)) ([34a8a18](34a8a18))
@arun3528
Copy link

🎉 This PR is included in version 1.112.5 🎉

The release is available on:

Your semantic-release bot 📦🚀

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.

4 participants