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

SDK Token: Remove UUID format from application_id. #116

Conversation

tomw1994991
Copy link
Contributor

@tomw1994991 tomw1994991 commented Sep 4, 2024

Looking at the mobile sdk docs here

The application_id field of sdk_token does not seem to be a UUID:

The application_id is the "Application ID" or "Bundle ID" that was already set up during development.

For iOS this is usually in the form of com.your-company.app-name.
To get this value manually, open xcode ios/YourProjectName, click on the project root, click the General tab, under Targets click your project name, and check the Bundle Identifier field.
To get this value programmatically in native iOS code, see Stack Overflow Page.

For Android this is usually in the form of com.example.yourapp.
To get this file manually, you can find it in your app's build.config. For example, in android/app/build.gradle, it is the value of applicationId.
To get this value programmatically in native Java code, see Stack Overflow Page.

Copy link
Contributor

@dvacca-onfido dvacca-onfido left a comment

Choose a reason for hiding this comment

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

@tomw1994991 thanks for your contribution, LGTM

Given that this is a bug making the endpoint unusable, we'll deliver soon a fix as a minor update.

@dvacca-onfido dvacca-onfido merged commit a33e3b2 into onfido:master Sep 5, 2024
4 checks passed
@tomw1994991 tomw1994991 deleted the remove-uuid-format-from-sdk-token-application-id branch September 6, 2024 07:51
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.

2 participants