#151 Try to simplify login stuff
Merged 3 years ago by frantisekz. Opened 3 years ago by jskladan.

@@ -66,8 +66,9 @@ 

      return redirect(url + '?oidc_token=%s' % token)

  

  

+ @app.route('/api/v1/logout')

  @app.route('/api/v1/oidc_logout')

- def api_v1_oidc_logout():

+ def api_v1_logout():

      oidc.logout()

      logout_user()

      flask.session.clear()

@@ -22,7 +22,7 @@ 

  from flask import jsonify

  from flask_login import current_user

  

- from oraculum import app, CACHE, oidc

+ from oraculum import app, CACHE

  from oraculum.utils import bodhi, cache_utils, celery_utils, calendars, dashboard_helpers, health_check, koschei, orphans, pagure, versions

  

  def handle_orphan_user():
@@ -83,7 +83,7 @@ 

          # Hit when user isn't logged in

          fas_groups = []

  

-     if oidc.user_loggedin and "packager" in fas_groups:

+     if "packager" in fas_groups:

          bzs = dashboard_user_data_bzs(user, authenticated=True)

      else:

          bzs = dashboard_user_data_bzs(user, authenticated=False)

Rely on flask_login to handle all the login-related stuff, and only use OIDC to actually really get the credentials during login.

Build succeeded.

Pull-Request has been merged by frantisekz

3 years ago