-
-
Notifications
You must be signed in to change notification settings - Fork 40
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
Unable to use get_columns_name_id as a non-superuser #31
Comments
Hi @jasjuang , thanks for the feedback. I'll look into this on the weekend. |
Fixed in version 0.2.14.1 from metabase_api import Metabase_API
mb = Metabase_API('https://...', 'username', 'password', is_admin=False) |
@vvaezian did something went wrong with the pip deployment for 0.2.14.1? After the upgrade, |
@jasjuang There was an issue in the setup file. Please upgrade to 0.2.14.2 (latest version) |
In the
get_columns_name_id
function, it checksfriendly_names_is_disabled
first before allowing us to use the function as seen in https://github.com/vvaezian/metabase_api_python/blob/master/metabase_api/metabase_api.py#L339. However, in thefriendly_names_is_disabled
function, it does aself.get('/api/setting')
as seen in https://github.com/vvaezian/metabase_api_python/blob/master/metabase_api/metabase_api.py#L372. This is causing a problem because according to https://github.com/metabase/metabase/blob/master/docs/api-documentation.md#setting,GET /api/setting/
is only available to superuser, so even thoughFriendly Table and Field Names
is already set to disable by the Admin, a non superuser is still unable to use theget_columns_name_id
function becauseself.get('/api/setting')
returnsFalse
for a non superuser. Is it possible to consider the scenarios for a non superuser?The text was updated successfully, but these errors were encountered: