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 2021/02/02 09:59:55 UTC

[GitHub] [shardingsphere] liuchaoheliang opened a new issue #9270: case: left join .... on .... ,the encrypt column cant not be rewrited ..

liuchaoheliang opened a new issue #9270:
URL: https://github.com/apache/shardingsphere/issues/9270


   version
   5.0.0-alpha
   
   my config 
   spring.shardingsphere.props.query-with-cipher-column=true
   spring.shardingsphere.rules.encrypt.tables.user_info.columns.name.cipherColumn=name_enc
   spring.shardingsphere.rules.encrypt.tables.user_info.columns.name.encryptorName=name_encryptor
   spring.shardingsphere.rules.encrypt.tables.user_info.columns.mobile.cipherColumn=mobile_enc
   spring.shardingsphere.rules.encrypt.tables.user_info.columns.mobile.encryptorName=mobile_encryptor
   spring.shardingsphere.rules.encrypt.tables.user_info.columns.ident.cipherColumn=ident_enc
   spring.shardingsphere.rules.encrypt.tables.user_info.columns.ident.encryptorName=idcard_encryptor
   spring.shardingsphere.rules.encrypt.tables.user_info.columns.value.cipherColumn=value_enc
   spring.shardingsphere.rules.encrypt.tables.user_info.columns.value.encryptorName=addr_encryptor
   spring.shardingsphere.rules.encrypt.tables.test_name.columns.name.cipherColumn=name_enc
   spring.shardingsphere.rules.encrypt.tables.test_name.columns.name.encryptorName=name_encryptor
   spring.shardingsphere.rules.encrypt.tables.users.columns.name.cipherColumn=name_enc
   spring.shardingsphere.rules.encrypt.tables.users.columns.name.encryptorName=name_encryptor
   
   2021-02-02 17:24:11.161  INFO 1832 --- [           main] ShardingSphere-SQL                       : Logic SQL: select ui.*,u.name,tn.name from user_info ui left join  users u on u.id = ui.id left join test_name tn on tn.name = u.name where ui.id > 0  
   2021-02-02 17:24:11.163  INFO 1832 --- [           main] ShardingSphere-SQL                       : Actual SQL: ds0 ::: select `ui`.`id`, `ui`.`cust_id`, `ui`.`unique_id`, `ui`.`name_enc` AS name, `ui`.`name_assisted`, `ui`.`ident_enc` AS ident, `ui`.`mobile_enc` AS mobile, `ui`.`password`, `ui`.`reg_time`, `ui`.`modify_time`, `ui`.`reg_source`, `ui`.`reg_app_channel`, `ui`.`reg_device_info`, `ui`.`status`, `ui`.`is_mobile_authed`, `ui`.`complate_cust_info`, `ui`.`time_inst`, `ui`.`time_upd`, `ui`.`has_checked`, `ui`.`value_enc` AS value, `ui`.`system`,u.name_enc AS name,tn.name_enc AS name from user_info ui left join  users u on u.id = ui.id left join test_name tn on **tn.name = u.name** where ui.id > 0  
   
   
   left join ....  on tn.name = u.name  ,there will be rewrite to tn.name_enc = u.name_enc
   
   
   


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [shardingsphere] tuohai666 closed issue #9270: case: left join .... on .... ,the encrypt column cant not be rewrited ..

Posted by GitBox <gi...@apache.org>.
tuohai666 closed issue #9270:
URL: https://github.com/apache/shardingsphere/issues/9270


   


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [shardingsphere] tuohai666 commented on issue #9270: case: left join .... on .... ,the encrypt column cant not be rewrited ..

Posted by GitBox <gi...@apache.org>.
tuohai666 commented on issue #9270:
URL: https://github.com/apache/shardingsphere/issues/9270#issuecomment-773099905


   Duplicated with https://github.com/apache/shardingsphere/issues/9291 .


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [shardingsphere] jingshanglu commented on issue #9270: case: left join .... on .... ,the encrypt column cant not be rewrited ..

Posted by GitBox <gi...@apache.org>.
jingshanglu commented on issue #9270:
URL: https://github.com/apache/shardingsphere/issues/9270#issuecomment-772171434


   > Hi @liuchaoheliang Thank for feedback, looks like bad support for `join` SQL when encrypting.
   > Hi @jingshanglu Do we parse `left join test_name tn on tn.name = u.name` into `SelectStatement`?
   
   @tristaZero Yes.


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [shardingsphere] tristaZero commented on issue #9270: case: left join .... on .... ,the encrypt column cant not be rewrited ..

Posted by GitBox <gi...@apache.org>.
tristaZero commented on issue #9270:
URL: https://github.com/apache/shardingsphere/issues/9270#issuecomment-771542536


   Hi @liuchaoheliang Thank for feedback, looks like bad support for `join` SQL when encrypting.
   Hi @jingshanglu Do we parse `left join test_name tn on tn.name = u.name` into `SelectStatement`?


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [shardingsphere] liuchaoheliang commented on issue #9270: case: left join .... on .... ,the encrypt column cant not be rewrited ..

Posted by GitBox <gi...@apache.org>.
liuchaoheliang commented on issue #9270:
URL: https://github.com/apache/shardingsphere/issues/9270#issuecomment-772419751


   > Hi @liuchaoheliang ! Thanks for your feedback!
   > We also found this problem and fixed it in the latest version (master branch).~
   > If you want to take advantage of this feature, you can try to install the master branch, or wait for our upcoming release version!
   
   ok ,thanks [!]
   
   
   
   
   
   


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [shardingsphere] tuohai666 closed issue #9270: case: left join .... on .... ,the encrypt column cant not be rewrited ..

Posted by GitBox <gi...@apache.org>.
tuohai666 closed issue #9270:
URL: https://github.com/apache/shardingsphere/issues/9270


   


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [shardingsphere] tuohai666 commented on issue #9270: case: left join .... on .... ,the encrypt column cant not be rewrited ..

Posted by GitBox <gi...@apache.org>.
tuohai666 commented on issue #9270:
URL: https://github.com/apache/shardingsphere/issues/9270#issuecomment-773099905


   Duplicated with https://github.com/apache/shardingsphere/issues/9291 .


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [shardingsphere] tristaZero commented on issue #9270: case: left join .... on .... ,the encrypt column cant not be rewrited ..

Posted by GitBox <gi...@apache.org>.
tristaZero commented on issue #9270:
URL: https://github.com/apache/shardingsphere/issues/9270#issuecomment-771542536


   Hi @liuchaoheliang Thank for feedback, looks like bad support for `join` SQL when encrypting.
   Hi @jingshanglu Do we parse `left join test_name tn on tn.name = u.name` into `SelectStatement`?


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [shardingsphere] tristaZero commented on issue #9270: case: left join .... on .... ,the encrypt column cant not be rewrited ..

Posted by GitBox <gi...@apache.org>.
tristaZero commented on issue #9270:
URL: https://github.com/apache/shardingsphere/issues/9270#issuecomment-772201517


   @jingshanglu Got it.
   
   Hi @JiekerTime , could you simulate his scenario to try to locate the cause? Let us see whether we can fix it easily.


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [shardingsphere] JiekerTime commented on issue #9270: case: left join .... on .... ,the encrypt column cant not be rewrited ..

Posted by GitBox <gi...@apache.org>.
JiekerTime commented on issue #9270:
URL: https://github.com/apache/shardingsphere/issues/9270#issuecomment-772313383


   Hi @liuchaoheliang ! Thanks for your feedback!
   We also found this problem and fixed it in the latest version (master branch).~
   If you want to take advantage of this feature, you can try to install the master branch, or wait for our upcoming release version!


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org