Skip to content

Allow Masquerading as a LTI user.#74

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

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

Conversation

@buckett

@buckett buckett commented Feb 2, 2026

Copy link
Copy Markdown
Member

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

Copilot AI left a comment

Copy link
Copy Markdown

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.

Comment thread src/test/java/edu/ksu/canvas/tests/user/UserRetrieverUTest.java Outdated

@nicholaswilson100 nicholaswilson100 left a comment

Copy link
Copy Markdown

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.

4 participants