Address some golint suggestions

- Error strings should not end with a dot.
- No need for `else` when if clause has return statement.
- Exported constants should be documented.
- Some minor changes about variables.

The following glint suggestions have not been implemented.
- Document all methods, or do not export them.
- Error variables should start with Err.

@eapache 
Looking at this again it actually makes more sense I think to move the log line *above* the if statement, and get rid of the duplicate part in the log line *inside* the if statement