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

SqlAsyncCollector.GetExtraProperties doesn't obey case-insensitive collation #1147

Open
ghanover opened this issue Nov 14, 2024 · 0 comments
Labels
bug Something isn't working output-binding

Comments

@ghanover
Copy link

  • Azure Functions SQL Extension or Extension Bundle Version:
    3.1.169-preview, but looking at code in github appears to be an issue with latest
  • Is this a deployed or local function:
    deployed to azure function app
  • What type of Database are you using? (Run SELECT @@VERSION as Version, SERVERPROPERTY('EngineEdition') as EngineEdition on your database)
    Microsoft SQL Azure (RTM) - 12.0.2000.8 Oct 2 2024 11:51:41 Copyright (C) 2022 Microsoft Corporation 5
  • List any custom settings for your function app. This could be a custom time-out defined specifically for your database server or optional configuration that can be customized for the app defined here.

Steps to Reproduce:

  1. create table with columns in uppercase
  2. access table with class properties in lowercase

Our app was previously .net6, using a very old SQL extension (0.1.483-preview). When we upgraded the app to .net8, we also upgraded the sql extension to 3.1.169-preview. Since then, we run into the error generated from GetExtraProperties

Default collation is SQL_Latin1_General_CP1_CI_AS, which is case insensitive, so mismatch case on column names should not be an issue.

@chlafreniere chlafreniere added bug Something isn't working output-binding labels Nov 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working output-binding
Projects
None yet
Development

No branches or pull requests

2 participants