Merge pull request #628 from Tencent/bugfix_logo_save_fail

Bugfix logo save fail
TencentBlueKing · Nov 9, 2021 · 4a58e7b · 4a58e7b
2 parents 6eb93e7 + d99f860
commit 4a58e7b
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 3 deletions.
diff --git a/paas2/VERSION b/paas2/VERSION
@@ -1 +1 @@
-2.13.12
+2.13.13
diff --git a/paas2/paas/saas/utils.py b/paas2/paas/saas/utils.py
@@ -289,12 +289,12 @@ def extract_logo_file(filename, path, saas_app_code):
             is_replace_applog = filecmp.cmp(logo_local_path, applogo_path)
 
         # replace saaslogo/*.png
-        with open(logo_local_path, "w") as logo_f:
+        with open(logo_local_path, "wb") as logo_f:
             logo_f.write(logo_content)
 
         # if user not defined it, the put into applogo
         if not os.path.exists(applogo_path) or is_replace_applog:
-            with open(applogo_path, "w") as logo_f:
+            with open(applogo_path, "wb") as logo_f:
                 logo_f.write(logo_content)
     except Exception:
         message = u"%s Unzip to get app.yml failed" % filename

diff --git a/paas2/release.md b/paas2/release.md
@@ -1,5 +1,8 @@
 Release Log
 ===============================
+# 2.13.13
+    - bugfix: smart logo save fail
+
 # 2.13.12
     - bugfix: bkauth_user table missing will 500 after upgrade to django 1.11.29