weizhouapache commented on PR #6812: URL: https://github.com/apache/cloudstack/pull/6812#issuecomment-1478364610
> > > > @BryanMLima With this pr, most account/domain details will not be decrypted. However, there might be some details (secure/hidden configuration) are still encrypted. The database migration process needs to be modified for these encrypted values. > > @weizhouapache I understand, however, the goal of this PR is to decrypt the values that are not in the `Secure/Hidden` category in the migration process, as they are wrongly encrypted. I do not see how decrypting and re-encrypting the value in the `Secure/Hidden` category has any relation to this PR goal, as they are not the problem to be solved. @BryanMLima The migration class gets the encrypted table/column by Encrypt annotation. In this pr, the Encrypt annotation is removed from two classes so the values in these two tables will not be migrated by the migration class. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@cloudstack.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
