| Commit message (Expand) | Author | Files | Lines |
|---|
| 2020-08-19 | Update amalgamation code (#842)•••* Update amalgamation code
* Drop go1.9 and go1.10 | mattn | 3 | -9975/+10880 |
| 2020-07-21 | Travis-CI: set go_import_path (#829)•••Set go_import_path to tell Travis-CI where to checkout the code, so the
Travis build can also work on forks. This is important when building
without Go modules support. | Olivier Mengué | 1 | -0/+2 |
| 2020-07-21 | Travis-CI: add Go 1.14.x, change order of Go versions (#830)•••Changes in Go versions for the Travis-CI builds:
- reverse order of Go versions so the latest ones are checked first
- add Go 1.14.x (latest stable) at the first position
- use 'tip' instead of 'master' (like https://tip.golang.org) and move
it just after 1.14.x | Olivier Mengué | 1 | -5/+6 |
| 2020-06-18 | Update README.md | Yasuhiro Matsumoto | 1 | -1/+3 |
| 2020-06-06 | upgrade amalgamation code (#821) | mattn | 3 | -1844/+3104 |
| 2020-06-05 | Add mock versions of SQLiteDriver and SQLiteConn for +build !cgo (#819)•••My app can use PostgreSQL and – optionally – SQLite. I would like to be
able to compile the app without cgo when SQLite isn't used, as this
removes the need for a C compiler which makes builds easier and faster,
especially for end-users.
In the simple case, this is not a problem go-sqlite3 already provides a
simple non-cgo mock so it compiles and gives a runtime error if you try
to use it anyway.
However, now I'd like to register a function for my SQLite connection to
match a PostgreSQL function like so:
sql.Register("sqlite3_custom", &sqlite3.SQLiteDriver{
ConnectHook: func(conn *sqlite3.SQLiteConn) error {
return conn.RegisterFunc("pow", pow, true); err != nil {
},
})
But this makes it quite hard to keep the same logic since it refers to
types that don't exist with CGO_ENABLED=0. I will need to create a db.go
with `+build !cgo` and db_cgo.go with `+buid cgo` which duplicates all
the logic but with the sqlite hooks. In my case, this actually affects
quite a lot; for example I have a helper function which connects and
runs migrations and whatnot which looks like:
type ConnectOptions struct {
Connect string // Connect string.
Schema []byte // Database schema to create on startup.
Migrate *Migrate
SQLiteHook func(*sqlite3.SQLiteConn) error
}
And I'd have to have two versions of that, too. You could perhaps do
something with interfaces, but because the sql.Register() call above
references the `sqlite3.SQLiteDriver.ConnectHook` struct field that's
not so straightforward (and wrapping stuff in interfaces probably won't
do much for the general clarity either).
This simplifies all of that by providing some common types that may be
used when setting up a SQLite connectin. I renamed the
`SQLiteDriverMock` to `&SQLiteDriver` for this reason. As far as I can
tell in my testing, this has no real downsides (but perhaps I missed
something?)
---
Note: it might also be worth doing something similar for error.go, as I
already have two variants of the below function (one with cgo as below,
and one without cgo which checks just PostgreSQL):
// ErrUnique reports if this error reports a UNIQUE constraint violation.
//
// This is the cgo version which works for PostgreSQL and SQLite.
func ErrUnique(err error) bool {
var sqlErr *sqlite3.Error
if errors.As(err, &sqlErr) && sqlErr.ExtendedCode == sqlite3.ErrConstraintUnique {
return true
}
var pqErr *pq.Error
if errors.As(err, &pqErr) && pqErr.Code == "23505" {
return true
}
return false
}
This is a lot more manageable than the ConnectHook case, but it would be
nicer if it would work without the need for build tags. | Martin Tournoij | 1 | -7/+18 |
| 2020-06-01 | Edited note in README (#817) | turtlemaster19 | 1 | -1/+1 |
| 2020-05-16 | Document requirements for cross compiling from OSX (#804)•••* Document requirements for cross compiling from OSX
Inspiration: https://github.com/mattn/go-sqlite3/issues/384#issuecomment-433584967
* Document cross compilation steps using xgo for MACOSX | Vishnu Mohandas | 1 | -3/+9 |
| 2020-05-14 | Enable all prefixes for named parameters and allow for unused named parameter...•••* Allow unused named parameters
Try to bind all named parameters and ignore those not used.
* Allow "@" and "$" for named parameters
* Add tests for named parameters
Co-authored-by: Guido Berhoerster <guido+go-sqlite3@berhoerster.name> | gber | 2 | -53/+116 |
| 2020-05-14 | Fix #808: remove goconvey (#809)•••* don't need smartystreets/goconvey any longer
* synchronise coveralls | Diego Becciolini | 1 | -2/+4 |
| 2020-05-03 | Fix for early termination of returned rows (#805)•••Once the regex encountered the first instance of a non-match, it would return without processing the rest of the rows in the statement. This change allows it to process the remaining, only setting the sqlite3_result_int to zero then continuing. This worked fine for the example as it only had one item to process. | ShanePerron | 1 | -1/+5 |
| 2020-04-16 | report actual error message if sqlite3_load_extension fails (#800)•••* report actual error message if sqlite3_load_extension fails
* more fixes and test cases
Co-authored-by: Jesse Rittner <jrittner@lutron.com> | rittneje | 5 | -25/+112 |
| 2020-04-16 | Add extension-functions.c info to README (#779) | Paul Bergeron | 1 | -0/+10 |
| 2020-04-16 | Update dependencies | Yasuhiro Matsumoto | 2 | -1/+7 |
| 2020-04-16 | feat: add go modules (#781)•••* feat: add go modules
* feat: upgrade goquery dependency | Karthik K | 2 | -0/+15 |
| 2020-02-27 | chore: remove debug code (#788) | Manfred Touron | 1 | -2/+0 |
| 2020-01-28 | upgrade amalgamation code (#777) | mattn | 3 | -5410/+9036 |
| 2020-01-23 | Update README.md | Yasuhiro Matsumoto | 1 | -0/+2 |
| 2020-01-09 | Fix typo (#763) | Sergey Bronnikov | 1 | -4/+4 |
| 2019-12-17 | fix typo in doc comment (#770) | rittneje | 1 | -3/+3 |
| 2019-12-17 | add SystemErrno to Error (#740)•••* adding SystemErrno to Error, and fixing error logic when open fails
* fix for old versions of libsqlite3 that do not have sqlite3_system_errno defined
* fixing pre-processor logic
| rittneje | 4 | -12/+77 |
| 2019-11-18 | upgrade amalgamation code | Yasuhiro Matsumoto | 3 | -3965/+5309 |
| 2019-11-18 | Add build constraints for non cgo | Yasuhiro Matsumoto | 41 | -40/+69 |
| 2019-10-31 | Improve readability of Backup() | codesoap | 1 | -3/+3 |
| 2019-10-30 | Added financial contributors to the README | Jess | 1 | -0/+31 |
| 2019-10-08 | Update FUNDING.yml | mattn | 1 | -1/+1 |
| 2019-10-07 | Add Go 1.12.x/1.13.x to CI. | Trevor Pounds | 1 | -0/+2 |
| 2019-09-23 | Fix typo in "_locking_mode" DSN handling | Andrii Zavorotnii | 1 | -1/+1 |
| 2019-09-23 | Fix Open() journal mode regression•••[why]
see https://github.com/mattn/go-sqlite3/issues/607
SQLite default journal mode is DELETE, but forcing it on open causes "database is locked"
if other connection exists with WAL mode, for example.
[how]
Don't set DELETE mode if not set in DSN explicitly.
[testing]
Run tests in my project where WAL mode is used.
| Andrii Zavorotnii | 1 | -5/+6 |
| 2019-09-06 | Fix context cancellation racy handling•••[why]
Context cancellation goroutine is not in sync with Next() method lifetime.
It leads to sql.ErrNoRows instead of context.Canceled often (easy to reproduce).
It leads to interruption of next query executed on same connection (harder to reproduce).
[how]
Do query in goroutine, wait when interruption done.
[testing]
Add unit test that reproduces error cases.
| Andrii Zavorotnii | 2 | -37/+147 |
| 2019-08-22 | Fix memory leak in ExpandedSQL (#738)•••Fixes: #733 | Gert-Jan Timmer | 1 | -0/+1 |
| 2019-08-22 | Fixed operator | G.J.R. Timmer | 1 | -2/+2 |
| 2019-08-22 | Fix _auth_* parameter check•••Fixes: #724 | G.J.R. Timmer | 2 | -3/+27 |
| 2019-08-22 | Closes #597 | G.J.R. Timmer | 8 | -9/+623 |
| 2019-08-19 | Issue #651: Fix of typo•••https://github.com/mattn/go-sqlite3/issues/651
| = | 1 | -1/+1 |
| 2019-08-19 | darwin/libsqlite3: Also use the homebrew include path•••When building on darwin with the `libsqlite3` tag, go-sqlite3 adds the
homebrew library path. It does not, however, add the homebrew include
path, which means that the MacOS sqlite3 header is used instead. On
my system, this results in build errors that look like this:
./sqlite3_load_extension.go:25:8: could not determine kind of name for C.sqlite3_enable_load_extension
./sqlite3_load_extension.go:33:8: could not determine kind of name for C.sqlite3_load_extension
Add the homebrew include path as well, so that he header matches the
libraries we're using.
Signed-off-by: George Dunlap <george.dunlap@citrix.com>
| George Dunlap | 1 | -0/+1 |
| 2019-07-16 | upgrade amalgamation code | Yasuhiro Matsumoto | 2 | -3109/+4059 |
| 2019-07-09 | docs: fix typo | Jason Cooke | 1 | -1/+1 |
| 2019-06-19 | Updated Ping to return ErrBadConn•••Added TestOpenContext
Added TestFileCopyTruncate
Added ping to doTestOpen
| MichaelS11 | 3 | -11/+238 |
| 2019-06-17 | Fix typo in readme | auxten | 1 | -1/+1 |
