Adding a Script in "./httpd-pre-init" does not work as expected about s2i-php-container HOT 2 CLOSED

Thanks for reporting this issue. There is inconsistency in what is in the code and what is in the readme.

README describes ./httpd-cfg , ./httpd-pre-init , ./httpd-ssl folders. But in code ./httpd-cfg , ./php-pre-start , ./httpd-ssl , ./php-post-assemble folders are used.
@hhorak You introduced the change. Which naming we want use?

@dirkrydvan So hotfix for this issue is to use php-pre-start folder instead of httpd-pre-init. But be prepared that a resolution of this issue can change naming.

from s2i-php-container.

@omron93 Yeah, naming is never easy :) my thinking behind this was to use the same entrypoint names where I'd expect the same content/use case for httpd and php container image -- that's the httpd-ssl a httpd-cfg case. However, my expectation for content/use case for post-assemble and pre-start entrypoints would be different in case of php, than in case of httpd container image; therefore different name. Anyway, my intention was what is in the code, the README is wrong in this case.

from s2i-php-container.