Cvmcosta / ltijs

Turn your application into a fully integratable LTI 1.3 tool provider.
https://cvmcosta.github.io/ltijs/
Apache License 2.0
299 stars 67 forks source link

Problem when configuring multipke canvas instances #133

Closed yuvalyellin closed 2 years ago

yuvalyellin commented 2 years ago

Since the canvas configuration uses a default url , we can’t configure multiple canvas instances to use the lti if the lti is configured with same clientId (which we don’t control).

Expected behavior Platform unique id should be created which uses more than platform url/clientId

Provider logs not relevant

Ltijs version

NodeJS version

Platform used

Additional context Raised multiple Canvas instances and defined the lti in them as the first tool to verify this really is an issue

Cvmcosta commented 2 years ago

Hello! This is a known issue with Canvas, there's nothing we can do about it really, since this issue only applies to self served versions of Canvas. The instructure hosted versions of Canvas are the only ones that "officially" support LTI since they use multi-tenancy (every registration shares the same URL with different Client IDs). The issue is that self hosted canvas can't sync the client ID so LTI registrations have issues.

bhavintce commented 1 year ago

@yuvalyellin & @Cvmcosta Have you found this issue any solution for multiple self-hosted domain work external app?