Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

添加服务端解决方案 #206

Merged
merged 2 commits into from
Feb 2, 2024
Merged

添加服务端解决方案 #206

merged 2 commits into from
Feb 2, 2024

Conversation

bingxin666
Copy link
Member

bingling-sama and others added 2 commits September 2, 2023 22:55
服务端没有权限写入所在文件夹
@bingxin666 bingxin666 linked an issue Feb 2, 2024 that may be closed by this pull request
at java.base/java.io.RandomAccessFile.<init>(RandomAccessFile.java:259)
at java.base/java.io.RandomAccessFile.<init>(RandomAccessFile.java:213)
at java.base/java.io.RandomAccessFile.<init>(RandomAccessFile.java:127)
// 该处虽然只是表示了无权限写入 latest.log ,但一般情况下这意味着服务端没有所在文件夹的写入权限,除非您单独为 latest.log 设置了操作权限
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

这里应该再加一个 java.nio.file.AccessDeniedException

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

#204 是根据这里添加的文档,无必要,因为有可能到显示java.nio.file.AccessDeniedException之前就已经爆炸了......相反lastest.log无法写入是最早报错的,因此基于最早报错填写的示例

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

不不不,服务端崩溃基本都是做后一个错误导致的

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

前面的错误如果很重要,服务端就不会运行到抛出后面错误的代码,而是会直接退出

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

前面的错误如果很重要,服务端就不会运行到抛出后面错误的代码,而是会直接退出

主要是latest.log无法写入这个操作相当有辨识度,毕竟不会有人闲的去改latest.log的权限
相反,java.nio.file.AccessDeniedException这个报错出现的原因会更加广泛,在客户端和服务端都可能概率出现
这个错误可以考虑单独添加在客户端和服务端的通用问题一栏中

Copy link
Member

@zyxkad zyxkad left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

那我就先Approve了,你要是觉得我说的对就再改一下

