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

Random failures when binding binary values #374

Closed
david-garcia-garcia opened this issue Apr 26, 2017 · 6 comments
Closed

Random failures when binding binary values #374

david-garcia-garcia opened this issue Apr 26, 2017 · 6 comments

Comments

@david-garcia-garcia
Copy link

  • Driver Version: 4.1.7-preview
  • OS Version and Distro: Windows Server 2012 R2
  • Can you connect to SQL Server via sqlcmd: YES
  • Environment details: e.g. NTS x64

Still needs investigation, I am seeing (and have received reports) of SQL Server random failing to properly bind binary data (varbinary(max) columns):

https://www.drupal.org/node/2866205

This issue is new, and the codebases have not change, so it must be some sort of regression in any of the latest releases.

Will keep an eye on the issue an try to build a repo script. Created the issue for the record.

@yitam
Copy link
Contributor

yitam commented Apr 26, 2017

Yes we did recently change the way we bind binary data in PDO (see pull request #297 ). Does this happen with varbinary(max) only? A repro script is much appreciated.

@Hadis-Knj
Copy link

@david-garcia-garcia are you able to reproduce this issue in 4.1.8-preview version as well?

@david-garcia-garcia
Copy link
Author

@Hadis-Fard The affected application was running on 4.1.7-preview, after upgrading to 4.1.8-preview the issues has not shown up again (yet). But I'm very reluctant to seeing this as being fixed, as the changelog for 4.1.8 does not seem to include anything related to parameter binding.

@Hadis-Knj
Copy link

@david-garcia-garcia We'll keep this issue open, keep me posted

@Hadis-Knj
Copy link

@david-garcia-garcia do you still face this issue with the GA release?

@meet-bhagdev
Copy link
Contributor

Closing due to inactivity. @david-garcia-garcia feel free to reopen this if you still have questions

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants