Skip to content
This repository has been archived by the owner on Mar 28, 2023. It is now read-only.

Revert back to using a regular mutex in the database #788

Merged
merged 3 commits into from
Dec 11, 2017
Merged

Revert back to using a regular mutex in the database #788

merged 3 commits into from
Dec 11, 2017

Conversation

cpacia
Copy link
Member

@cpacia cpacia commented Dec 11, 2017

It appears the the RWMutex is not playing well with the sqlcipher encryption
as it causes an error to be thrown stating that the db is encrypted. This
commit reverts back to using a regular mutex.

@cpacia
Revert back to using a regular mutex in the database
7ce8a79 
It appears the the RWMutex is not playing well with the sqlcipher encryption
as it causes an error to be thrown stating that the db is encrypted. This
commit reverts back to using a regular mutex.
@cpacia
Fix lock in tests
dadd265
@cpacia
Gofmt project
23673eb
@coveralls
Copy link

Coverage Status

Coverage remained the same at 26.884% when pulling 23673eb on lock into 216cf91 on master.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 26.884% when pulling 23673eb on lock into 216cf91 on master.

@cpacia cpacia merged commit d195a6f into master Dec 11, 2017
@cpacia cpacia deleted the lock branch December 12, 2017 17:51
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
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.
2 participants
@cpacia @coveralls