You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@shardingsphere.apache.org by GitBox <gi...@apache.org> on 2022/01/08 04:35:43 UTC
[GitHub] [shardingsphere] cicscpmi opened a new issue #14622: on encrypt ，add check function
cicscpmi opened a new issue #14622:
URL: https://github.com/apache/shardingsphere/issues/14622
## Feature Request
The encrypted field cannot support calculation operation. Can you add a check function to check whether the field operated by the current function contains the field to be decrypted?
--
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: notifications-unsubscribe@shardingsphere.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [shardingsphere] cicscpmi commented on issue #14622: on encrypt ，add check function
Posted by GitBox <gi...@apache.org>.
cicscpmi commented on issue #14622:
URL: https://github.com/apache/shardingsphere/issues/14622#issuecomment-1008246315
in EncryptMergedResult,for example:
if the column that the findColumnProjection finds is not a columnProject, such as expressionProject ,findColumnProjection returns Optional.empty() ;
Can you provide a function to find any column when the column findColumnProjection finds is an expressionProject ,and the column expression include any encrypt column.
For example:
in table t_users include id,name,pwd column,the pwd is a encrypt column.
when sql is select id ,name ,CONCAT(name,pwd) as val from t_user;
getValue(val) find this column(val) is not a columnProject . and the express is "CONCAT(name,pwd)" include pwd is a encrypt. Can you provide a function to check this ?
--
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: notifications-unsubscribe@shardingsphere.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [shardingsphere] strongduanmu commented on issue #14622: on encrypt ，add check function
Posted by GitBox <gi...@apache.org>.
strongduanmu commented on issue #14622:
URL: https://github.com/apache/shardingsphere/issues/14622#issuecomment-1017004517
> in EncryptMergedResult,for example: if the column that the findColumnProjection finds is not a columnProject, such as expressionProject ,findColumnProjection returns Optional.empty() ; Can you provide a function to find any column when the column findColumnProjection finds is an expressionProject ,and the column expression include any encrypt column. For example: in table t_users include id,name,pwd column,the pwd is a encrypt column. when sql is select id ,name ,CONCAT(name,pwd) as val from t_user; getValue(val) find this column(val) is not a columnProject . and the express is "CONCAT(name,pwd)" include pwd is a encrypt. Can you provide a function to check this ?
@cicscpmi Sorry for later reply. Even we add this check logic for expression which contains encrypt column, but the result is still not correct, because we can't distinguish encrypt value from expression value.
--
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: notifications-unsubscribe@shardingsphere.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [shardingsphere] tuichenchuxin commented on issue #14622: on encrypt ，add check function
Posted by GitBox <gi...@apache.org>.
tuichenchuxin commented on issue #14622:
URL: https://github.com/apache/shardingsphere/issues/14622#issuecomment-1008495423
@cicscpmi Are you willing to submit a pr?
--
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: notifications-unsubscribe@shardingsphere.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [shardingsphere] cheese8 commented on issue #14622: on encrypt ，add check function
Posted by GitBox <gi...@apache.org>.
cheese8 commented on issue #14622:
URL: https://github.com/apache/shardingsphere/issues/14622#issuecomment-1010695826
Assign it to me, I will try to enhance it. @strongduanmu @tuichenchuxin
--
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: notifications-unsubscribe@shardingsphere.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [shardingsphere] strongduanmu commented on issue #14622: on encrypt ，add check function
Posted by GitBox <gi...@apache.org>.
strongduanmu commented on issue #14622:
URL: https://github.com/apache/shardingsphere/issues/14622#issuecomment-1007949812
Hi @cicscpmi, can you provide more information to describe your feature request?
--
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: notifications-unsubscribe@shardingsphere.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org