Skip to content

Conversation

MaddyDev
Copy link
Contributor

Description

This pull request updates the C# Script (Csx) samples to run on the .NET 8 in-process hosting model.

Changes:

Added the FUNCTIONS_INPROC_NET8_ENABLED flag and set it to "1" in the local.settings.json file for the C# Script samples.
Reasoning:

The .NET 8 in-process model for Azure Functions requires this application setting to be explicitly enabled for C# Script projects. Without this flag, the function host would default to an older execution model.

This change ensures that the samples are up-to-date with the latest Azure Functions runtime capabilities and can be executed and tested correctly on the .NET 8 framework. This aligns the sample's configuration with the recommended setup for running C# Script functions on .NET 8.

Code Changes

  • Unit tests are added, if possible
  • Integration tests are added if the change is modifying existing behavior of one or more of the bindings
  • New or changed code follows the C# style guidelines defined in .editorconfig
  • All changes MUST be backwards compatible and changes to the shared az_func.GlobalState table must be compatible with all prior versions of the extension
  • Use the ILogger instance to log relevant information, especially information useful for debugging or troubleshooting
  • Use async and await for all long-running operations
  • Ensure proper usage and propagation of CancellationToken
  • T-SQL is safe from SQL Injection attacks through the use of SqlParameters and proper escaping/sanitization of input

Dependencies

Documentation

  • Add samples if the change is modifying or adding functionality
  • Update relevant documentation in the docs

@MaddyDev MaddyDev marked this pull request as ready for review July 23, 2025 18:54
@MaddyDev MaddyDev merged commit fceb5d4 into main Jul 23, 2025
2 checks passed
@MaddyDev MaddyDev deleted the maddy/addNet8EnabledFlagInprocSamples branch July 23, 2025 20:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants