bug: FIDO 2FA with OAuth not working on iOS

Open
#6,749 1 comment 0 reactions 0 assignees View on GitHub

Maintainers usually reply within 1 day

Nobody has claimed this yet.

Assessment

Difficulty
4/5
Estimated time
3-5 days
Newbie friendliness
35/100
Issue type
Bug
Clarity
Needs clarification
Activity status
Stale
Tech stack
github, ios, react-native, typescript

Research direction

No file or test is named in the issue. Start by reproducing GitHub OAuth login with a Yubikey on an iPhone running iOS, then compare the mobile flow with the working browser and desktop flows. Done means the security-key challenge completes successfully instead of immediately showing "Authentication failed".

Written by the indexing model from the issue text.

Description

🐛 bug
Describe the Bug

Hey there,

I've tried to log in via OAuth2 (GitHub) via the mobile app. The login works well on the browser and also on the desktop app. On the mobile however, after getting past the username/password, I end up on the "Use security key" option, which insta-fails with "Authentication failed".

The same flow works on Mattermost, so I'd assume there's some misconfiguration in the Rocket Chat app, not some platform or library limitation per se.

Steps to Reproduce

Erm, try to log in via GitHub SSO with a Yubikey as 2FA.

Expected Behavior

Get prompted for a security key challenge.

Actual Behavior

Instant error message

Rocket.Chat Server Version

7.11.0

Rocket.Chat App Version

4.65.0

Device Name

iPhone 14

OS Version

iOS

Additional Context

No response

Dominant language
TypeScript
Stars
2.4k
Forks
1.5k
Avg merge
1d 15h
Merged PRs (30d)
29

Getting set up

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

More from RocketChat/Rocket.Chat.ReactNative

All issues in RocketChat/Rocket.Chat.ReactNative

Similar issues

More TypeScript issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.