BasicLTIUtil.signProperties() does not work for ContentItemSelection messages about basiclti-util-java HOT 2 OPEN

Yeah, I think they should be probably included in BasicLTIConstants.validPropertyNames. I don't think this will break anyone, since if they're creating content-item launches, then they're not using this method anyway...
Reviewing http://www.imsglobal.org/specs/lticiv1p0/specification-3, here's the launch parameters that I think need adding:

Launch to provider:
content_item_return_url
accept_media_types
accept_media_types
accept_presentation_document_targets
accept_unsigned
accept_multiple
accept_copy_advice
title
text
auto_create
data

Launch back to consumer:
content_items
data
lti_log
lti_errormsg
lti_errorlog

Any I'm missing?

from basiclti-util-java.

that looks right to me.

i've since found the sakai/tsugi version https://github.com/sakaiproject/sakai/tree/master/basiclti/tsugi-util that seems to be more actively developed and already does what i need though, so for now i've switched to that.

from basiclti-util-java.