ariacom
commented
2 years ago Ok, it is difficult to check the implementation. You should provide a version that works in a basic case.
Thank you.
From: rearguard @.> Sent: jeudi, 3 mars 2022 11:30 To: ariacom/Seal-Report @.> Cc: Subscribed @.***> Subject: [ariacom/Seal-Report] Add whether OpenID Connect is configured (PR #64)
You can enable OpenID Connect using the AppSettings. To test OpenID Connect, you need to configure Authentication in appSettings after building IdentityServer. We need you to review the submission and consider the optimal solution for OpenID Connect. This submission is just a simple implementation.
You can view, comment on, or merge this pull request online at:
https://github.com/ariacom/Seal-Report/pull/64
Commit Summary
- 1900bd4 https://github.com/ariacom/Seal-Report/pull/64/commits/1900bd4e86e97e8905eda1f0bcd68f6e116d816e 新增 OpenID Connect 启用配置
- a1ac7e4 https://github.com/ariacom/Seal-Report/pull/64/commits/a1ac7e43e95ec680567f5a611d8ac779f640c307 default configuration
File Changes
(6 https://github.com/ariacom/Seal-Report/pull/64/files files)
- M Projects/SealWebServer/Controllers/HomeControllerSWI.cs https://github.com/ariacom/Seal-Report/pull/64/files#diff-becaf7e4437f4689624cbc203db26a43e5ca508681a98a73ff901eaa469d12d8 (16)
- M Projects/SealWebServer/Models/Configuration/Authentication.cs https://github.com/ariacom/Seal-Report/pull/64/files#diff-b33e2b8322f43f17c436c329750a719217e2c9dd1e0bf9e6859ece8ea135c696 (5)
- M Projects/SealWebServer/Startup.cs https://github.com/ariacom/Seal-Report/pull/64/files#diff-e16d5cd60f86b67e10f154f64b14bed759f9a587f9fc5754871f9ddb34cb799b (50)
- M Projects/SealWebServer/Views/Home/Main.cshtml https://github.com/ariacom/Seal-Report/pull/64/files#diff-70b0422800e381820f91e02cd13f820d0984ebc625ad4188354e909938a321de (9)
- M Projects/SealWebServer/appsettings.json https://github.com/ariacom/Seal-Report/pull/64/files#diff-a218dae42dc333419991ac526378bea40e1c2a7fc2430258dbf879e33b2af13c (3)
- M Repository/Security/Security.xml https://github.com/ariacom/Seal-Report/pull/64/files#diff-b407f63fce67f5227134560fcbf6e864c6fe4bc52ab8df12085562e3686af07f (3)
Patch Links:
- https://github.com/ariacom/Seal-Report/pull/64.patch
- https://github.com/ariacom/Seal-Report/pull/64.diff
— Reply to this email directly, view it on GitHub https://github.com/ariacom/Seal-Report/pull/64 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AC674AYG3G477UF67LVF2G3U6CIBTANCNFSM5P2BSR7A . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub . You are receiving this because you are subscribed to this thread. https://github.com/notifications/beacon/AC674A3EKOEZQRDNK4IFOPTU6CIBTA5CNFSM5P2BSR7KYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4RIJIMUA.gif Message ID: @. @.> >
You can enable OpenID Connect using the AppSettings. To test OpenID Connect, you need to configure Authentication in appSettings after building IdentityServer. We need you to review the submission and consider the optimal solution for OpenID Connect. This submission is just a simple implementation.