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

TMP: Run as not-root #5694

Draft

jfharden wants to merge 1 commit into master from tmp/build-app-as-non-root

Contributor

jfharden commented Jan 20, 2025 •

edited

Loading

WHAT YOU DID

Run as a build user in the builder continaer
Run as pay system user in the final container
*Update ENV spec to resolve deprecation warnings for ENV <name> <value> format

How to test

Checkout this branch
Run with pay local pay local up --cluster card --local connector
Make a test payment pay local payment

Code review checklist

Logging

only emit log lines at ERROR level which require immediate attention from a support engineer. These will trigger a zendesk alert.

Documentation

Updated README.md for any of the following ?

Introduced any new environment variables / removed existing environment variable
Added new API / updated existing API definition


          TMP: Run as not-root

160f278

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet