ccsimmons

I believe SQL server requires a comma. I just tried it with a colon and it won’t connect:

[message] => [Microsoft][ODBC Driver 17 for SQL Server]SQL Server Network Interfaces: Connection string is not valid [87].

See Example #3:
https://www.php.net/manual/en/function.sqlsrv-connect.php

It’s the same convention (comma) when you specify a port to connect using Sql Server Management Studio.

• This reply was modified 2 years, 9 months ago by ccsimmons.
• This reply was modified 2 years, 9 months ago by ccsimmons.

Got it. Thanks! I tried to code it the way you did the port check in the MySQL except checking , instead of :

Hopefully I did the pull request correctly.

Thanks

I’d be happy to do a pull request. That github link was a 404 for me though. Found your username in a search but didn’t see the repo.

Hi,

I think I have figured this out…

In the file db.php the following line does not take into account setting a port (external_login_option_db_port) for MSSQL:

$db_instance = sqlsrv_connect( exlog_get_option("external_login_option_db_host"), $connectionOptions);

Would it be possible to get this updated? We run MSSQL on a nonstandard port. Without the port the connection fails.

Thanks.

Were you able to resolve this. I have the same/similar issue.

Did you clear out the external database? When the external database is empty is when I receive a 500 error. Based on the code you linked me to I would have expected either the 600 or 601 or for the html returned to be:

<table>
    <thead>
        <tr>
            <th>username</th>
            <th>password</th>
            <th>first_name</th>
            <th>last_name</th>
            <th>role</th>
            <th>email</th>
        </tr>
    </thead>
    <tbody>
        <tr>
            <td colspan="6">No Users Found.</td>
        </tr>
    </tbody>
</table>

My gut reaction is it would be most secure to enter it in and have no access to view it again (as you stated above). Then users only enter data in the password field if they want to update the password.

I’m not a huge fan of putting plain text passwords in configuration files. It would be nice (and this is just a thought) if a user took their password and encrypted it and then used the encrypted string in the variable definition.

Done. I had 2 minor suggestions for the plugin. I have 2 very minor suggestions for the plugin. I’m going to do two separate posts to describe them to you. thanks again for your assistance.

Looks like it worked! Thanks!!!

Tom,

First off this plugin is really great. It has saved me a ton of work and I am very grateful for it. I am experiencing the same issue though where every successful login results in the user receiving the “Notice of Password Change” email. I’m using the “Disable Local Login” setting so I don’t mind if the password in WordPress is update, however is there any way to suppress triggering that email?

Just sent you a little beer money ??

-Cheers,
Chris

i’m getting this error now, too. i started out with 2.3.1 just a few weeks ago without the error. i installed several plugins tonight and now here it is: “ERROR: The username field is empty.”

anyone?