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.

Sign up for GitHub

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

Jump to bottom

appservice: Replace nginx with Starlette reverse proxy, add session status API #37

Merged
merged 7 commits into from
Sep 26, 2022
Merged

appservice: Replace nginx with Starlette reverse proxy, add session status API #37

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
7 commits
Select commit Hold shift + click to select a range
a902866
3scale: Fix capitalization of Connection: header value
martinpitt Sep 22, 2022
8ecf554
appservice: Replace nginx with aiohttp reverse proxy
martinpitt Sep 15, 2022
da1286c
appservice: Move to RHEL UBI image
martinpitt Sep 15, 2022
c0f82c9
appservice: Add initial status API
martinpitt Sep 19, 2022
1a8810b
appservice: Rewrite using starlette and httpx
martinpitt Sep 21, 2022
e1d438b
appservice: Hack around broken 3scale Connection: header
martinpitt Sep 21, 2022
7185e72
appservice: Retry redis connection
martinpitt Sep 22, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion 3scale/nginx.conf
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -55,7 +55,7 @@ http {
# proxy_set_header Connection $http_connection;
# .. but it doesn't: it always adds this header even for plain HTTP requests
# https://issues.redhat.com/browse/RHCLOUD-21326
proxy_set_header Connection "upgrade";
proxy_set_header Connection "Upgrade";

# Pass ETag header from Cockpit to clients.
# See: https://github.com/cockpit-project/cockpit/issues/5239
Expand Down
12 changes: 4 additions & 8 deletions appservice/Containerfile
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,12 +1,8 @@
FROM debian:bookworm
FROM docker.io/redhat/ubi9-minimal

RUN apt-get update && \
apt-get install -y python3 python3-redis nginx curl inetutils-ping procps && \
apt-get clean && \
rm /var/lib/apt/lists/*dists*

# allow unprivileged container user to run nginx and change configuration
RUN chmod -R a+rw /etc/nginx/ /var/lib/nginx/ /var/log/nginx/ /run
# iputils and procps-ng are just for debugging; drop for production
RUN microdnf install -y python3-pip iputils procps-ng && microdnf clean all
RUN pip3 install redis starlette httpx websockets uvicorn

COPY *.py /usr/local/bin/

Expand Down
Loading