Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Documentation must be updated for Isolated Worker function model #527

Open
kevinfoley opened this issue Apr 15, 2024 · 0 comments
Open

Documentation must be updated for Isolated Worker function model #527

kevinfoley opened this issue Apr 15, 2024 · 0 comments
Assignees
Labels
Health Triaged Triaged within product health

Comments

@kevinfoley
Copy link

kevinfoley commented Apr 15, 2024

The article "PlayFab CloudScript using Azure Functions Quickstart Guide" must be updated for the Azure Functions "Isolated Worker" model.

Currently, this page links to the article "Quickstart: Create a C# function in Azure using Visual Studio Code", which has the user create a new Function App that uses the .NET 8 Isolated Worker model. However, the rest of the PlayFab article shows code that only works under the In-Process model. For example,

[FunctionName("HelloWorld")]
        public static async Task<dynamic> Run(
            [HttpTrigger(AuthorizationLevel.Function, "get", "post", Route = null)] HttpRequest req,
            ILogger log)
        {

This code is specifically for the "In-Process" model and does not work under the "Isolated Worker" model.

A user who follows the current article as-is will end up with an invalid Function app that they cannot test.


Document Details

Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

@thomasgu thomasgu added the Health Triaged Triaged within product health label Apr 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Health Triaged Triaged within product health
Projects
None yet
Development

No branches or pull requests

4 participants