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

Documentation missing for v2+ #1

Open
doug-textnow opened this issue Apr 4, 2022 · 1 comment
Open

Documentation missing for v2+ #1

doug-textnow opened this issue Apr 4, 2022 · 1 comment

Comments

@doug-textnow
Copy link

The PerimeterX mobile SDK aar in JFrog is obfuscated and there is no accompanying javadoc or sources aar. This makes it difficult to use. In addition, the android sdk integration guide on the web is written for an older version of the SDK.

The code sample in this repo helps some, but still leaves questions. For example, what is the purpose of the boolean 4th parameter in PerimeterX.INSTANCE.start(application, appId, this, true)?

The following would make the SDK more usable:

  1. Exclude the public API from obfuscation/minification. You want to obfuscate in order to annoy the bad-guys who would try to circumvent what PerimeterX does? Cool, apply obfuscation to the internal secret-sauce. But no need to frustrate legitimate customers in the process. The API surface that's designed to be invoked from customer code should retain descriptive parameter names, comments & docs.

  2. Add javadoc aars to the JFrog repository. Being able to hover over a method name in the IDE and see what parameters are required and what the do really lowers the barrier to integration.

@ankitpunchh
Copy link

4th parameter is to enable/disable doctor check "enableDoctorCheck "

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

No branches or pull requests

2 participants