Closed (fixed)
Project:
OAuth2 Client
Version:
4.0.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Reporter:
Created:
11 Mar 2023 at 23:25 UTC
Updated:
18 May 2023 at 17:34 UTC
Jump to comment: Most recent
We recently noticed that our config sync was failing on the OAuth keys:
The configuration cannot be imported because it failed validation for the following reasons:
- Configuration oauth2_client.oauth2_client.ecampus depends on the veracross_oauth configuration that will not exist after import.
- Configuration oauth2_client.oauth2_client.fof depends on the veracross_oauth configuration that will not exist after import.
- Configuration oauth2_client.oauth2_client.transportation depends on the veracross_oauth configuration that will not exist after import.
- Configuration oauth2_client.oauth2_client.veracross_oauth depends on the veracross_oauth configuration that will not exist after import.
Digging deeper, it looks like this key:
langcode: en
status: true
dependencies:
config:
- veracross_oauth <--------
module:
- veracross_oauth
id: veracross_oauth
label: Veracross
description: ''
oauth2_client_plugin_id: veracross_oauth
credential_provider: key
credential_storage_key: veracross_oauthshould be this:
langcode: en
status: true
dependencies:
config:
- key.key.veracross_oauth <--------
module:
- veracross_oauth
id: veracross_oauth
label: Veracross
description: ''
oauth2_client_plugin_id: veracross_oauth
credential_provider: key
credential_storage_key: veracross_oauthStart 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:
Comments
Comment #3
fathershawnNice catch! I'll have a look at this since tests are not passing
Comment #4
fathershawnNice debugging @zerbash! I adjusted the approach to take advantage of some existing code. Tests are now passing. See if this fixes your setup?
Comment #5
zerbash commentedThis works for me -- but needs a plugin id...
Comment #6
fathershawnOh my! I guess I didn't actually run the export!! Nice fix.
Comment #8
fathershawnComment #9
fathershawnMy approach is not working on initial save. Refactoring to a blend of my approach and @zerbash initial approach.
Comment #12
fathershawn