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

sequelize sync #77

Closed
ryanburnette opened this issue Oct 20, 2018 · 4 comments
Closed

sequelize sync #77

ryanburnette opened this issue Oct 20, 2018 · 4 comments

Comments

@ryanburnette
Copy link

ryanburnette commented Oct 20, 2018

The sync snippet isn't working for me.

    throw new ERR_INVALID_ARG_TYPE('superCtor.prototype',
    ^

TypeError [ERR_INVALID_ARG_TYPE]: The "superCtor.prototype" property must be of type Function. Received type undefined
    at Object.inherits (util.js:1350:11)
    at new SequelizeSessionInit (/Users/ryan/Development/app/node_modules/connect-session-sequelize/lib/connect-session-sequelize.js:55:8)
    at Object.<anonymous> (/Users/ryan/Development/app/support/sync.js:16:15)
    at Module._compile (internal/modules/cjs/loader.js:689:30)
    at Object.Module._extensions..js (internal/modules/cjs/loader.js:700:10)
    at Module.load (internal/modules/cjs/loader.js:599:32)
    at tryModuleLoad (internal/modules/cjs/loader.js:538:12)
    at Function.Module._load (internal/modules/cjs/loader.js:530:3)
    at Function.Module.runMain (internal/modules/cjs/loader.js:742:12)
    at startup (internal/bootstrap/node.js:279:19)
    at bootstrapNodeJSCore (internal/bootstrap/node.js:752:3)

node 10.11.0
connect-session-sequelize 5.2.2
sequelize 4.40.0
sqlite3 4.0.2

@ryanburnette
Copy link
Author

Related nodejs/node#4179

@ryanburnette
Copy link
Author

Node 8.12 gives me this error

    throw new TypeError('The super constructor to "inherits" must ' +
    ^

TypeError: The super constructor to "inherits" must have a prototype
    at Object.inherits (util.js:942:11)
    at new SequelizeSessionInit (/Users/ryan/Development/app/node_modules/connect-session-sequelize/lib/connect-session-sequelize.js:55:8)
    at Object.<anonymous> (/Users/ryan/Development/app/support/sync.js:16:15)
    at Module._compile (module.js:653:30)
    at Object.Module._extensions..js (module.js:664:10)
    at Module.load (module.js:566:32)
    at tryModuleLoad (module.js:506:12)
    at Function.Module._load (module.js:498:3)
    at Function.Module.runMain (module.js:694:10)
    at startup (bootstrap_node.js:204:16)
    at bootstrap_node.js:625:3

@mweibel
Copy link
Owner

mweibel commented Oct 26, 2018

What do you mean by sync snippet?

Other than that, I guess we'd need to update to ES6 syntax for classes (and also switch to const/let instead of var .. etc) and drop support for the old node versions which don't support this syntax.
I'd welcome a PR with those changes.

@mweibel
Copy link
Owner

mweibel commented Feb 21, 2019

Closing due to inactivity.

@mweibel mweibel closed this as completed Feb 21, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants