Skip to content

82 external oauth roles #213

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
wants to merge 14 commits into
base: main
Choose a base branch
from
Open

82 external oauth roles #213

wants to merge 14 commits into from

Conversation

bramtweedegolf
Copy link
Contributor

@bramtweedegolf bramtweedegolf commented May 15, 2025

Map OpenADR OAuth2 scopes to internal roles.

Closes issue #82

@bramtweedegolf bramtweedegolf requested a review from pohlm01 May 15, 2025 08:37
@bramtweedegolf bramtweedegolf changed the title 82 external oauth roles [WIP] 82 external oauth roles May 15, 2025
Copy link

codecov bot commented May 15, 2025

Codecov Report

Attention: Patch coverage is 69.55446% with 123 lines in your changes missing coverage. Please review.

Project coverage is 81.13%. Comparing base (6c77c8f) to head (570d242).
Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
openleadr-vtn/src/jwt.rs 65.77% 102 Missing ⚠️
openleadr-vtn/src/state.rs 87.14% 9 Missing ⚠️
openleadr-client/src/bin/cli.rs 0.00% 5 Missing ⚠️
openleadr-client/src/error.rs 0.00% 5 Missing ⚠️
openleadr-client/src/bin/everest.rs 0.00% 1 Missing ⚠️
openleadr-vtn/src/main.rs 0.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #213      +/-   ##
==========================================
- Coverage   82.16%   81.13%   -1.03%     
==========================================
  Files          42       42              
  Lines        5690     5995     +305     
==========================================
+ Hits         4675     4864     +189     
- Misses       1015     1131     +116     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

bramtweedegolf and others added 8 commits May 16, 2025 11:31
Signed-off-by: Bram Tweedegolf <[email protected]>
Signed-off-by: Bram Tweedegolf <[email protected]>
Signed-off-by: Bram Tweedegolf <[email protected]>
Signed-off-by: Bram Tweedegolf <[email protected]>
* Add vim temp files to gitignore

Signed-off-by: Bram Tweedegolf <[email protected]>

* Minor whitespace cleanup

Signed-off-by: Bram Tweedegolf <[email protected]>

---------

Signed-off-by: Bram Tweedegolf <[email protected]>
@bramtweedegolf bramtweedegolf force-pushed the 82-external-oauth-roles branch from cbff89d to e96a14b Compare May 19, 2025 09:26
Signed-off-by: Bram Tweedegolf <[email protected]>
"write_programs" => scopes.push(Scope::WritePrograms),
"write_events" => scopes.push(Scope::WriteEvents),
"write_reports" => scopes.push(Scope::WriteReports),
_ => {}
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Log

@bramtweedegolf bramtweedegolf changed the title [WIP] 82 external oauth roles 82 external oauth roles May 23, 2025
@bramtweedegolf bramtweedegolf marked this pull request as ready for review May 23, 2025 07:39
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.

1 participant