From f43888bc2c4c6f7a96cc5da8e3d7f7346985d8ec Mon Sep 17 00:00:00 2001
From: longjuan <769022681@qq.com>
Date: Tue, 14 Mar 2023 10:58:17 +0800
Subject: [PATCH] fix: RSA key cannot be saved normally in the Windows
 environment (#3510)
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

<!--  Thanks for sending a pull request!  Here are some tips for you:
1. 如果这是你的第一次，请阅读我们的贡献指南：<https://github.com/halo-dev/halo/blob/master/CONTRIBUTING.md>。
1. If this is your first time, please read our contributor guidelines: <https://github.com/halo-dev/halo/blob/master/CONTRIBUTING.md>.
2. 请根据你解决问题的类型为 Pull Request 添加合适的标签。
2. Please label this pull request according to what type of issue you are addressing, especially if this is a release targeted pull request.
3. 请确保你已经添加并运行了适当的测试。
3. Ensure you have added or ran the appropriate tests for your PR.
-->

#### What type of PR is this?
/kind bug
/area core
<!--
添加其中一个类别：
Add one of the following kinds:

/kind bug
/kind cleanup
/kind documentation
/kind feature
/kind improvement

适当添加其中一个或多个类别（可选）：
Optionally add one or more of the following kinds if applicable:

/kind api-change
/kind deprecation
/kind failing-test
/kind flake
/kind regression
-->

#### What this PR does / why we need it:
修复Windows下开发环境不能正常保存Rsa密钥
#### Which issue(s) this PR fixes:

<!--
PR 合并时自动关闭 issue。
Automatically closes linked issue when PR is merged.

用法：`Fixes #<issue 号>`，或者 `Fixes (粘贴 issue 完整链接)`
Usage: `Fixes #<issue number>`, or `Fixes (paste link of issue)`.
-->
Fixes https://github.com/halo-dev/halo/issues/3497

#### Special notes for your reviewer:
windows不支持posix标准，也无法精细控制用户访问权限，因此在不支持posix的系统直接不设置权限。
https://stackoverflow.com/questions/21541455/how-to-check-if-the-os-is-posix-compliant
#### Does this PR introduce a user-facing change?

<!--
如果当前 Pull Request 的修改不会造成用户侧的任何变更，在 `release-note` 代码块儿中填写 `NONE`。
否则请填写用户侧能够理解的 Release Note。如果当前 Pull Request 包含破坏性更新（Break Change），
Release Note 需要以 `action required` 开头。
If no, just write "NONE" in the release-note block below.
If yes, a release note is required:
Enter your extended release note in the block below. If the PR requires additional action from users switching to the new release, include the string "action required".
-->

```release-note
None
```
---
 .../authentication/login/impl/RsaKeyService.java         | 9 +++++++--
 1 file changed, 7 insertions(+), 2 deletions(-)

diff --git a/src/main/java/run/halo/app/security/authentication/login/impl/RsaKeyService.java b/src/main/java/run/halo/app/security/authentication/login/impl/RsaKeyService.java
index f2f47598f..46b525336 100644
--- a/src/main/java/run/halo/app/security/authentication/login/impl/RsaKeyService.java
+++ b/src/main/java/run/halo/app/security/authentication/login/impl/RsaKeyService.java
@@ -6,6 +6,7 @@ import static java.nio.file.attribute.PosixFilePermission.OWNER_WRITE;
 
 import java.io.IOException;
 import java.nio.ByteBuffer;
+import java.nio.file.FileSystems;
 import java.nio.file.Files;
 import java.nio.file.Path;
 import java.nio.file.attribute.PosixFilePermissions;
@@ -123,8 +124,12 @@ public class RsaKeyService implements CryptoService {
             return Mono.fromRunnable(() -> {
                 try {
                     Files.createDirectories(path.getParent());
-                    Files.createFile(path,
-                        PosixFilePermissions.asFileAttribute(Set.of(OWNER_READ, OWNER_WRITE)));
+                    if (FileSystems.getDefault().supportedFileAttributeViews().contains("posix")) {
+                        Files.createFile(path,
+                            PosixFilePermissions.asFileAttribute(Set.of(OWNER_READ, OWNER_WRITE)));
+                    } else {
+                        Files.createFile(path);
+                    }
                 } catch (IOException e) {
                     // ignore the error
                     log.warn("Failed to create file for {}", path, e);