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

Add support for Twilio SMS #67

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Add support for Twilio SMS #67

wants to merge 1 commit into from

Conversation

SuperQ
Copy link

@SuperQ SuperQ commented Nov 11, 2023

Upstream now includes a Twilio example script.

  • Add support for adding new values to the config.inc.php.
  • Add variables for the Twilio example script.

Upstream now includes a Twilio example script.
* Add support for adding new values to the config.inc.php.
* Add variables for the Twilio example script.

Signed-off-by: SuperQ <superq@gmail.com>
@SuperQ
Copy link
Author

SuperQ commented Nov 11, 2023

I'm getting a strange build error when I try and build this locally.

STEP 4/5: RUN source /assets/functions/00-container &&      set -x &&      package update &&      package upgrade &&      mkdir -p /assets/install &&      curl -sSL -o /assets/install/v${SSP_VERSION}.tar.gz https://github.com/ltb-project/self-service-password/archive/v${SSP_VERSION}.tar.gz &&      package cleanup
/bin/sh: /assets/functions/00-container: line 1396: syntax error: unexpected "(" (expecting "done")
Error: error building at STEP "RUN source /assets/functions/00-container &&      set -x &&      package update &&      package upgrade &&      mkdir -p /assets/install &&      curl -sSL -o /assets/install/v${SSP_VERSION}.tar.gz https://github.com/ltb-project/self-service-password/archive/v${SSP_VERSION}.tar.gz &&      package cleanup": error while running runtime: exit status 2

This also errors when built from main. The line in question is here. It might be the fact that I'm using podman to build and not actual Docker.

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