I can't see the context for this comment.


On Thu, Sep 5, 2013 at 12:40 PM, Alexander Kryklia <notifications@github.com
> wrote:

> In common/lib/xmodule/xmodule/lti_module.py:
>
> > +        # parse headers to pass to template as part of context:
> > +        params = dict([param.strip().replace('"', '').split('=') for param in params.split('",')])
> > +
> > +        params[u'oauth_nonce'] = params[u'OAuth oauth_nonce']
> > +        del params[u'OAuth oauth_nonce']
> > +
> > +        # 0.14.2 (current) version of requests oauth library encodes signature,
> > +        # with 'Content-Type': 'application/x-www-form-urlencoded'
> > +        # so '='' becomes '%3D', but server waits for unencoded signature.
> > +        # Decode signature back:
> > +        params[u'oauth_signature'] = urllib.unquote(params[u'oauth_signature']).decode('utf8')
> > +        return params
> > +
> > +
> > +class LTIModuleDescriptor(LTIFields, MetadataOnlyEditingDescriptor):
> > +    """LTI Descriptor. No export/import to html."""
>
> That was not in requirements. Ask @Lyla-Fischer<https://github.com/Lyla-Fischer>about that please.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/edx/edx-platform/pull/804/files#r6186616>
> .
>