Skip to content

[#561] implementation: get_maximum_permissions_for_path - #602

Draft
d-w-moore wants to merge 3 commits into
irods:mainfrom
d-w-moore:561.m
Draft

[#561] implementation: get_maximum_permissions_for_path#602
d-w-moore wants to merge 3 commits into
irods:mainfrom
d-w-moore:561.m

Conversation

@d-w-moore

@d-w-moore d-w-moore commented Aug 2, 2024

Copy link
Copy Markdown
Collaborator

Check if the user named (may be a user or group) has privilege at or above the given access_level to the collection or data object named by path

@d-w-moore

d-w-moore commented Aug 2, 2024

Copy link
Copy Markdown
Collaborator Author

Needs test.

@d-w-moore
d-w-moore marked this pull request as draft August 2, 2024 04:59
@d-w-moore

d-w-moore commented Dec 2, 2024

Copy link
Copy Markdown
Collaborator Author

Looks as if we won't implement this, at least not in this way. close?

@alanking

alanking commented Dec 2, 2024

Copy link
Copy Markdown
Contributor

I vote we leave it open / in draft until an alternative implementation presents itself. Is there a reason you want to close this?

@d-w-moore

d-w-moore commented Dec 3, 2024

Copy link
Copy Markdown
Collaborator Author

I vote we leave it open / in draft until an alternative implementation presents itself. Is there a reason you want to close this?

No reason. I think we decided it's best implemented as a specific query.

@korydraughn

korydraughn commented Dec 3, 2024

Copy link
Copy Markdown
Contributor

We need to do more investigative work for #561.

Let's keep this in draft for now. We can circle back once the 3.0.0 milestone is complete.

@d-w-moore
user equality operator (for group membership test)
f93a93c
@d-w-moore
[_561] implement get_maximum_permissions_for_path functionality and test
70401a0 
A query is done to determine the highest permission level the given user
(or group) has on the target collection/data_object.

A specific query will be used if the calling user has the availability,
otherwise a reasonably efficient general query is used instead.
@d-w-moore

d-w-moore commented Oct 25, 2026

Copy link
Copy Markdown
Collaborator Author

Hm. Looks like I force-pushed over the old branch. Note for the future, don't do that. (Look for pre-existence of the branch name in GitHub. Duh.)

Well, luckily I am very confident of the new code being better....

@d-w-moore d-w-moore self-assigned this Oct 25, 2026
@d-w-moore d-w-moore changed the title [#561] experimental implementation: user_has_access [#561] implementation: get_maximum_permissions_for_path Oct 25, 2026
@d-w-moore

d-w-moore commented Oct 25, 2026

Copy link
Copy Markdown
Collaborator Author

Will mark ready for review once I've run the tests through with rudra's changes included

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

@d-w-moore d-w-moore

Labels

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@d-w-moore @alanking @korydraughn