-
Notifications
You must be signed in to change notification settings - Fork 0
Openid 20201105 #2
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Open
RubenGarcia
wants to merge
61
commits into
openid
Choose a base branch
from
openid_20201105
base: openid
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Adds a test for fetching the size attribute of two replicas of a given data object to see that they are different.
Replicas can have different sizes than the size in the catalog for a given data object (stale replicas). Therefore, this change adds a size attribute to iRODSReplica to allow users to get the size of a particular replica.
Escape all StringProperty values for packStruct. HTML escaping was already in place for a few spots, so these have been removed.
MySQL needs uppercased table and column names, so the TestSpecificQuery tests have been made to accommodate this requirement.
The implemented PAM protocol is based on the following steps 1 . AUTH_PLUG_REQ_AN is sent by passing a context containing a_user, a_pw and a_ttl (defaulted to 60) 2 . irods server responds with a temporary password 3 . A a new connection is created (a combination of disconnect + connect). 4 . The temporary password is used to login as a native protocol To use PAM set authentication_scheme='pam' when creating the iRODSSession object iRODSSession(user='yourPAMuser', password='yourPAMpassword, authentication_scheme='pam', [...]) As a plus: - created a generic AuthPluginMessage to replace the GSIAuthMessage - added more constants for better readability
Use start=1 on enumerate
Add example for searching metadata
Finding out the full resource hierarchy is important for dealing with replicas. In the past, we had a naming scheme that allowed us to work around it, but now that naming scheme doesn't provide what we need. Including resc_hier in the replicas information allows us to do this the right way
test_user_dn was failing, preventing 'rodsuser' from being removed, causing subsequent tests to fail on creation of the same user
Disabled by default; to enable, follow the usage session.permissions.get( object, report_raw_acls = True )
Signed-off-by: Patrice Linel <[email protected]>
When getting a connection, check when the connection was last used (This is done via connection's last_used_time field). If it was last used more than a specific amount (configurable), drop the connection and create a one. This is to address the issue with connections that are dropped after being not used for a long time.
…01105 Add latest 54 commits from grandparent branch and fix conflicts. Separate into a new branch in case any changes from the grandfather branch is incompatible with the old irods versions.
…her a connection needs to be re-created
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Changes from upstream which fix the creation of users in remote zones.