Closed (fixed)
Project:
Acquia Connector
Version:
8.x-1.x-dev
Component:
Code
Priority:
Major
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
14 Dec 2015 at 20:46 UTC
Updated:
11 Apr 2017 at 13:05 UTC
Jump to comment: Most recent
Comments
Comment #2
Andre-BThis introduces another issue as well:
The once exported configuration contains subscription information which might be outdated. Upon updating a site using regular deployment process the old configuration will be imported again (unless the configuration changed by cronjob on production machine will be somehow exported and put back into vcs).
Alternatively the whole configuration needs to be excluded.
Comment #3
larowlanuse the state api, that's what its for - my 2c
Comment #4
anavarreAgreed with #3 - That's what Migrate API does.
Comment #5
bkosborneAgreed. It seems that very little of what this module is storing in config should actually be stored in config. Using the State API makes the most sense.
Comment #6
bkosborneThis feels like a major to me, given that it disrupts config workflows and makes it easy to accidentally save sensitive data to version control.
Comment #7
bkosborneHow is anyone using this module while also using the config management system in any fashion?
It seems like the only option is to always go to your prod site, export the config, then copy the acquia connector settings file into your local site and commit that along with any other config changes that you're actually making.
Comment #8
bkosborneComment #9
irek02 commentedThanks the feedback, everyone, this will be fixed in near future.
Comment #10
bkosborneHi irek, I wonder if there's a status update on this? Specifically moving the state data out of configuration and into the state API? If not, are there any known workarounds to prevent this file from constantly being overwritten and modified via config-import / config-export?
Comment #11
irek02 commented@bkosborne, the fix will be released sometime next week.
Comment #12
bkosborneHi Irek, I see that the release was put out that removes the API creds from config and migrates them to state API.
Do you still plan on moving all the other stuff into state API, like subscription data? That's the stuff that causes some problems for me since it seems it changes based on things out my control, and things get wiped out during config import/export process.
Comment #13
irek02 commentedbkosborne, yes, the plan is to move all other config into the State API. I'm going to close this story as the security concern with the credentials was addressed. Would you like to create a new issue about the subscription_data part?
Comment #14
irek02 commentedComment #15
bkosborneDone: #2864697: Move subscription and SPI data out of config and into state API