Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Dapper 2.1.11 PR#1975 (auto-detect stored procedures as anything without whitespace) regression with non-breaking spaces #1986

Copy link
Copy link
@tlecomte

Description

@tlecomte
Issue body actions

Dear Dapper maintainers

Dapper 2.1.11, and in particular #1975, changes how the command type is determined. It breaks some of our code by unexpectedly inferring "Stored Procedure" for a plain "SELECT * FROM Table". We found that the source code actually contained non-breaking space (ASCII 160) instead the normal space (ASCII 32). That code was working fine in the previous Dapper version 2.1.4.

This is with a connection to SQL Server.

One possibility to solve this could be to use char.iswhitespace instead of the hardcoded list of whitespace characters CommandDefinition.cs#L113 ?

Thanks for your help and the work done in Dapper.

Reactions are currently unavailable

Metadata

Metadata

Assignees

Labels

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions

    Morty Proxy This is a proxified and sanitized view of the page, visit original site.