Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Support anonymous launches LTI 1.3 launches. #47

Merged
merged 1 commit into from
Sep 16, 2024

Conversation

buckett
Copy link
Member

@buckett buckett commented Sep 13, 2024

If the user isn't logged into the service providing access to the tool then when the launch is done we don't get a subject claim in the JWT.

This is still a valid launch and it's up to the consuming tool to decide how they want to handle these launches.

If the user isn't logged into the service providing access to the tool then when the launch is done we don't get a subject claim in the JWT.

This is still a valid launch and it's up to the consuming tool to decide how they want to handle these launches.
@nicholaswilson100
Copy link
Contributor

nicholaswilson100 commented Sep 16, 2024

Do we sometimes do different things in different tools with an anonymous launche?

@buckett
Copy link
Member Author

buckett commented Sep 16, 2024

Do we sometimes do something useful in the tool with anonymous launches?

Page deisgn needs to support anonymous launches.

Copy link
Contributor

@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.

Fab

Do we now need to go into separate tools to handle anonymous launches or not needed or already done?

@buckett
Copy link
Member Author

buckett commented Sep 16, 2024

Fab

Do we now need to go into separate tools to handle anonymous launches or not needed or already done?

I don't think any other tools needs to support anonymous launches at the moment.

@buckett buckett merged commit b9ea637 into master Sep 16, 2024
1 check passed
@buckett buckett deleted the anonymous-lti-launches branch September 16, 2024 08:40
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.

2 participants