Code review comment for lp:~wallyworld/goose/RunServer-userData-fix

Revision history for this message
Dimiter Naydenov (dimitern) wrote :

On 2013/01/30 04:50:43, wallyworld wrote:
> Please take a look.

NOT LGTM - as discussed, this change is not needed, because the problem
is we shouldn't encode userData in base64, as being []byte this is done
automatically by go (see the docs). I have a CL which just removes the
encoding and will propose it shortly.

https://codereview.appspot.com/7221070/

« Back to merge proposal