LTI: поддержка протокола LTI 1.3 для интеграции с внешними инструментами.#655
Open
KirillBorisovich wants to merge 28 commits intoInteIIigeNET:masterfrom
Open
LTI: поддержка протокола LTI 1.3 для интеграции с внешними инструментами.#655KirillBorisovich wants to merge 28 commits intoInteIIigeNET:masterfrom
KirillBorisovich wants to merge 28 commits intoInteIIigeNET:masterfrom
Conversation
…askId and ltiTaskId
…porting from an external tool, and also made validating jwt tokens
…pLinkingReturnController, added a check for matching toolId and course.LtiToolId
DedSec256
changed the title
DedSec256
changed the title
DedSec256
reviewed
Mar 7, 2026
HwProj.CoursesService/HwProj.CoursesService.API/Models/HomeworkTaskLtiLaunchData.cs
Show resolved
Hide resolved
DedSec256
reviewed
Mar 7, 2026
HwProj.CoursesService/HwProj.CoursesService.API/Models/HomeworkTaskLtiLaunchData.cs
Show resolved
Hide resolved
DedSec256
reviewed
Mar 7, 2026
HwProj.CoursesService/HwProj.CoursesService.API/Repositories/ITasksRepository.cs
Outdated
Show resolved
Hide resolved
DedSec256
reviewed
Mar 7, 2026
HwProj.CoursesService/HwProj.CoursesService.API/Services/CoursesService.cs
Outdated
Show resolved
Hide resolved
DedSec256
reviewed
Mar 7, 2026
HwProj.CoursesService/HwProj.CoursesService.API/Services/CoursesService.cs
Outdated
Show resolved
Hide resolved
DedSec256
reviewed
Mar 7, 2026
HwProj.CoursesService/HwProj.CoursesService.API/Controllers/TasksController.cs
Outdated
Show resolved
Hide resolved
DedSec256
reviewed
Mar 7, 2026
HwProj.CoursesService/HwProj.CoursesService.API/Controllers/TasksController.cs
Outdated
Show resolved
Hide resolved
DedSec256
reviewed
Mar 7, 2026
DedSec256
reviewed
Mar 7, 2026
DedSec256
reviewed
Mar 7, 2026
DedSec256
reviewed
Mar 7, 2026
HwProj.NotificationsService/HwProj.NotificationsService.API/appsettings.json
Outdated
Show resolved
Hide resolved
DedSec256
reviewed
Mar 7, 2026
HwProj.SolutionsService/HwProj.SolutionsService.API/Services/SolutionsService.cs
Show resolved
Hide resolved
DedSec256
reviewed
Mar 7, 2026
HwProj.APIGateway/HwProj.APIGateway.API/Lti/Configuration/LtiPlatformConfig.cs
Show resolved
Hide resolved
DedSec256
reviewed
Mar 7, 2026
| @@ -0,0 +1,19 @@ | |||
| namespace HwProj.APIGateway.API.Lti.Models; | |||
|
|
|||
| public class LtiPlatformConfig | |||
Contributor
There was a problem hiding this comment.
Suggested change
| public class LtiPlatformConfig | |
| internal class LtiPlatformConfig |
Author
There was a problem hiding this comment.
А нужно ли? Я в конструкторах использую часто где IOptions options. Чтобы сделать внутренним, мне нужно пренести всю логику в сервисы и сделать их (сервисы) все внутренними, нужно ли так делать? Просто больше в проекте нигде так не делается
DedSec256
reviewed
Mar 7, 2026
| public LtiSigningKeyConfig SigningKey { get; set; } | ||
| } | ||
|
|
||
| public class LtiSigningKeyConfig |
Contributor
There was a problem hiding this comment.
Suggested change
| public class LtiSigningKeyConfig | |
| internal class LtiSigningKeyConfig |
DedSec256
reviewed
Mar 7, 2026
| @@ -0,0 +1,14 @@ | |||
| namespace HwProj.APIGateway.API.Lti.Models | |||
| { | |||
| public class LtiToolConfig | |||
Contributor
There was a problem hiding this comment.
Suggested change
| public class LtiToolConfig | |
| internal class LtiToolConfig |
DedSec256
reviewed
Mar 7, 2026
HwProj.APIGateway/HwProj.APIGateway.API/Lti/Models/LtiToolDto.cs
Outdated
Show resolved
Hide resolved
DedSec256
reviewed
Mar 7, 2026
HwProj.APIGateway/HwProj.APIGateway.API/Lti/Models/LtiDeepLinkingContentItem.cs
Outdated
Show resolved
Hide resolved
DedSec256
reviewed
Mar 7, 2026
HwProj.APIGateway/HwProj.APIGateway.API/Lti/Services/LtiToolService.cs
Outdated
Show resolved
Hide resolved
DedSec256
reviewed
Mar 7, 2026
HwProj.APIGateway/HwProj.APIGateway.API/Models/Solutions/UserTaskSolutions.cs
Outdated
Show resolved
Hide resolved
DedSec256
reviewed
Mar 7, 2026
HwProj.APIGateway/HwProj.APIGateway.API/Lti/Services/LtiToolService.cs
Outdated
Show resolved
Hide resolved
DedSec256
reviewed
Mar 7, 2026
HwProj.APIGateway/HwProj.APIGateway.API/Lti/Services/LtiKeyService.cs
Outdated
Show resolved
Hide resolved
DedSec256
reviewed
Mar 7, 2026
HwProj.APIGateway/HwProj.APIGateway.API/Lti/Services/LtiKeyService.cs
Outdated
Show resolved
Hide resolved
DedSec256
reviewed
Mar 7, 2026
Comment on lines
+9
to
+14
| string clientId, | ||
| string toolId, | ||
| string courseId, | ||
| string targetLinkUri, | ||
| string userId, | ||
| string nonce); |
Contributor
There was a problem hiding this comment.
я бы сделал класс с get/init пропертями
Author
There was a problem hiding this comment.
это метод, случайно не дописал public
DedSec256
reviewed
Mar 7, 2026
| } | ||
|
|
||
| string tokenString = form["JWT"]!; | ||
| var handler = new JwtSecurityTokenHandler(); |
Contributor
There was a problem hiding this comment.
точно мы должны его пересоздавать? Можно ли инжектить?
DedSec256
reviewed
Mar 7, 2026
|
|
||
| var resultList = new List<object>(); | ||
|
|
||
| if (unverifiedToken.Payload.TryGetValue(itemsClaimName, out var itemsObject)) |
Contributor
There was a problem hiding this comment.
Действительно ли структура такая жидкая?
DedSec256
reviewed
Mar 7, 2026
| <body> | ||
| <p>Задача выбрана. Возвращаемся в HwProj...</p> | ||
| <script> | ||
| var payload = {responsePayloadJson}; |
DedSec256
reviewed
Mar 7, 2026
Contributor
DedSec256
left a comment
DedSec256
left a comment
There was a problem hiding this comment.
Крутая работа!
Вот основные замечания сейчас
DedSec256
reviewed
Mar 7, 2026
HwProj.APIGateway/HwProj.APIGateway.API/Lti/Controllers/LtiDeepLinkingReturnController.cs
Show resolved
Hide resolved
KirillBorisovich
force-pushed
the
feat/lti-development
branch
from
dccfb9f to
7d2c0fb
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Реализация поддержки протокола LTI 1.3 для интеграции с внешними инструментами.