TO-DO: Code Review by ADEPT

[bexcitement]

@jgaul

[jgaul]

@bexcitement -- initial feedback:

• .pyc files should be removed, and added to .gitignore
• i don't know if this convention is explicitly written anywhere, but typically modules are lowercase. it would also be good to be explicit that this is Optimizely's SDK, so how about "optimizely_canvas_sdk"?
• you could just put all the code for this in init.py to keep it even simpler.
• per https://github.com/optimizely/styleguides/blob/master/python.md#naming, function names should be all lower case, with underscores: extract_user_context
• you don't really need the overhead of an object (CanvasValidation) for anything. you can just have a single function extractUserContext (using internal helper functions if you like), so then users can just do:

import optimizely_canvas_sdk

user_context = optimizely_canvas_sdk.extract_user_context(signed_request, optimizely_oauth_client_secret)

• we should be sure we don't have code duplication like self.signed_request.split('.') multiple times (i believe i originally wrote that inefficient code--sorry!)
• once we've finalized the function, we'll bundle it up with a setup.py, figure out how to publish it on PyPI, and add unit tests