Add deprecation warnings to "docker-entrypoint.sh"

Author: tianon

docker-entrypoint.sh

@@ -179,7 +179,8 @@ for conf in "${!configDefaults[@]}"; do
 done
 
 # if long and short hostnames are not the same, use long hostnames
-if [ "$(hostname)" != "$(hostname -s)" ]; then
+if [ -z "${RABBITMQ_USE_LONGNAME:-}" ] && [ "$(hostname)" != "$(hostname -s)" ]; then
+	echo >&2 "WARNING: relying on 'docker-entrypoint.sh' implied 'RABBITMQ_USE_LONGNAME=true' setting, which will be removed in a future release! (https://github.com/docker-library/rabbitmq/pull/424)"
 	: "${RABBITMQ_USE_LONGNAME:=true}"
 fi
 
@@ -195,6 +196,10 @@ if [ "${RABBITMQ_ERLANG_COOKIE:-}" ]; then
 		echo "$RABBITMQ_ERLANG_COOKIE" > "$cookieFile"
 	fi
 	chmod 600 "$cookieFile"
+
+	# TODO show warning about use of RABBITMQ_ERLANG_COOKIE variable ??
+	#  - https://github.com/docker-library/rabbitmq/pull/424#discussion_r459983266
+	#  - https://github.com/rabbitmq/rabbitmq-prometheus/commit/83cff5a4b80d4683a6a0a32db979ca00ec1774b3
 fi
 
 configBase="${RABBITMQ_CONFIG_FILE:-/etc/rabbitmq/rabbitmq}"
@@ -392,6 +397,13 @@ if [ "$1" = 'rabbitmq-server' ] && [ "$shouldWriteConfig" ]; then
 			rabbit_set_config 'load_definitions' "$managementDefinitionsFile"
 		fi
 	fi
+
+	echo >&2 "WARNING: 'docker-entrypoint.sh' generated/modified the RabbitMQ configuration file, which will no longer happen in a future release! (https://github.com/docker-library/rabbitmq/pull/424)"
+	echo >&2
+	echo >&2 "Generated end result, for reference:"
+	echo >&2 "------------------------------------"
+	cat >&2 "$newConfigFile"
+	echo >&2 "------------------------------------"
 fi
 
 combinedSsl='/tmp/rabbitmq-ssl/combined.pem'
@@ -403,6 +415,8 @@ if [ "$haveSslConfig" ] && [[ "$1" == rabbitmq* ]] && [ ! -f "$combinedSsl" ]; t
 		cat "$RABBITMQ_SSL_KEYFILE"
 	} > "$combinedSsl"
 	chmod 0400 "$combinedSsl"
+
+	echo >&2 "WARNING: relying on 'docker-entrypoint.sh' generating a combined SSL file, which will no longer happen in a future release! (https://github.com/docker-library/rabbitmq/pull/424)"
 fi
 if [ "$haveSslConfig" ] && [ -f "$combinedSsl" ]; then
 	# More ENV vars for make clustering happiness
@@ -412,6 +426,8 @@ if [ "$haveSslConfig" ] && [ -f "$combinedSsl" ]; then
 	sslErlArgs="-pa $ERL_SSL_PATH -proto_dist inet_tls -ssl_dist_opt server_certfile $combinedSsl -ssl_dist_opt server_secure_renegotiate true client_secure_renegotiate true"
 	export RABBITMQ_SERVER_ADDITIONAL_ERL_ARGS="${RABBITMQ_SERVER_ADDITIONAL_ERL_ARGS:-} $sslErlArgs"
 	export RABBITMQ_CTL_ERL_ARGS="${RABBITMQ_CTL_ERL_ARGS:-} $sslErlArgs"
+
+	# (no WARNING here, because this will only happen if we already displayed the "combined SSL file" WARNING above, so it would be redundant)
 fi
 
 exec "$@"