Open
Conversation
Check for unknown session id in get()
- added option param for db file name (seperate from table name)
- updated readme
…econds). bumped nom version and set the sqlite3 dependancy to be the latest master branch on github
Clean out expired sessions from database regularly
Add support for Express 4.x
User renamed to MapBox... This might fix the error on Cloud9IDE where this package cannot be installed.
Update package.json
added touch protocol
Reference latest packaged sqlite instead of last revision
There's no reason to force the file extension. It should be up to the developer to provide the desired file name with or without file extension. Moreover, forcing file extension may prevent sessions to be store in a single database unless the main database file also uses the '.db' file extension.
Remove database file extension
Use WAL mode for the database
Bump sqlite3 to ^4.0.0 for nodeJs v10 and npm v6
Express does not bundle middleware like `session` anymore. Use the express-session middleware instead to avoid errors. Error: Most middleware (like session) is no longer bundled with Express and must be installed separately. Please see https://github.com/senchalabs/connect#middleware.
Updated usage docs for express 4.x
updated to latest sqlite3 plugin
Add missing license file to match the license type specified in the package.json file.
Add missing LICENSE file
Added new option 'createDirIfNotExists'
Implemented SQLiteStore.prototype.all
This design allows the library to remain flexible about database connection management in your application. Also delete the shared cache tests since are not meaningful anymore, furthermore its use is discouraged https://www.sqlite.org/sharedcache.html#use_of_shared_cache_is_discouraged
pass the db connection object into the store object for more flexibility
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
18 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Sqlite use synchronous-mode by default. It's slow and is not necessary every time.
Can you change option parsing to apply some db parameters?
For example, for all keys (other than "dir", "db" and "table") execute
db.run("pragma ? = ?", [#key#, #value#], function (err) {/* ignore error or console.log */});