apache / shardingsphere-elasticjob-ui

Administrator console of ElasticJob
https://shardingsphere.apache.org/
Apache License 2.0
162 stars 96 forks source link

feat: add support for Casdoor authentication #174

Closed jakiuncle closed 2 years ago

jakiuncle commented 2 years ago

Use Casdoor for auth.

Features:

Support OIDC, OAuth 2.0, SAML, LDAP With Casbin based authorization management, Casdoor supports ACL, RBAC, ABAC, RESTful accessing control models Front-end and back-end separate architecture, Casdoor supports high concurrency, provides web-based managing UI and have mature solution for springboot project https://casdoor.org/docs/integration/spring-boot Casdoor supports Github, Google, QQ, WeChat third-party applications login, and support the extension of third-party login with plugins. Phone verification code, email verification code and forget password features. Accessing logs auditing and recording. Casdoor supports integration with existing systems using db sync method, users can transition to Casdoor smoothly. Casdoor supports mainstream databases: MySQL, PostgreSQL, SQL Server etc, and support the extension of new database with plugins.

Continue:https://github.com/apache/shardingsphere-elasticjob-ui/pull/151

jakiuncle commented 2 years ago

@TeslaCN plz review

TeslaCN commented 2 years ago

@TeslaCN plz review

Thank you. I'll try this soon.

codecov-commenter commented 2 years ago

Codecov Report

Merging #174 (bc239c7) into master (6f24d8f) will increase coverage by 30.00%. The diff coverage is 30.00%.

@@             Coverage Diff             @@
##           master     #174       +/-   ##
===========================================
+ Coverage        0   30.00%   +30.00%     
===========================================
  Files           0        1        +1     
  Lines           0       10       +10     
===========================================
+ Hits            0        3        +3     
- Misses          0        7        +7     
Impacted Files Coverage Δ
...ticjob-lite-ui-frontend/src/views/login/casdoor.js 30.00% <30.00%> (ø)

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

jakiuncle commented 2 years ago

In my computer,it have't problem. Can you send a viedo ?

jakiuncle commented 2 years ago

https://user-images.githubusercontent.com/88994283/199233647-2ba86003-73f6-4392-b9e3-c096ced5e2df.mp4

jakiuncle commented 2 years ago

If you don't mind, you can add my qq 2509913103 to facilitate our communication

TeslaCN commented 2 years ago

Screencast from 11-02-2022 11:07:37 AM.webm

jakiuncle commented 2 years ago

You can try login out first , and try it again. I dont konw why you can go main page without logining

TeslaCN commented 2 years ago

You can try login out first , and try it again. I dont konw why you can go main page without logining

All the APIs returned 500, so the frontend could not redirect to login page.

jakiuncle commented 2 years ago

Hi i want send a video to you but i can't send a file size more than 10MB in github. How can I send it to you.

TeslaCN commented 2 years ago

Hi i want send a video to you but i can't send a file size more than 10MB in github. How can I send it to you.

You may send to my email wuweijie@apache.org

TeslaCN commented 2 years ago

Hi @jakiuncle I found the way to reproduce.

  1. Start frontend and backend service.
  2. Login with account.
  3. Restart backend service.
  4. Do any operation then got 500. (In master branch, the frontend will redirect to login page automatically)
jakiuncle commented 2 years ago

Hi, i fix this error. You can try it again.

TeslaCN commented 2 years ago

Hi @jakiuncle bc239c7 fixed the problem that cannot redirect to login page. But after login, 500 still occurred. Could you check it again?

image

image

jakiuncle commented 2 years ago

Hi,i found why this error appear and fix it.

jakiuncle commented 2 years ago

@TeslaCN plz review

TeslaCN commented 2 years ago

@TeslaCN plz review

Sorry for the delay, I will try this soon.

tristaZero commented 2 years ago

This feature looks cool, Thanks for your contribution! @jakiuncle Once it's merged, maybe we could consider a post to let more users know about it?

jakiuncle commented 2 years ago

Yes, it's a good idea.

jakiuncle commented 2 years ago

Thank you. It's a pleasant cooperation

tristaZero commented 2 years ago

@jakiuncle Welcome, @TeslaCN we can discuss the article content and our community editor can help us edit the article and post it on the global channel to help more people know about it. ;-)

jakiuncle commented 2 years ago

ok