on.nemtina/persistence
1
0
Fork 0
Code Issues Pull Requests 1 Packages Projects Releases Wiki Activity

#384 Авторизации + получение id пользователя в контроллерах #3

Merged
on.nemtina merged 7 commits from PersistenceClient into master 2024-11-25 17:09:22 +05:00
Conversation 1 Commits 7 Files Changed 25 +449 -277
rs.efremov commented 2024-11-25 10:51:08 +05:00
Collaborator
Copy Link
No description provided.
on.nemtina was assigned by rs.efremov 2024-11-25 10:51:08 +05:00
rs.efremov added 5 commits 2024-11-25 10:51:08 +05:00
on.nemtina approved these changes 2024-11-25 12:33:45 +05:00
on.nemtina left a comment
Owner
Copy Link

Еще бы добавить метод генерации токена по логину и паролю

Еще бы добавить метод генерации токена по логину и паролю
Persistence.API/DependencyInjection.cs
@ -63,2 +40,2 @@
}
});
var needUseKeyCloak = configuration.GetSection("NeedUseKeyCloak").Get<bool>();
if (needUseKeyCloak)
on.nemtina commented 2024-11-25 11:15:04 +05:00
Owner
Copy Link

Блок кода с if-else c настройками аутентификации для сваггера желательно убрать в отдельный метод

Блок кода с if-else c настройками аутентификации для сваггера желательно убрать в отдельный метод
👍 1
Persistence.API/DependencyInjection.cs
@ -88,0 +116,4 @@
var needUseKeyCloak = configuration
.GetSection("NeedUseKeyCloak")
.Get<bool>();
if (needUseKeyCloak) services.AddAuthentication(JwtBearerDefaults.AuthenticationScheme)
on.nemtina commented 2024-11-25 11:22:17 +05:00
Owner
Copy Link

Тут тоже как будто хочется 2 отдельных метода использовать: AddAuthenticationKeyCloak и AddAuthenticationJWT

Тут тоже как будто хочется 2 отдельных метода использовать: AddAuthenticationKeyCloak и AddAuthenticationJWT
rs.efremov commented 2024-11-25 14:20:39 +05:00
Author
Collaborator
Copy Link

AddKeycloackSecurity и AddDefaultSecurity. Просто и то и другое по факту bearer jwt

AddKeycloackSecurity и AddDefaultSecurity. Просто и то и другое по факту bearer jwt
Persistence.Client/PersistenceClientFactory.cs Outdated
@ -0,0 +6,4 @@
namespace Persistence.Client
{
public class PersistenceClientFactory
on.nemtina commented 2024-11-25 12:24:20 +05:00
Owner
Copy Link

Желательно добавить комментарий к этим 2-м фабрикам: к PersistenceClientFactory и TestHttpClientFactory

Желательно добавить комментарий к этим 2-м фабрикам: к PersistenceClientFactory и TestHttpClientFactory
👍 1
rs.efremov added 1 commit 2024-11-25 14:29:48 +05:00
on.nemtina added 1 commit 2024-11-25 17:09:10 +05:00
on.nemtina merged commit f5ed62eb7d into master 2024-11-25 17:09:22 +05:00
Sign in to join this conversation.
Reviewers
No reviewers
on.nemtina
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
No Assignees
on.nemtina
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: on.nemtina/persistence#3
No description provided.
Delete Branch "PersistenceClient"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?