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_oauth

should 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_oauth
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:

Comments

zerbash created an issue. See original summary.

fathershawn’s picture

Assigned: Unassigned » fathershawn
Status: Active » Needs work

Nice catch! I'll have a look at this since tests are not passing

fathershawn’s picture

Status: Needs work » Needs review

Nice debugging @zerbash! I adjusted the approach to take advantage of some existing code. Tests are now passing. See if this fixes your setup?

zerbash’s picture

This works for me -- but needs a plugin id...

fathershawn’s picture

Status: Needs review » Reviewed & tested by the community

Oh my! I guess I didn't actually run the export!! Nice fix.

  • FatherShawn committed 06d47ec9 on 4.0.x authored by zerbash
    Issue #3347425 by zerbash, FatherShawn: Config is not storing key module...
fathershawn’s picture

Status: Reviewed & tested by the community » Fixed
fathershawn’s picture

Status: Fixed » Needs work

My approach is not working on initial save. Refactoring to a blend of my approach and @zerbash initial approach.

  • FatherShawn committed 3e734ca4 on 4.0.x
    Issue #3347425 by FatherShawn, zerbash: Config is not storing key module...

  • FatherShawn committed 487b9cfb on 4.0.x
    Issue #3347425 by FatherShawn, zerbash: Migration items can be skipped...
fathershawn’s picture

Status: Needs work » Fixed

Status: Fixed » Closed (fixed)

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