Closed (fixed)
Project:
Drupal driver for SQL Server and SQL Azure
Version:
7.x-1.x-dev
Component:
User interface
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
20 Dec 2011 at 04:07 UTC
Updated:
9 Mar 2013 at 10:50 UTC
Jump to comment: Most recent file
Comments
Comment #1
Akaoni commentedHere's a patch.
Comment #2
Akaoni commentedCNR.
Updated summary.
Comment #2.0
Akaoni commentedUpdate remaining tasks
Comment #3
artusamakWith your patch the connection form to the database won't be required but nothing is implementing the connection through Windows authentication.
The problem is not limited to the installation phase, you potentialy have to connect to the database on every query and just removing required fields is not enough, there is no automatic fallback on windows auth.
I'm not sure that it's compatible with Drupal.
Comment #3.0
Akaoni commentedAdded to remaining tasks
Comment #4
Akaoni commented@Artusamak: Sorry mate, I probably should have put more info in the issue summary about how this works.
By default, the PHP SQLSRV driver uses Windows authentication.
Drupal passes a username and password to the connect function which makes it use SQL Server authentication.
If you use an empty string for username
and passwordhowever, it falls back to Windows authentication.Doco here:
http://msdn.microsoft.com/en-us/library/cc296198.aspx
By making the username not required in the install form
(password is already not required), this setup is stored in settings.php and therefore used for every query run by Drupal.For example, here's the settings in my instance of Drupal 7.10 running this patch and Windows authentication:
I haven't done exhaustive testing, but it works just fine for me. ;)
Edit: I did some more testing and it seems that only username needs to be empty for the driver to fall back to Windows authentication.
Comment #5
damien tournoud commentedSounds reasonable. Merged into 7.x-1.x. Thanks for the patch!
Comment #6.0
(not verified) commentedAdded more info about how this works.