@bingxin666 bingxin666 merged commit 1e42cf7 into GlobeMC:dev Feb 2, 2024
3 of 4 checks passed
@bingxin666 bingxin666 deleted the main branch February 2, 2024 15:26
Big-Cake-jpg added a commit that referenced this pull request Feb 3, 2024
* [add] 添加 Forge 其它崩溃原因 (#148)

* ✨ update: 再次更新Analyzer, 使用MCLA API并修复一些错误 #140 (#149)

* use MCLA web assembly module as the first analyzer

* fixed styles

* use vue-router instead of window.href

* fix overflow button animation

* nice! cdn fixed; makes the cdn url configurable

* makes the cdn cache limit larger

* we don't need cdn when redirect to github

* add MCLA error catcher

* use newer API to read file

* Update docs/.vitepress/theme/components/Analyzer.vue

* fixed zip read

* finally fixed expand transition

* use requestAnimationFrame instead of setTimeout(0)

* remove an extra argument

* move mcla into another file

* export mcla github database prefix

* switching to ts

* revert back the origin error compares

* fix isBtnDisabled

* add analyzing icon

* fix expand transition

* finish to show multiple parsed errors & solutions

* change type names

* revert some docs changed by accident

* support gzip

* support zlib

* remove unused variable

* fixed lint issues

* ignore some specific tarball paths

* run docs:format

* fixed hr display

* fix button hint when error is unrecord

* fix analyze dropped files

* use iterator instead of array

---------

Co-authored-by: Big_Cake <2577117205@qq.com>

* fix upload button disabled forever issue

* support version control for MCLA

* Support read & analysis multiple files

* fix a debug log

* remove nosense logs

* run eslint

* remove the extra `window.` prefix

* bump MCLA to v0.4.9

* ✨ feat: vitepress plugins

nprogress, back-to-top, codeplock fold, viewerjs

* bump mcla to v0.4.13 (#155)

* 支持Web worker, 使用多线程分析保证浏览器渲染流畅 (#156)

* Bump MCLA to v0.4.18

* auto run docs:format in pr-check.yml

* update pr-check.yml

* update pr-check.yml

* format test

* use docs:lint but not docs:format

* 🌈 style(css): add `vp-c-brand-1`

* add @typescript-eslint/parser

* run docs:format

* fix lint

* 更名

* move src/mcla_worker.js -> public/scripts/mcla_worker.js

* run pnpm run docs:format

* fix pr-check

* fix format

* let the format hint output to stderr

* ✨ update: 更新 Java 链接 (#162)

* 更新java链接

* Update mods.md

* Update system.md

* 更改java下载为cdn

* ✨ feat(husky): add git hooks

add husky
add zhlint
format & lint everything

now we can run `docs:zhlint` to format Chinese docs
also automatically run before commiting changes

* ✨ update: 添加更多 Mod 问题 (#165)

* 添加 Forge 其它崩溃原因

* 添加 mod 问题

* 完善 Mod 问题

* 使用 Mod 全称

* 添加 mod 链接

* 🛠️ fix: 修复 zhlint 和 Actions (#166)

* add pre-commit

* fix zhlint (but not really fixed)

* fix the filename `.DS_Store`

* update contribute/contributing.md

* Update CRASH-UPLOAD.yml

当日志很长时,用户无法粘贴到textarea或者无法提交超出长度的issue, 所以加了个markdown格子让用户可以直接拖拽上传文件

* Update CRASH-UPLOAD.yml

Fix serval issue

* Add a header for log file in CRASH-UPLOAD.yml

* Revert "Update CRASH-UPLOAD.yml"

* Update CRASH-UPLOAD.yml

不加 render 的 textarea 就是 markdown 格子

* 🛠️ bump MCLA to v0.4.22 (#182)

* bump MCLA to v0.4.21

* use vue-tsc for check .vue file, and fix serval errors

* fix umami defination for eslint

* bump MCLA to v0.4.22

* fix style

* Add LogViewer and Github OAuth

* revert CRASH-UPLOAD.yml

* 添加错误: 在服务端运行仅客户端模组
close #181

* bumb MCLA to v0.4.23

* format use space

* reforged auth flow
换了个更麻烦的登录流程(雾)
但是现在测试的时候不用在github里把redirect_uri改来改去了(啊这)

* remove the old token when it was expired

* bump github ci node version to 20

* bump node version to 20 for pr-check

* Update pr-check.yml

改了一下名称
试图刷新cloudfare

* add MCLA loading progress (#190)

* add MCLA loading progress
fix most tsc problems
resolve Vue warn "Missing required prop *"

* ignore TS2353 so far, wait for T-miracle/vitepress-plugin-comment-with-giscus#10

* format code

* Add Utf-8崩溃

* Fix tsconfigs for webworker scripts

* use @ instead of relative paths

* 加入死马的百度的验证 验证完可以删

* fix: remove unused @ts-expect-error

* 添加服务端解决方案 (#206)

服务端没有权限写入所在文件夹

Co-authored-by: bingling_sama <bingling_sama@qq.com>

* 大量的修改,我觉得这堆可以推生产分支了
更新依赖、修复 PWA 查询字符串问题、深浅色模式切换使用 View Transitions API、完善部分词条本地化、启用 cleanUrls、资源文件强缓存、图片懒加载、sitemap 自动生成、修改 Shiki 主题、显示行号。

* 少推了一个东西

* 更新用语

---------

Co-authored-by: 柚子柚子 <youzi5201086@163.com>
Co-authored-by: Kevin Z <zyxkad@gmail.com>
Co-authored-by: 233355607 <81961962+2623684696@users.noreply.github.com>
Co-authored-by: 思遥方 <92374733+Seayay@users.noreply.github.com>
Co-authored-by: 2623684696 <qw23355607@outlook.com>
Co-authored-by: bingling-sama <bingling_sama@qq.com>
Co-authored-by: Pigeon0v0 <60414767+Pigeon0v0@users.noreply.github.com>
Co-authored-by: Enlysure <asd2376947530@gmail.com>
Co-authored-by: Rovniced <52170460+Rovniced@users.noreply.github.com>
Co-authored-by: bingxin666 <73421892+bingxin666@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

📄[UPLOAD-CRASH] 服务端权限问题
4 participants