You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
It appears however that the position matters to the underlying mysql binary, and arguments like --defaults-file need to come before additional arguments (like dbname or -e foo.sql or -A etc...)
# Clone Damon's branch:
git clone git@github.com:nr-dbuckwalter/shell-plugins.git -b fix-mysql-arg-order
# Build the mysql plugin locally
cd shell-plugins
make mysql/build
# Validate it got installed here:
$ ls -la ~/.config/op/plugins/local/
-rwxr-xr-x 1 mina staff 20390690 30 Nov 16:19 mysql
# Test it - you'll get an ugly warning due to the above setup till properly released
$ mysql --version
#############################################################################
# WARNING: 'mysql' is not from the official registry. #
# Only proceed if you are the developer of 'mysql'. #
# Otherwise, delete the file at /Users/mina/.config/op/plugins/local/mysql. #
#############################################################################
mysql Ver 8.2.0 for macos14.0 on arm64 (Homebrew)
Platform or tool
mysql
Desired or expected behavior
Invoking
mysql additional arguments
should work as if invoking nativemysql
binaryCurrent behavior
Supplying additional arguments while using the 1Password mysql shell plugin integration returns error:
Relevant log output
I investigated a bit, and given an invocation like
mysql dbname
The plugin prepares the temporary file and appends it to the CLI arguments like so:
/opt/homebrew/opt/mysql-client/bin/mysql dbname --defaults-file=/var/folders/hp/7hxtb9_968d3vj4q13z28vjc0000gn/T/1PasswordShellPlugins-413974828/my.cnf mysql: [ERROR] unknown variable 'defaults-file=/var/folders/hp/7hxtb9_968d3vj4q13z28vjc0000gn/T/1PasswordShellPlugins-413974828/my.cnf'.
It appears however that the position matters to the underlying
mysql
binary, and arguments like--defaults-file
need to come before additional arguments (likedbname
or-e foo.sql
or-A
etc...)This order works:
op CLI version
2.23.0
The text was updated successfully, but these errors were encountered: