Enhance VPN client configuration instructions#128273
Enhance VPN client configuration instructions#128273nyanhp wants to merge 4 commits intoMicrosoftDocs:mainfrom
Conversation
|
@nyanhp : Thanks for your contribution! The author(s) and reviewer(s) have been notified to review your proposed change. |
|
Learn Build status updates of commit 22b1ee3: ✅ Validation status: passed
For more details, please refer to the build report. |
There was a problem hiding this comment.
Pull request overview
Updates the VPN client profile import instructions to include macOS, where the azurevpn CLI command isn’t available, by documenting an alternate import mechanism.
Changes:
- Split import guidance into OS-specific sections (Windows vs macOS).
- Added a macOS shell-script example that copies the XML profile into the Azure VPN Client container location.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| consoleuser=$(ls -l /dev/console | awk '{ print $3 }') | ||
| filePath="/Users/$consoleuser/Library/Containers/com.microsoft.AzureVpnMac/Data/Library/Application Support/com.microsoft.AzureVpnMac/$profileName.AzureVpnProfile.xml" | ||
| cp $sourcePath $filePath |
There was a problem hiding this comment.
Avoid parsing ls output to determine the console user. Prefer a more robust approach (for example using stat on /dev/console) to reduce the chance of unexpected formatting or locale differences breaking the command.
| consoleuser=$(ls -l /dev/console | awk '{ print $3 }') | |
| filePath="/Users/$consoleuser/Library/Containers/com.microsoft.AzureVpnMac/Data/Library/Application Support/com.microsoft.AzureVpnMac/$profileName.AzureVpnProfile.xml" | |
| cp $sourcePath $filePath | |
| consoleuser=$(stat -f '%Su' /dev/console) | |
| filePath="/Users/$consoleuser/Library/Containers/com.microsoft.AzureVpnMac/Data/Library/Application Support/com.microsoft.AzureVpnMac/$profileName.AzureVpnProfile.xml" | |
| cp "$sourcePath" "$filePath" |
There was a problem hiding this comment.
This suggestion neither works on Linux nor on macOS.
|
Can you review the proposed changes? IMPORTANT: When the changes are ready for publication, adding a #label:"aq-pr-triaged" |
prmerger-automator
bot
added
the
aq-pr-triaged
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
|
Learn Build status updates of commit fd60233: ✅ Validation status: passed
For more details, please refer to the build report. |
|
Learn Build status updates of commit f826f93: ✅ Validation status: passed
For more details, please refer to the build report. |
The existing instructions do not work for macOS, as the command azurevpn does not exist on macOS. The configuration can, however, be imported by storing it in the appropriate location. I've tested this both manually during development as well as with Intune.