-
Notifications
You must be signed in to change notification settings - Fork 443
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
Please update documents with the latest config schema #858
Comments
@yianni-ververis the document about new config options is updated in http://tediousjs.github.io/tedious/api-connection.html any feedback to make it better is deeply welcomed 🙏 |
hey, there is no example regarding how a config object should look like. |
@pranaytanniru It's a bit of a shortcoming of the current documentation. Here's an example config that makes use of the {
"server": "localhost",
"authentication": {
"type": "default",
"options": {
"userName": "sa",
"password": "yourStrong(!)Password",
}
},
"options": {
"port": 1433,
"database": "master"
}
} |
I didn't get any error message except I was using Typescript and it turns out that the |
For a full list of the connection configuration options, you can check out the constructor that sets the configuration object under tedious/src/connection.js. E.g.,
|
This is the warning I get by using the examples in the repository.
tedious deprecated The "config.userName" property is deprecated and future tedious versions will no longer support it. Please switch to using the new "config.authentication" property instead. src/server/models/utilities/AzureSQL.js:36:25
Please update documents with the latest config schema that uses "config.authentication"
The text was updated successfully, but these errors were encountered: