Skip to content

Allow Masquerading as a LTI user.#74

Merged
buckett merged 2 commits intooxctl:masterfrom
buckett:act-as-lti
Feb 3, 2026
Merged

Allow Masquerading as a LTI user.#74
buckett merged 2 commits intooxctl:masterfrom
buckett:act-as-lti

Conversation

@buckett
Copy link
Member

@buckett buckett commented Feb 2, 2026

This allows a LTI user to be masqueraded as which is always available.

This allows a LTI user to be masqueraded as which is always available.
Copilot AI review requested due to automatic review settings February 2, 2026 15:46
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR adds support for masquerading as LTI (Learning Tools Interoperability) users in the Canvas API client library, allowing API calls to be made on behalf of LTI 1.3 users.

Changes:

  • Added new readAsLtiUser() and writeAsLtiUser() methods to enable LTI user masquerading
  • Implemented LTI masquerading using the lti_1_3_id: prefix in API calls
  • Added unit tests demonstrating LTI masquerading functionality for both read and write operations

Reviewed changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
src/main/java/edu/ksu/canvas/constants/CanvasConstants.java Added constant MASQUERADE_LTI_USER for LTI user masquerading
src/main/java/edu/ksu/canvas/interfaces/CanvasReader.java Added readAsLtiUser() method to reader interface
src/main/java/edu/ksu/canvas/interfaces/CanvasWriter.java Added writeAsLtiUser() method to writer interface
src/main/java/edu/ksu/canvas/impl/BaseImpl.java Implemented LTI masquerading logic with lti_1_3_id: prefix handling
src/test/java/edu/ksu/canvas/tests/user/UserRetrieverUTest.java Added test for reading as LTI user
src/test/java/edu/ksu/canvas/tests/course/UserManagerUTest.java Added test for writing as LTI user

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link

@nicholaswilson100 nicholaswilson100 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lgtm

@buckett buckett merged commit 666549c into oxctl:master Feb 3, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants