Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Merge globus 4 & 5 implementations of authz module #871

Open
dvstans opened this issue Dec 30, 2022 · 0 comments
Open

Merge globus 4 & 5 implementations of authz module #871

dvstans opened this issue Dec 30, 2022 · 0 comments
Labels
Component: GridFTP Auth Module Relates to GridFTP authorization library Priority: Medium Above average priority Type: Refactor Imlplementation change, same functionality

Comments

@dvstans
Copy link
Collaborator

dvstans commented Dec 30, 2022

95% of the code is the same between the globus 4 and 5 implementations of the authz module. These should be merged into one and coded to auto-detect which version of globus is installed. This can be done by testing for the client id environment var that is present only in globus 5. Note this applies to the devel branch only.
@dvstans dvstans added Component: GridFTP Auth Module Relates to GridFTP authorization library Type: Refactor Imlplementation change, same functionality Priority: Medium Above average priority labels Dec 30, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Component: GridFTP Auth Module Relates to GridFTP authorization library Priority: Medium Above average priority Type: Refactor Imlplementation change, same functionality
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@dvstans