ome / omero-py

Python project containing Ice remoting code for OMERO
https://www.openmicroscopy.org/omero
GNU General Public License v2.0
22 stars 32 forks source link

omero db password should be a string #175

Closed manics closed 4 years ago

manics commented 4 years ago

Closes https://github.com/ome/omero-py/issues/112

Testing: omero db password should output a valid psql string

joshmoore commented 4 years ago

Reminder to self to set OMERODIR:

(py36) /tmp $omero db password
Please enter password for OMERO root user:
Please re-enter password for OMERO root user:
PasswordUtil failed: b'Error: Could not find or load main class ome.security.auth.PasswordUtil\n'

Previous behavior:

(py36) /tmp $export OMERODIR=/opt/ome9/dist/
(py36) /tmp $omero db password
Please enter password for OMERO root user:
Please re-enter password for OMERO root user:
UPDATE password SET hash = 'b'kYMkc9SyhupHyJP1Cl2Alg=='' WHERE experimenter_id = 0;

With this PR:

(py36) /tmp $pip install -U https://github.com/manics/omero-py/archive/omero-db-password-str.zip
Collecting https://github.com/manics/omero-py/archive/omero-db-password-str.zip
  Using cached https://github.com/manics/omero-py/archive/omero-db-password-str.zip
Requirement already satisfied, skipping upgrade: future in /opt/anaconda/envs/py36/lib/python3.6/site-packages (from omero-py==5.6.0) (0.18.1)
Requirement already satisfied, skipping upgrade: numpy in /opt/anaconda/envs/py36/lib/python3.6/site-packages (from omero-py==5.6.0) (1.17.2)
Requirement already satisfied, skipping upgrade: Pillow in /opt/anaconda/envs/py36/lib/python3.6/site-packages (from omero-py==5.6.0) (6.2.1)
Requirement already satisfied, skipping upgrade: zeroc-ice<3.7,>=3.6.4 in /opt/anaconda/envs/py36/lib/python3.6/site-packages (from omero-py==5.6.0) (3.6.5)
Building wheels for collected packages: omero-py
  Building wheel for omero-py (setup.py) ... done
  Created wheel for omero-py: filename=omero_py-5.6.0-cp36-none-any.whl size=2788506 sha256=d375d07ebf29270d5bc1cf7d7d0f2a7d563be435c17862a58bd9da58e2e3aead
  Stored in directory: /Users/jamoore/Library/Caches/pip/wheels/85/0a/aa/ab03d98b28a0642371a3ff0371213900699d846bbb0cb78fad
Successfully built omero-py
Installing collected packages: omero-py
  Found existing installation: omero-py 5.6.0
    Uninstalling omero-py-5.6.0:
      Successfully uninstalled omero-py-5.6.0
Successfully installed omero-py-5.6.0
(py36) /tmp $omero db password
Please enter password for OMERO root user:
Please re-enter password for OMERO root user:
UPDATE password SET hash = 'kYMkc9SyhupHyJP1Cl2Alg==' WHERE experimenter_id = 0;

:+1: Propose immediate release as 5.6.1.

joshmoore commented 4 years ago
(py36) /opt/omero-py $omero db password
Please enter password for OMERO root user:
Please re-enter password for OMERO root user:
PasswordUtil failed: Error: Could not find or load main class ome.security.auth.PasswordUtil

(py36) /opt/omero-py $

:+1: Any thoughts on the release, anyone?

joshmoore commented 4 years ago

Discussion of whether or not to release an omero-py 5.6.1 moved to https://github.com/ome/omero-py/pull/176