Issues with tedious #604

dang-1234 · 2018-01-31T21:46:22Z

I just installed mssql in to my new environment and I see these errors:
It works without errors in my old environment.

Any ideas?

Thanks!

### tedious deprecated The IntN data type is internal and will be removed. node_modules\mssql\lib\tedious.js:63:20
tedious deprecated The BitN data type is internal and will be removed. node_modules\mssql\lib\tedious.js:71:40
tedious deprecated The FloatN data type is internal and will be removed. node_modules\mssql\lib\tedious.js:73:20
tedious deprecated The MoneyN data type is internal and will be removed. node_modules\mssql\lib\tedious.js:78:20
tedious deprecated The NumericN data type is internal and will be removed. node_modules\mssql\lib\tedious.js:82:44
tedious deprecated The DecimalN data type is internal and will be removed. node_modules\mssql\lib\tedious.js:83:44
tedious deprecated The DateTimeN data type is internal and will be removed. node_modules\mssql\lib\tedious.js:85:20
tedious deprecated The TimeN data type alias is deprecated, please use Time instead. node_modules\mssql\lib\tedious.js:88:20
tedious deprecated The DateN data type alias is deprecated, please use Date instead. node_modules\mssql\lib\tedious.js:89:20
tedious deprecated The DateTime2N data type alias is deprecated, please use DateTime2 instead. node_modules\mssql\lib\tedious.js:90:20
tedious deprecated The DateTimeOffsetN data type alias is deprecated, please use DateTimeOffset instead. node_modules\mssql\lib\tedious.js:91:20

The text was updated successfully, but these errors were encountered:

cepm-nate · 2018-02-06T00:19:11Z

I'm seeing this as well. Brand new install of node-mssql.

CrescentCraftorium · 2018-02-06T14:48:41Z

+1 - Version 4.1.0

Edit:

Rolling tedious back to Version 2.2.4 gets rid of these messages. May be an issue with that package.

serkandurusoy · 2018-02-08T16:27:04Z

Yup, and I'm getting

tedious deprecated The `IntN` data type is internal and will be removed. at ../../../../../node_modules/mssql-updated/lib/tedious.js:63:20

coming in from https://github.com/patriksimek/node-mssql/blob/master/lib/tedious.js#L63

deepakbajaj · 2018-02-08T17:20:23Z

Can you suggest how to roll back tedious to suggested version to solve the issue.

CrescentCraftorium · 2018-02-08T17:44:15Z

Either install that specific version with
npm install --save tedious@2.2.4
or edit your package.json to reflect that version and run npm update

"dependencies": {
  "mssql": "^4.1.0",
  "tedious": "2.2.4"
}

I did this as a troubleshooting step though. I don't know if mssql requires the latest version of tedious to operate properly or not, so this may break other features

serkandurusoy · 2018-02-08T17:53:12Z

@WebGaze nice workaroung, though I'd prefer moving forward instead of backward ;)

deepakbajaj · 2018-02-08T18:03:41Z

Thanks for quick reply :)

magnusjt · 2018-02-14T15:09:43Z

Why keep inaccurate dependencies in the first place? ("tedious": "^2.0.0"). I know it's supposed to be backwards compatible, but that's often just in theory.

reksc · 2018-03-09T10:53:42Z

Hi, is there a plan to include aca00f0 into a stable release? I have plenty of messages in my log due to this while using TypeORM.

Switched all the deprecated type to the recommended ones

mottykohn · 2018-03-16T14:40:03Z

Just realized @davepreston also created a fix, but not sure how that PR fixes the problem it only changes the switch from tds to sql. I think both have to be pulled to fully fix the problem.

Fix tediousjs#604

saostad · 2018-04-04T19:11:52Z

What's the solution?
Any Help please?

mottykohn · 2018-04-05T17:14:41Z

@saostad for now you can do the following:
npm install https://github.com/justinwoo/node-mssql.git#fixes
Please note that this is not the official way of doing things but it should work. Maybe https://github.com/justinwoo can help you he is reachable on tweeter as posted on link.

jonstuebe · 2018-04-14T17:28:10Z

@mottykohn that is missing a couple of things. I added the rest of the deprecations in a new pr: #643

jxjj · 2018-04-30T19:50:44Z

If anyone lands here just wanting to silence the deprecation warnings, you can add an environment variable NO_DEPRECATION=tedious, as pointed out in tediousjs/tedious/pull/691.

For example, inpackage.json with ava as a testrunner:

"scripts": {
   "test": "NO_DEPRECATION=tedious ava"
}

luker2 · 2018-05-03T15:06:35Z

Referencing @jonstuebe commit in package.json as described here works for me temporarily.

  "dependencies": {
    "mssql": "git+https://github.com/tediousjs/node-mssql.git#c9209d8b44141561de4fdf20bc43e30c140b5da7",
  }

UPDATE: I actually began receiving error fatal: reference is not a tree during npm install. Didn't dig into it too much, I'm back to defining "tedious": "2.2.4" in package.json

frostbytedata · 2018-05-24T02:12:56Z

So as a dev is it safe for me to suppress these errors for now until an update comes from Knex or Tedious (both)? I feel dirty just sweeping them under the rug.

shawnmgoulet · 2018-06-18T20:36:30Z

+1 for v4.1.0 - @WebGaze workaround throwing it back to v2.2.4 still works, looks like this PR submitted April 14th addresses it, but still hasn't been merged????? 🤔🤔

patriksimek · 2018-07-03T21:08:57Z

Fixed in #643. Sorry for the delay.

ves-krd · 2018-07-07T01:39:26Z

Still not fixed for me...

patriksimek · 2018-07-07T19:08:01Z

@ves-krd What version of node-mssql do you use?

MyPublicGitHubAcct · 2018-09-02T18:43:49Z

I am seeing something very similar today -

tedious deprecated The `IntN` data type is internal and will be removed. node_modules/mssql/lib/tedious.js:63:20
tedious deprecated The `BitN` data type is internal and will be removed. node_modules/mssql/lib/tedious.js:71:40
tedious deprecated The `FloatN` data type is internal and will be removed. node_modules/mssql/lib/tedious.js:73:20
tedious deprecated The `MoneyN` data type is internal and will be removed. node_modules/mssql/lib/tedious.js:78:20
tedious deprecated The `NumericN` data type is internal and will be removed. node_modules/mssql/lib/tedious.js:82:44
tedious deprecated The `DecimalN` data type is internal and will be removed. node_modules/mssql/lib/tedious.js:83:44
tedious deprecated The `DateTimeN` data type is internal and will be removed. node_modules/mssql/lib/tedious.js:85:20
tedious deprecated The `TimeN` data type alias is deprecated, please use `Time` instead. node_modules/mssql/lib/tedious.js:88:20
tedious deprecated The `DateN` data type alias is deprecated, please use `Date` instead. node_modules/mssql/lib/tedious.js:89:20
tedious deprecated The `DateTime2N` data type alias is deprecated, please use `DateTime2` instead. node_modules/mssql/lib/tedious.js:90:20
tedious deprecated The `DateTimeOffsetN` data type alias is deprecated, please use `DateTimeOffset` instead. node_modules/mssql/lib/tedious.js:91:20
tedious deprecated The `UniqueIdentifierN` data type alias is deprecated, please use `UniqueIdentifier` instead. node_modules/mssql/lib/tedious.js:93:20

with the following dependencies...

"dependencies": {
    "express": "^4.16.3",
    "mssql": "^4.1.0",
    "tedious": "^2.6.4"
  },

mottykohn · 2018-09-03T01:20:52Z

@MyPublicGitHubAcct You need version 4.2.0 via npm i mssql@next for this fix.

dang-1234 changed the title ~~Issues ith tedious~~ Issues with tedious Jan 31, 2018

simoneb mentioned this issue Feb 12, 2018

feature: Deprecate data type aliases. tediousjs/tedious#691

Merged

davepreston added a commit to davepreston/node-mssql that referenced this issue Feb 21, 2018

remove deprecated types in tedious Issue tediousjs#604

aca00f0

mottykohn added a commit to mottykohn/node-mssql that referenced this issue Mar 16, 2018

Fix tediousjs#604

7b98782

Switched all the deprecated type to the recommended ones

justinwoo added a commit to justinwoo/node-mssql that referenced this issue Apr 3, 2018

Merge pull request tediousjs#629 from mottykohn/patch-1

4e09e51

Fix tediousjs#604

patriksimek closed this as completed Jul 3, 2018

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Issues with tedious #604

Issues with tedious #604

dang-1234 commented Jan 31, 2018 •

edited

cepm-nate commented Feb 6, 2018

CrescentCraftorium commented Feb 6, 2018 •

edited

serkandurusoy commented Feb 8, 2018

deepakbajaj commented Feb 8, 2018

CrescentCraftorium commented Feb 8, 2018 •

edited

serkandurusoy commented Feb 8, 2018

deepakbajaj commented Feb 8, 2018

magnusjt commented Feb 14, 2018

reksc commented Mar 9, 2018

mottykohn commented Mar 16, 2018

saostad commented Apr 4, 2018

mottykohn commented Apr 5, 2018

jonstuebe commented Apr 14, 2018

jxjj commented Apr 30, 2018

luker2 commented May 3, 2018 •

edited

frostbytedata commented May 24, 2018

shawnmgoulet commented Jun 18, 2018

patriksimek commented Jul 3, 2018

ves-krd commented Jul 7, 2018

patriksimek commented Jul 7, 2018

MyPublicGitHubAcct commented Sep 2, 2018

mottykohn commented Sep 3, 2018

Issues with tedious #604

Issues with tedious #604

Comments

dang-1234 commented Jan 31, 2018 • edited

cepm-nate commented Feb 6, 2018

CrescentCraftorium commented Feb 6, 2018 • edited

serkandurusoy commented Feb 8, 2018

deepakbajaj commented Feb 8, 2018

CrescentCraftorium commented Feb 8, 2018 • edited

serkandurusoy commented Feb 8, 2018

deepakbajaj commented Feb 8, 2018

magnusjt commented Feb 14, 2018

reksc commented Mar 9, 2018

mottykohn commented Mar 16, 2018

saostad commented Apr 4, 2018

mottykohn commented Apr 5, 2018

jonstuebe commented Apr 14, 2018

jxjj commented Apr 30, 2018

luker2 commented May 3, 2018 • edited

frostbytedata commented May 24, 2018

shawnmgoulet commented Jun 18, 2018

patriksimek commented Jul 3, 2018

ves-krd commented Jul 7, 2018

patriksimek commented Jul 7, 2018

MyPublicGitHubAcct commented Sep 2, 2018

mottykohn commented Sep 3, 2018

dang-1234 commented Jan 31, 2018 •

edited

CrescentCraftorium commented Feb 6, 2018 •

edited

CrescentCraftorium commented Feb 8, 2018 •

edited

luker2 commented May 3, 2018 •

edited