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

Using OAEP cipher mode to encrypt unique id #2656

Open
wants to merge 2 commits into
base: dev
Choose a base branch
from

Conversation

wmathurin
Copy link
Contributor

NB: Falling back on PKCS1 at decryption time to successfully decode unique id encoded with MSDK 12 or older

At this point PKCS1 is only used in fall back scenarios - to decrypt payloads encrypted for older version of MSDK:

  • to decrypt encrypted push notifications, if server generated key was created with PKCS1 cipher mode
  • to decrypt unique id, if it was encrypted with PKCS1 cipher mode

NB: Falling back on PKCS1 at decryption time to successfully decode unique id encoded with MSDK 12 or older

At this point PKCS1 is only used in fall back scenarios - to decrypt payloads encrypted for older version of MSDK:
- to decrypt encrypted push notifications, if server generated key was created with PKCS1 cipher mode
- to decrypt unique id, if it was encrypted with PKCS1 cipher mode
@mobilesdk-bot
Copy link

1 Warning
⚠️ No Lint Results.

Tests results for SalesforceReact

Generated by 🚫 Danger

@mobilesdk-bot
Copy link

1 Warning
⚠️ No Lint Results.

Tests results for MobileSync

Generated by 🚫 Danger

@mobilesdk-bot
Copy link

1 Warning
⚠️ No Lint Results.

Tests results for SalesforceHybrid

Generated by 🚫 Danger

@mobilesdk-bot
Copy link

1 Warning
⚠️ No Lint Results.

Tests results for SmartStore

Generated by 🚫 Danger

@wmathurin
Copy link
Contributor Author

NB: manually tested upgrade - login to app that uses old generateUniqueIdIfNoneStored and then relaunching app with the new generateUniqueIdIfNoneStored and making sure uniqueId was read and written back using new cipher mode.

@mobilesdk-bot
Copy link

1 Warning
⚠️ No Lint Results.

Tests results for SalesforceSDK

Generated by 🚫 Danger

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants