iwayvietnam / zimbra-multi-tenancy

Zimbra Multi-tenancy
GNU Affero General Public License v3.0
21 stars 12 forks source link

Account mới không được khởi tạo thành công trên server Zimbra bằng cấp Tenant #37

Closed thangnguyennang closed 9 years ago

thangnguyennang commented 9 years ago

Khi đăng nhập cấp Tenant để khởi tạo account mới, sau khi ấn nút "Save" để hoàn tất thao tác thì kiểm tra trên server Zimbra không thấy xuất hiện account mới được khởi tạo (account mới chỉ được khởi tạo trên module).

nguyennv commented 9 years ago

Test với bản zimbra 7.x thì vẫn thêm được account vào zimbra. Có thể là do lỗi phiên bản zimbra. Enable debug và view log để xem nguyên nhân lỗi

thangnguyennang commented 9 years ago

Có khả năng lớn lỗi liên quan đến phiên bản Zimbra (sẽ test thêm với phiên bản Zimbra 7.x). Tuy nhiên, với môi trường test module hiện tại thì lỗi vẫn tồn tại (đã bổ sung ảnh thông báo lỗi đính kèm):

thangnguyennang commented 9 years ago

Với môi trường test đề cập như trên, nếu thay đổi cách khởi tạo tài khoản mới từ bằng tay sang theo lô (khởi tạo nhiều tài khoản mới cùng 1 lúc) thì không bị lỗi (tài khoản mới được khởi tạo thành công cả trên module và trên server Zimbra). Như vậy, lỗi đã report không hoàn toàn liên quan đến phiên bản Zimbra sử dụng.

Trường hợp khởi tạo tài khoản mới thành công bằng cách khởi tạo tài khoản theo lô này lại phát sinh lỗi khác: Khi cập nhật password mới cho tài khoản mới được khởi tạo thì password mới không có hiệu lực đăng nhập trên server Zimbra (chỉ đăng nhập được bình thường theo password cũ).

thangnguyennang commented 9 years ago

Lỗi được xác định liên quan đến policy cho password được thiết lập mặc định trong COS của Zimbra. Vì module chưa có khả năng check tính hợp lệ của password mỗi khi khởi tạo tài khoản mới bằng tay/theo lô (password phải được đối chiếu với policy cho password được thiết lập mặc định trong COS), nên việc xử lý khởi tạo tài khoản mới sẽ bị lỗi (phía Zimbra không chấp nhận password truyền vào từ module do chưa đúng yêu cầu policy). Trước mắt, có thể xử lý lỗi theo hướng sau: Module sẽ bổ sung tính năng check tính hợp lệ của password (tự động đối chiếu với policy cho password được thiết lập mặc định trong COS) trước khi cho phép người dùng khai báo các thông tin tiếp theo trong quá trình khởi tạo tài khoản mới. Theo đó, nếu người dùng khai báo pasword chưa đúng qui định, module sẽ hiển thị thông báo lỗi để người dùng khai báo lại password (ví dụ policy cho password: độ dài password tối thiểu phải 6 ký tự trong đó có tối thiểu 1 ký tự viết hoa, 1 ký tự viết thường, và 1 số).

Ngoài ra, cũng cần lưu ý việc cập nhật một số trường (password, full name, work title...) trên module để làm sao có hiệu lực song song trên server Zimbra.

nguyennv commented 9 years ago

Đã thêm tính năng kiểm tra tính hợp lệ của account khi add/update vào zimbra server. Khi tài khoản/ mật khẩu không hợp lệ. ZMT sẽ hiển thị thông báo được nhận về từ phía zimbra server.

thangnguyennang commented 9 years ago

Phiên bản mới nhất của module không hoạt động đúng yêu cầu (tính năng kiểm tra tính hợp lệ của account khi add/update vào server Zimbra không work) khi khởi tạo account mới cả bằng tay và theo lô (xem thông tin chi tiết lỗi tại issue #38).

nguyennv commented 9 years ago

Đã fix lỗi sync account to zimbra server khi import account. Còn lỗi tạo account bằng tay thì ko test ra được lỗi (có thể là do chưa clear cache khi update code nên vẫn tồn tại lỗi cũ). Lưu ý là khi update cần phải clear cache tại Configration->Performance để Drupal update lại code mới nhất.

thangnguyennang commented 9 years ago

Đã có thể khởi tạo account mới thành công cả bằng tay và theo lô với phiên bản code mới nhất (Tested trên 2 phiên bản Zimbra 7.x và 8.x).