removing preview and fixing issue with aad (#7042)

Azure · Nov 30, 2023 · 8511b68 · 8511b68
1 parent 4acf92f
commit 8511b68
Show file tree

Hide file tree

Showing 4 changed files with 11 additions and 4 deletions.
diff --git a/src/bastion/HISTORY.rst b/src/bastion/HISTORY.rst
@@ -2,6 +2,11 @@
 
 Release History
 ===============
+0.3.0
+++++++
+* Removing preview flag.
+* Fix for AAD login.
+
 0.2.7
 ++++++
 * add support for auth type password in RDP connection

diff --git a/src/bastion/azext_bastion/azext_metadata.json b/src/bastion/azext_bastion/azext_metadata.json
@@ -1,4 +1,4 @@
 {
- "azext.isPreview": true,
+ "azext.isPreview": false,
  "azext.minCliCoreVersion": "2.43.0"
 }
diff --git a/src/bastion/azext_bastion/custom.py b/src/bastion/azext_bastion/custom.py
@@ -289,8 +289,8 @@ def rdp_bastion_host(cmd, target_resource_id, target_ip_address, resource_group_
  profile = Profile(cli_ctx=cmd.cli_ctx)
  access_token = profile.get_raw_token()[0][2].get("accessToken")
  logger.debug("Response %s", access_token)
- web_address = f"https://{bastion_endpoint}/api/rdpfile?resourceId={target_resource_id}&format=rdp"
- f"&rdpport={resource_port}&enablerdsaad={enable_mfa}"
+ web_address = f"https://{bastion_endpoint}/api/rdpfile?resourceId={target_resource_id}&format=rdp" \
+  f"&rdpport={resource_port}&enablerdsaad={enable_mfa}"
 
  headers = {
  "Authorization": f"Bearer {access_token}",
@@ -310,6 +310,8 @@ def rdp_bastion_host(cmd, target_resource_id, target_ip_address, resource_group_
  rdpfilepath = os.path.join(tempdir, 'conn_{}.rdp'.format(uuid.uuid4().hex))
  _write_to_file(response, rdpfilepath)
 
+ logger.warning("Saving RDP file to: %s", rdpfilepath)
+
  command = [_get_rdp_path()]
  if configure:
  command.append("/edit")

diff --git a/src/bastion/setup.py b/src/bastion/setup.py
@@ -10,7 +10,7 @@
 
 
 # HISTORY.rst entry.
-VERSION = '0.2.7'
+VERSION = '0.3.0'
 
 # The full list of classifiers is available at
 # https://pypi.python.org/pypi?%3Aaction=list_classifiers