From c6f696f392b03fb04712380bda6892c5fdefb0cd Mon Sep 17 00:00:00 2001 From: Daniel Bright Date: Tue, 21 May 2024 08:11:51 -0700 Subject: [PATCH] verbiage from cx --- README.md | 3 ++- bin/compile | 11 ++++++----- 2 files changed, 8 insertions(+), 6 deletions(-) diff --git a/README.md b/README.md index 7178465..d611b24 100644 --- a/README.md +++ b/README.md @@ -5,7 +5,8 @@ allows an application to use an [stunnel](http://stunnel.org) to connect securel Heroku Redis. It is meant to be used in conjunction with other buildpacks. > [!WARNING] -> This buildpack is not compatible with `heroku-24`+ and is not needed for Redis 6+, which supports native TLS. +> This buildpack isn’t compatible with [`heroku-24`](article link?) and later. You don’t need this buildpack for Redis 6+, which supports native TLS. + > For more information, see [Securing Heroku Redis](https://devcenter.heroku.com/articles/heroku-redis#security-and-compliance). ## Usage diff --git a/bin/compile b/bin/compile index bc20a04..36809f2 100755 --- a/bin/compile +++ b/bin/compile @@ -14,11 +14,12 @@ BUILD_DIR=$1 BUILDPACK_DIR="$(dirname $(dirname $0))" if ! command -v stunnel4 > /dev/null; then - echo "! stunnel not detected! stunnel not supported on heroku-24+" - echo "! This buildpack is not required for Redis 6+. Remove this buildpack using:" - echo "! $ heroku buildpacks:remove heroku/redis" - echo "! And then follow the instructions for using Redis' native TLS support:" - echo "! https://devcenter.heroku.com/articles/heroku-redis#security-and-compliance" + + echo "! This buildpack uses stunnel, which isn’t supported on heroku-24 and later." + echo "! You don’t need this buildpack for Redis 6+. Remove it with the command:" + echo "! $ heroku buildpacks:remove heroku/redis" + echo "! To use Redis’ native TLS support, see https://devcenter.heroku.com/articles/heroku-redis#security-and-compliance." + exit 1 fi