Feature #26783

Support settings via OS Environment

Added by Robert Lemke about 8 years ago. Updated almost 7 years ago.

Status:
Rejected
Priority:
Could have
Category:
Configuration
Start date:
2011-05-12
Due date:
% Done:

0%

PHP Version:
Has patch:
No
Complexity:

Description

For sensitive data or other purposes it would be nice to support altering settings by some environment variables.

Example:

(YAML):
FLOW3:
persistence:
backendOptions:
password: 'mypassword'

(.htaccess):

SetEnv FLOW3_FLOW3_PERSISTENCE_BACKENDOPTIONS_PASSWORD "mypassword"


Related issues

Related to TYPO3.Flow - Feature #45851: Allow referencing environment variables in Settings.yaml Needs Feedback 2013-02-26

History

#1 Updated by Karsten Dambekalns about 8 years ago

  • Target version changed from 1.0 beta 1 to 1230

#2 Updated by Mr. Hudson almost 8 years ago

  • Status changed from New to Under Review

Patch set 1 of change I89877637d440e060c45dc73011e65a01b36302c4 has been pushed to the review server.
It is available at http://review.typo3.org/5731

#3 Updated by Christian Müller almost 8 years ago

  • Assignee set to Christian Müller
  • Target version changed from 1230 to 1.1

#4 Updated by Robert Lemke over 7 years ago

  • Has patch set to No

Thanks for starting with the implementation!

I suggest that we follow the general UNIX conventions and write the environment variable names all uppercase. Additionally, I'd rather not use an additional prefix such as "F3" – can you just iterate over the active package keys, replace "." by "_", make it all uppercase and then consider the related environment variables? So, in the end we'd have: TYPO3_FLOW3_SECURITY_ENABLE = 1

#5 Updated by Gerrit Code Review over 7 years ago

Patch set 2 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/5731

#6 Updated by Gerrit Code Review over 7 years ago

Patch set 3 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/5731

#7 Updated by Gerrit Code Review over 7 years ago

Patch set 4 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/5731

#8 Updated by Christian Müller over 7 years ago

How should we proceed with this? The reviews show a dislike for it, I would love to discuss the points shortly, I still feel it can have valid use-cases...

#9 Updated by Christian Müller about 7 years ago

  • Target version changed from 1.1 to 1.1 beta 1

#10 Updated by Gerrit Code Review about 7 years ago

Patch set 5 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/5731

#11 Updated by Karsten Dambekalns about 7 years ago

  • Target version changed from 1.1 beta 1 to 1.1

#12 Updated by Karsten Dambekalns about 7 years ago

  • Target version changed from 1.1 to 2.0 beta 1

#13 Updated by Christian Müller almost 7 years ago

  • Status changed from Under Review to Rejected

I abandon the change. Maybe we come up with some totally different way of setting stuff, but OS Environment seems not a good idea for

a) Security Reasons
b) Caching
c) General Performance as all ENV need to be checked
d) Every possible way of implementing has some drawbacks, like the Setting must exist in some Settings already, you cannot set a new setting.

Also available in: Atom PDF