click here for details... Sakai Executive Director Position Search now open
Issue Details (XML | Word | Printable)

Key: BLTI-24
Type: Bug Bug
Status: Resolved Resolved
Resolution: Fixed
Priority: Major Major
Assignee: Lance Speelmon
Reporter: Lance Speelmon
Votes: 0
Watchers: 0
Operations

If you were logged in you would be able to see more operations.
BasicLTI

Maximum length of a context_id is too constraining - migrate to SHA-1 hash for siteId

Created: 31-Jan-2010 08:39   Updated: 07-Mar-2010 06:54
Component/s: Basiclti-portlet
Affects Version/s: 1.0.0-b01
Fix Version/s: 1.1.0-b01

Time Tracking:
Not Specified

Issue Links:
Relate
 


 Description  « Hide
The current implementation of siteId uses a url safe Base64 algorithm that limits maximum size of the remote context_id to about 72 chars. Since we prepend the lti_key to context_id, this length becomes even more constraining. Move to a fixed length one-way hash algorithm. See: BLTI-23

 All   Comments   Work Log   Change History      Sort Order: Ascending order - Click to sort in descending order
Lance Speelmon added a comment - 31-Jan-2010 08:46
Implemented using a SHA-1 hash of the effective context_id and then stores the original context_id in a site.property "lti_context_id" which will be useful for later reference. Since SHA-1 hashes to 40 chars, that would leave us 59 chars (i.e. 58 + ":") to use for LTI key. This also means that the new maximum supported size of an effective context_id is the maximum message size of SHA-1: maximum length of (264 ? 1) bits.

Committed revision 73035.

Lance Speelmon added a comment - 31-Jan-2010 08:50
Added these comments to class. Committed revision 73036.