Skip to content

Fix scope handling#60

Merged
cjh1 merged 1 commit into
OpenCHAMI:mainfrom
cjh1:scopes
Jan 21, 2026
Merged

Fix scope handling#60
cjh1 merged 1 commit into
OpenCHAMI:mainfrom
cjh1:scopes

Conversation

@cjh1
Copy link
Copy Markdown
Member

@cjh1 cjh1 commented Jan 15, 2026

StringSlice expects to receive the whole comma-separated value.

Summary and Scope

Fix OAuth2 scope handling

Testing

List the environments in which these changes were tested.

Tested on:

  • Local development environment

Test description:

  • Configured scopes and ensure the values were parsed correcty.

Pull Request Checklist

  • Version number(s) incremented, if applicable
  • Copyrights updated
  • License file intact
  • Target branch correct
  • CHANGELOG.md updated
  • Testing is appropriate and complete, if applicable

StringSlice expects to receive the whole comma-separated value.

Signed-off-by: Chris Harris <cjh@lbl.gov>
Copy link
Copy Markdown
Contributor

@davidallendj davidallendj left a comment

Choose a reason for hiding this comment

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

LGTM

@cjh1 cjh1 merged commit d90152b into OpenCHAMI:main Jan 21, 2026
11 of 13 checks passed
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.

2 participants