You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@ozone.apache.org by GitBox <gi...@apache.org> on 2021/10/07 22:53:49 UTC

[GitHub] [ozone] bharatviswa504 opened a new pull request #2722: HDDS-5823. Change the key permission to 600.

bharatviswa504 opened a new pull request #2722:
URL: https://github.com/apache/ozone/pull/2722


   ## What changes were proposed in this pull request?
   
   Modify the public/private key file permissions to 600.
   
   ## What is the link to the Apache JIRA
   https://issues.apache.org/jira/browse/HDDS-5823
   
   ## How was this patch tested?
   
   Updated tests.
   


-- 
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: issues-unsubscribe@ozone.apache.org

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



---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@ozone.apache.org
For additional commands, e-mail: issues-help@ozone.apache.org


[GitHub] [ozone] bharatviswa504 commented on pull request #2722: HDDS-5823. Change the key permission to 600.

Posted by GitBox <gi...@apache.org>.
bharatviswa504 commented on pull request #2722:
URL: https://github.com/apache/ozone/pull/2722#issuecomment-938997861


   > Thanks @bharatviswa504 for the patch. LGTM, but I have one question: should we apply similar change to `CRLCodec` and `CertificateCodec`?
   
   I think we should be fine to bring this change certificate and CRL, I believe those files does not need execute permission.
   I have opened HDDS-5840 to handle 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: issues-unsubscribe@ozone.apache.org

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



---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@ozone.apache.org
For additional commands, e-mail: issues-help@ozone.apache.org


[GitHub] [ozone] bharatviswa504 merged pull request #2722: HDDS-5823. Change the key permission to 600.

Posted by GitBox <gi...@apache.org>.
bharatviswa504 merged pull request #2722:
URL: https://github.com/apache/ozone/pull/2722


   


-- 
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: issues-unsubscribe@ozone.apache.org

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



---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@ozone.apache.org
For additional commands, e-mail: issues-help@ozone.apache.org


[GitHub] [ozone] bharatviswa504 commented on pull request #2722: HDDS-5823. Change the key permission to 600.

Posted by GitBox <gi...@apache.org>.
bharatviswa504 commented on pull request #2722:
URL: https://github.com/apache/ozone/pull/2722#issuecomment-940180679


   Thank You @adoroszlai for the review


-- 
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: issues-unsubscribe@ozone.apache.org

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



---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@ozone.apache.org
For additional commands, e-mail: issues-help@ozone.apache.org