Commit Graph

9 Commits

Author SHA1 Message Date
John Graham-Cumming
a3307c097b Fix small problem that occurred when merging from upstream 2013-11-20 08:53:14 -08:00
Nick Sullivan
aeb6aa26ec Merge branch 'master' of https://github.com/jgrahamc/redoctober
Conflicts:
	src/redoctober/core/core_test.go
	src/redoctober/passvault/passvault_test.go
2013-11-20 08:32:11 -08:00
John Graham-Cumming
ab19d6a5cb First time a second set of eyes has looked at the code
The major work here has been:

1. Clean up some go idioms to make the code more idiomatic and simplify
   functions where possible to minimize number of lines where there was
   unnecessary code.

2. Document as many of the functions and packages as possible.

3. Add a test suite for the padding package.

4. Force 'go fmt' when 'make' is performed.

5. Better handling of errors throughout the code. There were quite a few
   that were ignored, now they are handled and reported on (e.g. if the
   JSON in the password vault is broken it will no longer be ignored, etc.)

6. Changed the names of a number of functions to make them clearer.

The code could now do with someone who knows it well to go through and
document everything clearly for release.
2013-11-19 19:27:19 -08:00
Nick Sullivan
e9a2f77007 Update command line call for redoctober 2013-11-19 17:53:12 -08:00
Nick Sullivan
afda151f08 Add real test cases for passvault. 2013-11-19 14:51:37 -08:00
Nick Sullivan
897abdfb59 Use the correct testing framework and JSON. 2013-11-19 09:56:38 -08:00
John Graham-Cumming
d172058d5d Clean up comments and run 'go gmt' across code 2013-11-19 04:11:38 -08:00
Nick Sullivan
4e1aa78fb8 [redoctober] Initial logic for alpha server 2013-11-18 17:29:05 -08:00
grittygrease
0358ef9e63 Initial commit 2013-10-30 12:35:26 -07:00