Skip to content

read license from envt variable #2259

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

Merged
merged 5 commits into from
Aug 26, 2022

Conversation

sinhaashish
Copy link
Contributor

@sinhaashish sinhaashish commented Aug 23, 2022

Earlier the license were read from config , and only admin had the privilige to read those the config , so any normal user with read write permission were seeing AGPL license even when they had the license .

With this PR the license is being read from envt variable which is set in MinIO at initialization.

Copy link
Collaborator

@bexsoft bexsoft left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please run make prettify in portal_ui folder

kannappanr
kannappanr previously approved these changes Aug 24, 2022
@sinhaashish sinhaashish force-pushed the skip-admin-validation branch 4 times, most recently from b7ad4d9 to 564104a Compare August 25, 2022 08:58
@sinhaashish sinhaashish force-pushed the skip-admin-validation branch from 564104a to 0cc79eb Compare August 25, 2022 10:16
Copy link
Member

@prakashsvmx prakashsvmx left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👍

Copy link
Collaborator

@bexsoft bexsoft left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@bexsoft bexsoft merged commit 9821beb into minio:master Aug 26, 2022
@sinhaashish sinhaashish deleted the skip-admin-validation branch August 29, 2022 03:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants