diff --git a/BUILD/build_deb_local.sh b/BUILD/build_deb_local.sh index 897371fc0..0640673ef 100755 --- a/BUILD/build_deb_local.sh +++ b/BUILD/build_deb_local.sh @@ -76,5 +76,5 @@ if [ "$BUILD_RABBITMQ" == "yes" ] ; then cp _build/*.deb . cp *.deb .. fi -sudo dpkg -i ../maxscale*.dev +sudo dpkg -i ../maxscale*.deb set +x diff --git a/maxscale-system-test/CMakeLists.txt b/maxscale-system-test/CMakeLists.txt index 2dc71682e..859bbaca9 100644 --- a/maxscale-system-test/CMakeLists.txt +++ b/maxscale-system-test/CMakeLists.txt @@ -1019,6 +1019,10 @@ add_test_executable(mxs1926_killed_server.cpp mxs1926_killed_server mxs1926_kill # https://jira.mariadb.org/browse/MXS-1932 add_test_executable(mxs1932_hidden_cnf.cpp mxs1932_hidden_cnf replication LABELS REPL_BACKEND) +# MXS-1958: Users with insert-only privileges don't work +# https://jira.mariadb.org/browse/MXS-1958 +add_test_executable(mxs1958_insert_priv.cpp mxs1958_insert_priv replication LABELS REPL_BACKEND) + configure_file(templates.h.in templates.h @ONLY) include(CTest) diff --git a/maxscale-system-test/Documentation/RELEASE.md b/maxscale-system-test/Documentation/RELEASE.md index 7e97c0458..53cd40ab9 100644 --- a/maxscale-system-test/Documentation/RELEASE.md +++ b/maxscale-system-test/Documentation/RELEASE.md @@ -26,6 +26,10 @@ also the version number being changed. So, ensure that the `maxscale-x.y.z-ttN` has been created and pushed to the repository. +``` +git push origin refs/tags/maxscale-x.y.z-ttN +``` + **NOTE** The tentative suffix - `-ttN` - is **only** used when specifying the tag for the build, it must **not** be present in any other names or paths. @@ -140,7 +144,7 @@ the final tag ``` $ git checkout maxscale-x.y.z-ttN $ git tag -a -m "Tag for MaxScale x.y.z" maxscale-x.y.z -$ git push --tags origin +$ git push origin refs/tags/maxscale-x.y.z ``` and remove the tentative tag(s) ``` diff --git a/maxscale-system-test/mxs1958_insert_priv.cpp b/maxscale-system-test/mxs1958_insert_priv.cpp new file mode 100644 index 000000000..44d5a36eb --- /dev/null +++ b/maxscale-system-test/mxs1958_insert_priv.cpp @@ -0,0 +1,25 @@ + +#include "testconnections.h" + +int main(int argc, char** argv) +{ + TestConnections test(argc, argv); + + test.repl->connect(); + execute_query(test.repl->nodes[0], "CREATE USER 'insert_only'@'%%' IDENTIFIED BY 'insert_only'"); + execute_query(test.repl->nodes[0], "CREATE DATABASE insert_db"); + execute_query(test.repl->nodes[0], "CREATE TABLE insert_db.t1(id INT)"); + execute_query(test.repl->nodes[0], "GRANT INSERT ON insert_db.t1 TO 'insert_only'@'%%'"); + test.repl->sync_slaves(); + + MYSQL* conn = open_conn(test.maxscales->rwsplit_port[0], test.maxscales->IP[0], "insert_only", "insert_only", false); + test.assert(mysql_errno(conn) == 0, "User without SELECT privileges should be allowed to connect"); + mysql_close(conn); + + execute_query(test.repl->nodes[0], "DROP USER 'insert_only'@'%%'"); + execute_query(test.repl->nodes[0], "DROP DATABASE insert_db"); + test.repl->sync_slaves(); + test.repl->disconnect(); + + return test.global_result; +} diff --git a/server/core/gateway.cc b/server/core/gateway.cc index a4b42f5fb..02edd9c24 100644 --- a/server/core/gateway.cc +++ b/server/core/gateway.cc @@ -1899,12 +1899,9 @@ int main(int argc, char **argv) goto return_main; } - MXS_NOTICE("MariaDB MaxScale %s started", MAXSCALE_VERSION); + MXS_NOTICE("MariaDB MaxScale %s started (Commit: %s)", MAXSCALE_VERSION, MAXSCALE_COMMIT); MXS_NOTICE("MaxScale is running in process %i", getpid()); -#ifdef SS_DEBUG - MXS_NOTICE("Commit: %s", MAXSCALE_COMMIT); -#endif cleanup_old_process_datadirs(); if (!cnf->config_check) {