You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@cloudstack.apache.org by GitBox <gi...@apache.org> on 2022/11/21 10:45:16 UTC

[GitHub] [cloudstack] weizhouapache commented on pull request #6907: Fix C2S VPN in parallel to S2S VPN

weizhouapache commented on PR #6907:
URL: https://github.com/apache/cloudstack/pull/6907#issuecomment-1321855896

   @GutoVeronezi 
   as you said, PR #5375 changed the structure from 
   `<IP> %any : PSK "<PSK>" `
   to
   `<IP> : PSK "<PSK>" `
   With this PR, the structure is changed to
   ` : PSK "<PSK>" `
   
   
   I have few questions
   - Have you ever tested the result by reverting #5375 ?
   - Since public IP is removed from the configuration in this PR, might it be potential security issue ?
   
   


-- 
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-unsubscribe@cloudstack.apache.org

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