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.

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

Chore: Adding "allowed_groups" Configuration Parameter to Generic OAuth Method #69025

Merged
merged 2 commits into from
May 30, 2023

Conversation

vsychov
Copy link
Contributor

@vsychov vsychov commented May 24, 2023

What is this feature?
This PR introduces a new configuration parameter, allowed_groups, for the Generic OAuth method.

Why do we need this feature?
This new feature enhances the functionality of the Generic OAuth method by allowing the specification of groups that have access to the service. This allows for fine-tuned access control and increases security and specificity in permissions settings.

Who is this feature for?
This feature is beneficial for system administrators or developers who want to exert more precise control over who can access their services. It's especially useful in environments with a variety of user roles or teams that require different levels of access to the same services.

Which issue(s) does this PR fix?:
This PR is not directly address any existing issues.

Special notes for your reviewer:

Please check that:

  • It works as expected from a user's perspective.
  • If this is a pre-GA feature, it is behind a feature toggle.
  • The docs are updated, and if this is a notable improvement, it's added to our What's New doc.

@vsychov vsychov requested review from torkelo, a team and chri2547 as code owners May 24, 2023 22:11
@vsychov vsychov requested review from IevaVasiljeva and danielkenlee and removed request for a team May 24, 2023 22:11
@CLAassistant
Copy link

CLAassistant commented May 24, 2023

CLA assistant check
All committers have signed the CLA.

@grafanabot grafanabot added area/backend type/docs pr/external This PR is from external contributor labels May 24, 2023
@IevaVasiljeva IevaVasiljeva added this to the 10.1.x milestone May 26, 2023
Copy link
Contributor

@IevaVasiljeva IevaVasiljeva left a comment

Choose a reason for hiding this comment

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

Looks good, thanks for your contribution!

@vsychov vsychov changed the title Adding "allowed_groups" Configuration Parameter to Generic OAuth Method chore: Adding "allowed_groups" Configuration Parameter to Generic OAuth Method May 26, 2023
@vsychov vsychov changed the title chore: Adding "allowed_groups" Configuration Parameter to Generic OAuth Method Chore: Adding "allowed_groups" Configuration Parameter to Generic OAuth Method May 26, 2023
@Jguer
Copy link
Contributor

Jguer commented May 26, 2023

A small section of doc could be added under ### Groups / Teams of the generic oauth index.md.

Apart from that implementation looks good 👍 , thanks for your contribution

@vsychov
Copy link
Contributor Author

vsychov commented May 26, 2023

A small section of doc could be added under ### Groups / Teams of the generic oauth index.md.

@Jguer, done.

@Jguer Jguer merged commit e7e70db into grafana:main May 30, 2023
5 checks passed
grobinson-grafana pushed a commit that referenced this pull request May 31, 2023
…th Method (#69025)

* feat: add allowed_groups for Generic OAuth

* docs: add allowed_groups more docs for Generic OAuth
@ricky-undeadcoders ricky-undeadcoders modified the milestones: 10.1.x, 10.1.0 Aug 1, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
add to changelog area/backend no-backport Skip backport of PR pr/external This PR is from external contributor type/docs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants