Skip to content

Commit

Permalink
Merge pull request #1083 from 3scale/merge-master-to-3.6-stable
Browse files Browse the repository at this point in the history
Bring changes in master to 3.6-stable
  • Loading branch information
davidor authored Jul 4, 2019
2 parents e18db10 + bcdbf21 commit a89a51e
Show file tree
Hide file tree
Showing 6 changed files with 37 additions and 5 deletions.
4 changes: 4 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -6,6 +6,10 @@ and this project adheres to [Semantic Versioning](http://semver.org/).

## [Unreleased]

### Added

- Extended variables in Liquid template operations [PR #1081](https://github.com/3scale/APIcast/pull/1081)

## [3.6.0-beta1] - 2019-06-18

### Added
Expand Down
19 changes: 16 additions & 3 deletions Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,19 @@ RUNTIME_IMAGE ?= $(BUILDER_IMAGE)-runtime

DEVEL_IMAGE ?= apicast-development
DEVEL_DOCKERFILE ?= Dockerfile-development

DEVEL_DOCKER_COMPOSE_FILE ?= docker-compose-devel.yml
DEVEL_DOCKER_COMPOSE_VOLMOUNT_MAC_FILE ?= docker-compose-devel-volmount-mac.yml
DEVEL_DOCKER_COMPOSE_VOLMOUNT_DEFAULT_FILE ?= docker-compose-devel-volmount-default.yml

os = "$(shell uname -s)"

# if running on Mac
ifeq ($(os),"Darwin")
DEVEL_DOCKER_COMPOSE_VOLMOUNT_FILE = $(DEVEL_DOCKER_COMPOSE_VOLMOUNT_MAC_FILE)
else
DEVEL_DOCKER_COMPOSE_VOLMOUNT_FILE = $(DEVEL_DOCKER_COMPOSE_VOLMOUNT_DEFAULT_FILE)
endif

S2I_CONTEXT ?= gateway

Expand Down Expand Up @@ -190,12 +202,13 @@ development: USER := $(shell id -u $(USER))
endif
development: .docker/lua_modules .docker/local .docker/cpanm .docker/vendor/cache
development: ## Run bash inside the development image
- $(DOCKER_COMPOSE) -f $(DEVEL_DOCKER_COMPOSE_FILE) up --detach
@echo "Running on $(os)"
- $(DOCKER_COMPOSE) -f $(DEVEL_DOCKER_COMPOSE_FILE) -f $(DEVEL_DOCKER_COMPOSE_VOLMOUNT_FILE) up -d
@ # https://github.com/moby/moby/issues/33794#issuecomment-312873988 for fixing the terminal width
$(DOCKER_COMPOSE) -f $(DEVEL_DOCKER_COMPOSE_FILE) exec -e COLUMNS="`tput cols`" -e LINES="`tput lines`" --user $(USER) development bash
$(DOCKER_COMPOSE) -f $(DEVEL_DOCKER_COMPOSE_FILE) -f $(DEVEL_DOCKER_COMPOSE_VOLMOUNT_FILE) exec -e COLUMNS="`tput cols`" -e LINES="`tput lines`" --user $(USER) development bash

stop-development: ## Stop development environment
- $(DOCKER_COMPOSE) -f $(DEVEL_DOCKER_COMPOSE_FILE) down
- $(DOCKER_COMPOSE) -f $(DEVEL_DOCKER_COMPOSE_FILE) -f $(DEVEL_DOCKER_COMPOSE_VOLMOUNT_FILE) down

rover: $(ROVER)
@echo $(ROVER)
Expand Down
7 changes: 7 additions & 0 deletions docker-compose-devel-volmount-default.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,7 @@
version: '2.2'
services:
development:
volumes:
- .:/home/centos/
- .docker/cpanm:/home/centos/.cpanm

6 changes: 6 additions & 0 deletions docker-compose-devel-volmount-mac.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,6 @@
version: '2.2'
services:
development:
volumes:
- .:/home/centos/:cached
- .docker/cpanm:/home/centos/.cpanm:delegated
2 changes: 0 additions & 2 deletions docker-compose-devel.yml
Original file line number Diff line number Diff line change
Expand Up @@ -6,11 +6,9 @@ services:
- redis
working_dir: /home/centos/
volumes:
- .:/home/centos/:cached
- .docker/lua_modules:/home/centos/lua_modules
- .docker/local:/home/centos/local
- .docker/vendor/cache:/home/centos/vendor/cache
- .docker/cpanm:/home/centos/.cpanm:delegated
# no need to access those from docker
- /home/centos/.docker
- /home/centos/.git
Expand Down
4 changes: 4 additions & 0 deletions gateway/src/apicast/policy/ngx_variable.lua
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,10 @@ local function context_values()
uri = ngx.var.uri,
host = ngx.var.host,
remote_addr = ngx.var.remote_addr,
remote_port = ngx.var.remote_port,
scheme = ngx.var.scheme,
server_addr = ngx.var.server_addr,
server_port = ngx.var.server_port,
headers = ngx.req.get_headers(),
http_method = ngx.req.get_method(),
}
Expand Down

0 comments on commit a89a51e

Please sign in to comment.