Fix small pylint issues
This commit is contained in:
		
							parent
							
								
									ff52fde8a5
								
							
						
					
					
						commit
						36324708db
					
				
					 1 changed files with 3 additions and 3 deletions
				
			
		|  | @ -148,7 +148,7 @@ def github_oauth_callback(): | ||||||
| 
 | 
 | ||||||
|   # Retrieve the user's information. |   # Retrieve the user's information. | ||||||
|   user_data = get_user(github_login, token) |   user_data = get_user(github_login, token) | ||||||
|   if not user_data or not 'login' in user_data: |   if not user_data or 'login' not in user_data: | ||||||
|     return render_ologin_error('GitHub') |     return render_ologin_error('GitHub') | ||||||
| 
 | 
 | ||||||
|   username = user_data['login'] |   username = user_data['login'] | ||||||
|  | @ -168,7 +168,8 @@ def github_oauth_callback(): | ||||||
|                           headers={'Accept': 'application/vnd.github.moondragon+json'}) |                           headers={'Accept': 'application/vnd.github.moondragon+json'}) | ||||||
| 
 | 
 | ||||||
|     organizations = set([org.get('login').lower() for org in get_orgs.json()]) |     organizations = set([org.get('login').lower() for org in get_orgs.json()]) | ||||||
|     if not (organizations & set(github_login.allowed_organizations())): |     matching_organizations = organizations & set(github_login.allowed_organizations()) | ||||||
|  |     if not matching_organizations: | ||||||
|       err = """You are not a member of an allowed GitHub organization. |       err = """You are not a member of an allowed GitHub organization. | ||||||
|                Please contact your system administrator if you believe this is in error.""" |                Please contact your system administrator if you believe this is in error.""" | ||||||
|       return render_ologin_error('GitHub', err) |       return render_ologin_error('GitHub', err) | ||||||
|  | @ -351,4 +352,3 @@ def dex_oauth_attach(): | ||||||
|     return render_ologin_error(dex_login.public_title, err) |     return render_ologin_error(dex_login.public_title, err) | ||||||
| 
 | 
 | ||||||
|   return redirect(url_for('web.user_view', path=user_obj.username, tab='external')) |   return redirect(url_for('web.user_view', path=user_obj.username, tab='external')) | ||||||
| 
 |  | ||||||
|  |  | ||||||
		Reference in a new issue