aboutsummaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
...
| * | | Fix buildYasuhiro Matsumoto2018-09-111-1/+1
|/ / / | | | | | | | | | Related on #623
* | | Merge pull request #623 from graf0/feature/#619mattn2018-09-113-0/+102
|\ \ \ | |_|/ |/| | implementation of set_authorizer interface
| * | clearer const formattingGrzegorz Marszałek2018-08-301-8/+13
| | | | | | | | | | | | with comments what const are used for what
| * | implementation of set_authorizer interfaceGrzegorz Marszałek2018-08-253-3/+100
| |/
* | Merge pull request #621 from AndrewMurrell/mastermattn2018-08-241-6/+6
|\ \ | |/ |/| Update go doc comments to refer to correct interfaces.
| * Update go doc comments to refer to correct interfaces.AndrewMurrell2018-08-211-6/+6
|/ | | | | | sql.Driver, sql.Conn, sql.Tx sql.Stmt, and sql.Rows are not interfaces. Updated the comments to refer to the correct interfaces: driver.Driver, driver.Conn, driver.Tx, driver.Stmt, and driver.Rows.
* Ensure that SqliteStmt.closed property is guarded.Collin Van Dyck2018-07-191-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Because the closed property of the SQLiteRows's *SqliteStmt was not guarded, it was causing an issue during context cancellation. https://github.com/segmentio/go-sqlite3/blob/be424d27acde822f080bdcd8a7ae6abd4d7d801e/sqlite3.go#L1785-L1796 When a statement is performing a query(), if it determines that the context has been canceled, it will launch a goroutine that closes the resulting driver.Rows if it's not already completed. If the driver.Rows is not done (and the context has been canceled), it will interrupt the connection and more importantly, perform a rows.Close(). The method rows.Close() guards the closed bool with a sync.Mutex to set it to true. If a reader is reading from the SqliteRow, it will call Next() and that performs this check: https://github.com/segmentio/go-sqlite3/blob/be424d27acde822f080bdcd8a7ae6abd4d7d801e/sqlite3.go#L1915-L1917 Because this is not guarded, a data race ensues, and this was actually caught by the Go race detector recently. I didn't include a test case here because the fix seemed straightforward enough and because race conditions are hard to test for. It's been verified in another program that this fixes the issue. If tests should be provided I'm more than happy to do so.
* Fix typo in readmeIngve Vormestrand2018-07-021-1/+1
|
* fix/306Gert-Jan Timmer2018-06-283-28/+3
| | | | | * ppc * ppc64 * ppc64le
* fix/306Gert-Jan Timmer2018-06-282-0/+28
| | | | | | | Add: -lpthread for ppc / ppc64 Issue: #306 [ci ckip]
* Add: OpenBSDGert-Jan Timmer2018-06-132-0/+2
| | | Fixes #321
* Moved solaris flag from _other.go => _solaris.goGert-Jan Timmer2018-06-132-1/+1
|
* Merge pull request #592 from GJRTimmer/fix/test/coveragemattn2018-06-135-1282/+565
|\ | | | | fix/test/coverage
| * Add Tests for Crypt EncodersGert-Jan Timmer2018-06-122-1/+57
| | | | | | * Increase coverage
| * Rewrite Tests UserAuthGert-Jan Timmer2018-06-123-1281/+508
| | | | | | | | * Removed Goconvey * Fix coveralls.io
* | Merge pull request #596 from mattn/fix/travismattn2018-06-131-3/+1
|\ \ | | | | | | Update Travis-CI
| * | Update Travis-CIGert-Jan Timmer2018-06-121-3/+1
|/ / | | | | | | | | | | Moved `sqlite_vacuum_incr` and `sqlite_vtable` to module job. * `sqlite_vacuum_incr` will only store the required auto vacuum information into the datbase page, and not peform an vacuum therefor this can be added easily to the module job. - `sqlite_vtable` does not require an seperate job because user will include the tag in combination with others.
* | fix/511Gert-Jan Timmer2018-06-121-3/+8
| | | | | | | | | | Closes #511 [skip ci]
* | fix/209Gert-Jan Timmer2018-06-121-1/+19
|/ | | | | Closes #209 [skip ci]
* Use t.Skip for TestUpsert to be compliant with the actual cade base.Yoann Cerda2018-06-121-4/+1
|
* fix upsert log typos.Yoann Cerda2018-06-121-1/+3
|
* bump sqlite3 3.24.0 and add TestUpsert.Yoann Cerda2018-06-124-3469/+5729
|
* Merge pull request #591 from mattn/fix/118mattn2018-06-122-2/+51
|\ | | | | fix/118
| * fix/118Gert-Jan Timmer2018-06-122-2/+51
| | | | | | | | | | | | * Added TestMultiBlobs * Removed dead code Fixes #118
* | Merge pull request #574 from GJRTimmer/update/travismattn2018-06-122-60/+2
|\ \ | |/ |/| Follow official Release Policy
| * Removed Windows Cross-CompileGert-Jan Timmer2018-05-291-40/+0
| | | | | | [ci skip]
| * Removed Golang:1.8Gert-Jan Timmer2018-05-292-20/+2
| | | | | | Updated package for follow official Golang Release Policy.
* | Adds GetFilename (sqlite3_db_filename) to SqliteConnZachary Gramana2018-06-121-0/+11
| |
* | Merge pull request #586 from mattn/feature/userauthmattn2018-06-0817-115/+2955
|\ \ | | | | | | Feature/userauth
| * | Fix: test suite remove created filesGert-Jan Timmer2018-06-051-5/+33
| | |
| * | ADD: User Authentication Password EncodersGert-Jan Timmer2018-06-053-0/+422
| | | | | | | | | Allow user to choose how to encode passwords with connection string overrides of embedded `sqlite_crypt` function.
| * | ADD: sqlite_auth to goconvey test suiteGert-Jan Timmer2018-06-051-0/+2
| | |
| * | Update User Authentication DocumentationGert-Jan Timmer2018-06-051-2/+28
| | | | | | | | | References: #581
| * | Add: goconvey to Travis-CIGert-Jan Timmer2018-06-051-1/+1
| | |
| * | Add inital documentationGert-Jan Timmer2018-06-051-0/+95
| | | | | | | | | References: #581
| * | Update TestUserAuthentication*Gert-Jan Timmer2018-06-042-58/+67
| | | | | | | | | | | | | | | | | | * Update goconvey profile * Divided tests in categories Reference #580
| * | Implemented goconvey for User Authentication TestsGert-Jan Timmer2018-06-046-158/+1066
| | | | | | | | | Reference #580
| * | Add additional testsGert-Jan Timmer2018-06-011-16/+141
| | | | | | | | | Reference: #580
| * | Update User AuthenticationGert-Jan Timmer2018-06-013-47/+207
| | | | | | | | | | | | | | | | | | * Update bindings * Add user authentication sql functions Reference #579
| * | StashGert-Jan Timmer2018-05-314-15/+86
| | | | | | | | | [ci skip]
| * | UPD: TestAuthCreateDatabaseGert-Jan Timmer2018-05-311-9/+10
| | | | | | | | | | | | | | | * Renamed test * Implemented 'exists' test * WIP #580
| * | UPD: User AuthenticationGert-Jan Timmer2018-05-312-8/+29
| | | | | | | | | | | | | | | Implemented table check; only activate User Authentication on a database which has no UA enabled. Closes #582
| * | Fix: Free memoryGert-Jan Timmer2018-05-311-4/+45
| | |
| * | ADD: User authenticationGert-Jan Timmer2018-05-308-4/+821
| | | | | | | | | | | | | | | | | | | | | | | | * User Authentication Implementation * Rename file to conform to fileformat `sqlite3_*_omit.go` * Updated sqlite3-binding.* with new upgrade tool * Add: callbackRetNil required for error type return because of adding `RegisterFunc`s directly on the connection. * Add: TestCreateAuthDatabase
| * | Rewrite Upgrade ToolGert-Jan Timmer2018-05-303-111/+225
| | | | | | | | | | | | | | | | | | * Reformat code * Add download for sqlite-src-* * Add extract for sqlite source * Add auto merge of UserAuth module into Amalgamation
* | | Merge pull request #583 from lucasmrod/bug/#542-nil-byte-slice-to-null-blobmattn2018-05-312-4/+27
|\ \ \ | |/ / |/| | Add nil check in bind and a test
| * | Add zero-length slice testLucas Manuel Rodriguez2018-05-311-0/+4
| | |
| * | Add nil check in bind and a testLucas Manuel Rodriguez2018-05-302-4/+23
|/ /
* | Merge pull request #577 from GJRTimmer/update/docsmattn2018-05-311-0/+11
|\ \ | | | | | | Add: Reference to Spatialite
| * | Add: Reference to SpatialiteGert-Jan Timmer2018-05-301-0/+11
|/ / | | | | | | | | Closes #512 [ci ckip]