+
Skip to content

Get unpriv nginx working #793

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

Closed
wants to merge 2 commits into from
Closed

Get unpriv nginx working #793

wants to merge 2 commits into from

Conversation

modem7
Copy link

@modem7 modem7 commented Dec 4, 2023

Added custom script for nginx.conf replacing the nginx.conf file.

Added healthcheck endpoint at /healthz

Optimised nginx.conf with caching and IP headers for security logging

Added PORT variable to change port nginx listens on more easily (default 8080, but works with 80 also).

Added custom script for nginx.conf

Added healthcheck endpoint at /healthz

Optimised nginx.conf with caching and IP headers for security logging
Copy link

vercel bot commented Dec 4, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
it-tools ✅ Ready (Inspect) Visit Preview Feb 22, 2024 11:12pm

Copy link

sonarqubecloud bot commented Dec 4, 2023

SonarCloud Quality Gate failed.    Quality Gate failed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot E 2 Security Hotspots
Code Smell A 1 Code Smell

No Coverage information No Coverage information
0.0% 0.0% Duplication

idea Catch issues before they fail your Quality Gate with our IDE extension sonarlint SonarLint

@modem7
Copy link
Author

modem7 commented Dec 4, 2023

I changed the RUN command at

it-tools/Dockerfile

Lines 14 to 15 in f1a90f0

RUN npm install -g pnpm
RUN pnpm i --frozen-lockfile
to better utilise docker layer caching during rebuilds.

Most of the code was taken from another project of mine (https://github.com/modem7/docker-rickroll) which utilises the unprivileged image and has additional scripts and nginx.conf settings to make things work that you may be interested in.

@modem7
Copy link
Author

modem7 commented Feb 22, 2024

@CorentinTh - would it be possible to review this PR to see if it's worthwhile implementing?

It should resolve the issues that were seen with #714 and #716

@modem7
Copy link
Author

modem7 commented Jul 12, 2025

closing as it obviously won't be implemented.

@modem7 modem7 closed this Jul 12, 2025
@sharevb
Copy link
Contributor

sharevb commented Jul 12, 2025

Hi @modem7 , implemented in my fork

And if you arr interested in up to date version of it-tools, I made kind of a fork here : https://github.com/sharevb/it-tools (https://sharevb-it-tools.vercel.app/ and docker images https://github.com/sharevb/it-tools/pkgs/container/it-tools)

@modem7
Copy link
Author

modem7 commented Jul 12, 2025

Hi @modem7 , implemented in my fork

And if you arr interested in up to date version of it-tools, I made kind of a fork here : https://github.com/sharevb/it-tools (https://sharevb-it-tools.vercel.app/ and docker images https://github.com/sharevb/it-tools/pkgs/container/it-tools)

I'll have a look this evening! Thank you!

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.

2 participants
点击 这是indexloc提供的php浏览器服务,不要输入任何密码和下载