-
Notifications
You must be signed in to change notification settings - Fork 2
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 #137: Error in after.sql #140
Conversation
Use the original table names from 1.1.0 - mobile_app - mobile_ssl_pinning
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks Ok.
INSERT INTO mus_mobile_app SELECT * FROM mobile_app; | ||
|
||
-- For table certificate | ||
INSERT INTO mus_certificate SELECT * FROM mobile_ssl_pinning; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We have to select columns in a specific order and also insert them into mus_mobile_domain
. Moreover, we have to fill the pem
column.
Co-authored-by: Zdeněk Černý <[email protected]>
Output will look like this: | ||
|
||
``` | ||
DH4v3SCoDRDUAFBD |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The whole section Creating First Admin User
is analog to documentation in PowerAuth-Cloud, but I afraid that users may tend to use this concrete value instead of their randomly generated ones. cc @romanstrobl
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm OK with it.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Use the original table names from 1.1.0