fixes

dotnet · Jul 16, 2024 · d2d5ff1 · d2d5ff1
1 parent aeaf5bd
commit d2d5ff1
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 2 deletions.
diff --git a/docs/azure/sdk/protocol-convenience-methods.md b/docs/azure/sdk/protocol-convenience-methods.md
@@ -107,6 +107,7 @@ The preceding code demonstrates the following `System.ClientModel` protocol meth
 
 > [!NOTE]
 > The preceding code configures the `ClientErrorBehaviors.NoThrow` for the `RequestOptions`. This option prevents non-success service responses status codes from throwing an exception, which means the app code should manually handle the response status code checks.
+
 ---
 
 ## Protocol and convenience method usage guidance

diff --git a/docs/azure/sdk/snippets/protocol-convenience-methods/AzureCoreProtocol/Program.cs b/docs/azure/sdk/snippets/protocol-convenience-methods/AzureCoreProtocol/Program.cs
@@ -19,5 +19,12 @@
     prompt, 
     new RequestContext() { ErrorOptions = ErrorOptions.NoThrow});
 
+// Any response code other than 200 from Azure ContentSafety AnalyzeText REST API is not considered a success response
+// See REST API details at https://azure-ai-content-safety-api-docs.developer.azure-api.net/api-details#api=content-safety-service-2023-10-01&operation=TextOperations_AnalyzeText
+if (response.Status != 200) 
+{
+    throw new RequestFailedException(response); 
+}
+
 // Display the results
 Console.WriteLine(response.Content);
diff --git a/docs/azure/sdk/snippets/protocol-convenience-methods/SCMConvenience/Program.cs b/docs/azure/sdk/snippets/protocol-convenience-methods/SCMConvenience/Program.cs
@@ -7,8 +7,8 @@
 ChatClient chatClient = client.GetChatClient("gpt-4");
 
 // Call the convenience method
-ClientResult<ChatCompletion> completion
+ChatCompletion completion
     = chatClient.CompleteChat("What is Azure?");
 
 // Display the results
-Console.WriteLine($"{completion.Value.Role}: {completion.Value.Content}");
+Console.WriteLine($"{completion.Role}: {completion.Content}");