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.

Sign up for GitHub

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

Jump to bottom

fix: must invoke key pragma before do any other interaction with the sqlite #8478

Merged
merged 1 commit into from Jan 15, 2022
Merged

fix: must invoke key pragma before do any other interaction with the sqlite #8478

merged 1 commit into from Jan 15, 2022

Conversation

yolopunk
Copy link
Contributor

@yolopunk yolopunk commented Dec 22, 2021 •
edited

fix: must invoke key pragma before trying to do any other interaction with the sqlite

See documentation: SQLite Encryption Extension

Using the "key" PRAGMA
You must invoke this pragma before trying to do any other interaction with the database.

Closed: #8475

Description of change

Pull-Request Checklist

  • Code is up-to-date with the master branch
  • npm run lint passes with this change
  • npm run test passes with this change
  • This pull request links relevant issues as Fixes #8475
  • There are new or updated unit tests validating the change
  • Documentation has been updated to reflect this change
  • The new commits follow conventions explained in CONTRIBUTING.md

@yolopunk yolopunk force-pushed the better-sqlite3-prepareDatabase branch 2 times, most recently from 0353080 to bfc8f49 Compare December 24, 2021 06:59
@yolopunk yolopunk force-pushed the better-sqlite3-prepareDatabase branch from bfc8f49 to ee2bfe6 Compare December 30, 2021 09:43
@yolopunk yolopunk changed the title fix: adjust the order of prepareDatabase() execution for better-sqlit… fix: must invoke key pragma before do any other interaction with the sqlite Dec 30, 2021
@pleerock pleerock merged commit 546b3ed into typeorm:master Jan 15, 2022
@pleerock
Copy link
Member

Thanks a lot!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

How to set options when using better-sqlite3 with sqlcipher?
2 participants
@yolopunk @pleerock