Improve logo validation during package uploading

[Ekaterina Chernova]

1) Enable logo to be not only in PNG format (jpeg should also need to be allowed at least)
2) Add restriction on logo size to be <= 500 Kb

[Rohini Choudhary (enthurohini)]

Hello Ekaterina Chernova,

I am unable to find the file where the logo validations are defined. Please help me out.

[Ekaterina Chernova (efedorova)]

Hi!

here it is https://github.com/openstack/murano/blob/29dd968da5a6307f46e4005b8b31416375ceb0b8/murano/packages/package_base.py#L116

please, contact me by mail or irc directly is you have any questions

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: master
Review: https://review.openstack.org/240080

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/240080
Committed: https://git.openstack.org/cgit/openstack/murano/commit/?id=a1e546a23abb3064e4f9e43f3fde7a6b9448bf15
Submitter: Jenkins
Branch: master

commit a1e546a23abb3064e4f9e43f3fde7a6b9448bf15
Author: Akanksha <email address hidden>
Date: Thu Oct 29 04:43:03 2015 +0530

    Improve logo validation during package uploading

    Used PIL to allow support for png, jpeg and gif file formats
    for upload. An exception is raised if the image size exceeds
    500 KiB.

    Change-Id: Ifd2b8bf689214022664b7bccf846c0de63cc0bfe
    Closes-Bug:1497257

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: master
Review: https://review.openstack.org/292026

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/292026
Committed: https://git.openstack.org/cgit/openstack/murano/commit/?id=b92b7e6add92ace1b8a06760feaab5187baf15ec
Submitter: Jenkins
Branch: master

commit b92b7e6add92ace1b8a06760feaab5187baf15ec
Author: Akanksha <email address hidden>
Date: Sat Mar 12 20:29:19 2016 +0530

    Adds a test for logo validation

    Added tests for new logo validation behaviour which
    restricts file type of the logo image uploaded.
    Also fix i18 import in package_base.py

    Closes-Bug: #1497257
    Related-Bug: #1560547
    Change-Id: I772e4b1fc56de86171938a536197b253a131645c

[Doug Hellmann (doug-hellmann)]

This issue was fixed in the openstack/murano 3.0.0.0b1 development milestone.