ariacom / Seal-Report

Database Reporting Tool and Tasks (.Net)
https://sealreport.org
Other
1.43k stars 466 forks source link

OpenId #63

Closed rearguard-hu closed 2 years ago

rearguard-hu commented 2 years ago

Add OpenID Connect support Add some configuration

ariacom commented 2 years ago

Great, thank you, can you provide test cases to validate the code ? How can we test this feature ? Can you also write you comments in english, thank you.

rearguard-hu commented 2 years ago

This has been resolved through the new configuration, and the content has been submitted for merging

---- 回复的原邮件 ---- | 发件人 | Seal @.> | | 日期 | 2022年03月03日 18:09 | | 收件人 | @.> | | 抄送至 | @.**@.> | | 主题 | Re: [ariacom/Seal-Report] OpenId (PR #63) |

I have made some small fix, however it seems thats the previous "Basic Authentication" is not working anymore. Can you check this please ?

— Reply to this email directly, view it on GitHub, or unsubscribe. Triage notifications on the go with GitHub Mobile for iOS or Android. You are receiving this because you authored the thread.Message ID: @.***>

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.