Skip to content

Commit 6a997f5

Browse files
committed
review update - comment wrt pam, native, and anonymous login
1 parent 933aad6 commit 6a997f5

1 file changed

Lines changed: 2 additions & 0 deletions

File tree

irods/session.py

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -261,6 +261,8 @@ def _configure_account(self, **kwargs):
261261
missing_file_path = []
262262
error_args = []
263263
pw = creds['password'] = self.get_irods_password(session_ = self, file_path_if_not_found = missing_file_path, **creds)
264+
# For native authentication, a missing password should be flagged as an error for non-anonymous logins.
265+
# However, the pam_password case has its own internal checks.
264266
if auth_scheme.lower() not in PAM_AUTH_SCHEMES:
265267
if not pw and creds.get('irods_user_name') != 'anonymous':
266268
if missing_file_path:

0 commit comments

Comments
 (0)