Skip to content

Conversation

@shu1513
Copy link
Contributor

@shu1513 shu1513 commented Jan 31, 2025

ssl is no longer supported. I actually don't know what is the correct way but deleting "ssl: {}" does make it work.

ssl is no longer supported. I actually don't know what is the correct way but deleting "ssl: {}" does make it work.
@vejol vejol added the part 13 label Feb 7, 2025
@watteja
Copy link
Contributor

watteja commented Mar 13, 2025

A bit earlier in the module, it's mentioned that dialectOptions might be necessary on Heroku specifically.

So I suspect this is an artifact from that environment. I can confirm however that this won't work on Postgres server running in Docker container for example, since SSL is not supported out of the box. There is a way to rebuild the image using certificate-based authentication and OpenSSL, but I really think that's outside of the scope of this module, even for the standards of this course.

So I agree that the line should be left to simple const sequelize = new Sequelize(process.env.DATABASE_URL), especially now that Heroku doesn't offer a free tier and Fly.io doesn't support managed Postgres.

Just bear in mind that you should changed it consistently in all occurrences or none at all.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants