feat: enable openapi spec swagger link in api register

src/apic-extension/azext_apic_extension/_help.py

@@ -31,4 +31,7 @@
         text: |
           az apic api register -g api-center-test -n contosoeuap --api-location "examples/cli-examples/spec-examples/openai.json" --environment-id public
           az apic api register -g api-center-test -n contosoeuap --api-location "examples/cli-examples/spec-examples/openai.yml" --environment-id public
+      - name: Register api by providing spec url.
+        text: |
+          az apic api register -g api-center-test -n contosoeuap --api-location "https://petstore.swagger.io/v2/swagger.json" --environment-id public
 """

src/apic-extension/azext_apic_extension/custom.py

@@ -162,26 +162,47 @@ def register_apic(cmd, api_location, resource_group, service_name, environment_i
     # Load the JSON file
     if api_location:
 
-        # TODO Future Confirm its a file and not link
-        with open(str(api_location), 'rb') as f:
-            rawdata = f.read()
-            result = chardet.detect(rawdata)
-            encoding = result['encoding']
-
-        # TODO - read other file types later
         value = None
-        if str(api_location).endswith('.yaml') or str(api_location).endswith('.yml'):
-            with open(str(api_location), 'r', encoding=encoding) as f:
-                content = f.read()
-                data = yaml.safe_load(content)
-                if data:
-                    value = content
-        if (str(api_location).endswith('.json')):
-            with open(str(api_location), 'r', encoding=encoding) as f:
-                content = f.read()
-                data = json.loads(content)
-                if data:
-                    value = content
+        # Read the spec content from URL
+        if str(api_location).startswith('https://') or str(api_location).startswith('http://'):
+            try:
+                # Fetch the content from the URL
+                response = requests.get(api_location)
+                # Raise an error for bad status codes
+                response.raise_for_status()
+                # Try to parse the content as JSON
+                try:
+                    data = json.loads(response.content)
+                except json.JSONDecodeError:
+                    try:
+                        # If JSON parsing fails, try to parse as YAML
+                        data = yaml.safe_load(response.content)
+                    except yaml.YAMLError as e:
+                        logger.error("Error parsing data from %s: %s", api_location, e)
+                        data = None
+                # If we could parse the content(json or yaml), convert it to a json format string
+                value = json.dumps(data) if data else None
+            except requests.exceptions.RequestException as e:
+                logger.error("Error fetching data from %s: %s", api_location, e)
+                value = None
+        else:
+            # Confirm its a file and not link
+            with open(str(api_location), 'rb') as f:
+                rawdata = f.read()
+                result = chardet.detect(rawdata)
+                encoding = result['encoding']
+
+            # TODO - read other file types later
+            if str(api_location).endswith('.yaml') or str(api_location).endswith('.yml'):
+                with open(str(api_location), 'r', encoding=encoding) as f:
+                    content = f.read()
+                    data = yaml.safe_load(content)
+                    value = content if data else None
+            if (str(api_location).endswith('.json')):
+                with open(str(api_location), 'r', encoding=encoding) as f:
+                    content = f.read()
+                    data = json.loads(content)
+                    value = content if data else None
 
         # If we could not read the file, return error
         if value is None:

src/apic-extension/azext_apic_extension/tests/latest/recordings/test_register_with_json_spec.yaml

@@ -71,10 +71,8 @@ interactions:
       - CONFIG_NOCACHE
       x-content-type-options:
       - nosniff
-      x-ms-ratelimit-remaining-subscription-global-writes:
-      - '2999'
       x-ms-ratelimit-remaining-subscription-writes:
-      - '199'
+      - '1199'
       x-msedge-ref:
       - 'Ref A: ABABBCBF318942349913A2BAAEEBC50C Ref B: MAA201060516047 Ref C: 2024-10-09T08:19:53Z'
       x-powered-by:
@@ -112,7 +110,7 @@ interactions:
       cache-control:
       - no-cache
       content-length:
-      - '449'
+      - '448'
       content-type:
       - application/json; charset=utf-8
       date:
@@ -131,10 +129,8 @@ interactions:
       - CONFIG_NOCACHE
       x-content-type-options:
       - nosniff
-      x-ms-ratelimit-remaining-subscription-global-writes:
-      - '2999'
       x-ms-ratelimit-remaining-subscription-writes:
-      - '199'
+      - '1199'
       x-msedge-ref:
       - 'Ref A: 517F261485BF4D8FB1E0C2D974274BFC Ref B: MAA201060516039 Ref C: 2024-10-09T08:19:55Z'
       x-powered-by:
@@ -207,10 +203,8 @@ interactions:
       - CONFIG_NOCACHE
       x-content-type-options:
       - nosniff
-      x-ms-ratelimit-remaining-subscription-global-writes:
-      - '2999'
       x-ms-ratelimit-remaining-subscription-writes:
-      - '199'
+      - '1199'
       x-msedge-ref:
       - 'Ref A: 2006204F91514BF494E5D9616CAC87BB Ref B: MAA201060515051 Ref C: 2024-10-09T08:19:56Z'
       x-powered-by:
@@ -675,10 +669,8 @@ interactions:
       - CONFIG_NOCACHE
       x-content-type-options:
       - nosniff
-      x-ms-ratelimit-remaining-subscription-global-writes:
-      - '2999'
       x-ms-ratelimit-remaining-subscription-writes:
-      - '199'
+      - '1199'
       x-msedge-ref:
       - 'Ref A: 9A17F989BD03498E839563259636550F Ref B: MAA201060515053 Ref C: 2024-10-09T08:19:58Z'
       x-powered-by:
@@ -742,8 +734,6 @@ interactions:
       - CONFIG_NOCACHE
       x-content-type-options:
       - nosniff
-      x-ms-ratelimit-remaining-subscription-global-reads:
-      - '3749'
       x-msedge-ref:
       - 'Ref A: 3A6EB45269BE4D76BC76123E5D6A53E0 Ref B: MAA201060513027 Ref C: 2024-10-09T08:20:00Z'
       x-powered-by:
@@ -777,7 +767,7 @@ interactions:
       cache-control:
       - no-cache
       content-length:
-      - '449'
+      - '448'
       content-type:
       - application/json; charset=utf-8
       date:
@@ -1327,10 +1317,8 @@ interactions:
       - CONFIG_NOCACHE
       x-content-type-options:
       - nosniff
-      x-ms-ratelimit-remaining-subscription-global-writes:
-      - '2999'
       x-ms-ratelimit-remaining-subscription-writes:
-      - '199'
+      - '1199'
       x-msedge-ref:
       - 'Ref A: 333EF8440AAA48B78D9BC181AA8F1C74 Ref B: MAA201060516049 Ref C: 2024-10-09T08:20:03Z'
       x-powered-by: