Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feature: better config and environment handling #53

Merged
merged 3 commits into from
Mar 4, 2015

Conversation

thibaultcha
Copy link
Member

This PR introduces easier to use and understand configurations for Kong, as well as better bundled environments.

  • kong.yml: default configuration file. git-ignored.
  • The configuration file has an nginx property which is a multi-line string containing the nginx configuration.
  • scripts/config.lua: responsible for outputting this nginx property to a proper file. Also able to create new configuration for TESTS and DEVELOPMENT environments.
  • bin/kong: start will now use config.lua to output an nginx.conf file before running nginx.
  • Makefile: Simplification + make dev now creates TESTS and DEVELOPMENT environments.

As a result, unit and integration tests now run in the same environment (DB keyspace).

kong/yml now contains the nginx configuration as a string, and bin/kong
is responsible for extracting it before running nginx.

config.lua handles anything related to configuration files (creating new
config for environments or extracting the nginx config).
thibaultcha added a commit that referenced this pull request Mar 4, 2015
Feature: better config and environment handling
@thibaultcha thibaultcha merged commit 29e016a into master Mar 4, 2015
@thibaultcha thibaultcha deleted the feature/config branch March 4, 2015 03:21
ctranxuan pushed a commit to streamdataio/kong that referenced this pull request Aug 25, 2015
Feature: better config and environment handling
gszr added a commit that referenced this pull request Jun 17, 2021
Ensure the `multiValueHeaders` field is handled in AWS Lambda proxy
integration responses.

Fix #53.
gszr pushed a commit that referenced this pull request Jul 7, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant