Problem/Motivation

  • In the Drupal 11 version of SimpleOauth, the Scope configuration is not created by default anymore. See: https://www.drupal.org/project/simple_oauth/issues/3263590
  • For configuring SPA, we now need to create a Scope entity before being able to login to the SPA. Create it in the enableSpaApp method.
  • This code should not be merged into 7.x. Cloud Orchestrator 7.x uses Simple Oauth 5.2.5, which does not require this change.
CommentFileSizeAuthor
#3 Dynamic_scopes___Cloud_Orchestrator.png40.13 KBbaldwinlouie

Issue fork cloud-3516926

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

baldwinlouie created an issue. See original summary.

baldwinlouie opened merge request !2334

baldwinlouie’s picture

baldwinlouie commented
Status: Active » Needs review
StatusFileSize
new Dynamic_scopes___Cloud_Orchestrator.png40.13 KB

@yas, Please review this patch. This add support for creating the Scope entity for drupal/simple_oauth:6.x

image

yas’s picture

yas
🇯🇵 he/him
Primary language Japanese
Location California 🇺🇸
commented
Title: In enableSpaApp() function, for 8.x, create the Oauth Scope entity » Add creation of OAuth Scope entity in enableSpaApp() for drupal/cloud:8.x due to missing setup in SimpleOauth
Issue summary: View changes
Status: Needs review » Reviewed & tested by the community

@baldwinlouie

Thank you for the MR. I’ll merge the patch into 8.x; and close this issue as Fixed.

  • yas committed d55755a2 on 8.x authored by baldwinlouie 
    Issue #3516926 by baldwinlouie, yas: Add creation of OAuth Scope entity...
yas’s picture

yas
🇯🇵 he/him
Primary language Japanese
Location California 🇺🇸
commented
Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.