search

kurotu / VRCQuestTools

Unity editor extension to support uploading VRChat avatars for Oculus Quest
https://kurotu.github.io/VRCQuestTools/
MIT License
218 stars 15 forks source link

更新通知がエラーになりテストが出来ない場合がある（更新確認機能は削除すべき） #12

Closed Narazaka closed 1 year ago

Narazaka commented 1 year ago

説明

再現手順

期待する動作

スクリーンショット 必要に応じてスクリーンショットを添付してください。

コンソールログ

6bfa589>:0)
KRT.VRCQuestTools.Services.GitHubService+<GetLatestRelease>d__3.MoveNext () (at Packages/com.github.kurotu.vrc-quest-tools/Editor/Services/GitHubService.cs:40)
--- End of stack trace from previous location where exception was thrown ---
System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw () (at <eae584ce26bc40229c1b1aa476bfa589>:0)
System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess (System.Threading.Tasks.Task task) (at <eae584ce26bc40229c1b1aa476bfa589>:0)
System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification (System.Threading.Tasks.Task task) (at <eae584ce26bc40229c1b1aa476bfa589>:0)
System.Runtime.CompilerServices.TaskAwaiter.ValidateEnd (System.Threading.Tasks.Task task) (at <eae584ce26bc40229c1b1aa476bfa589>:0)
System.Runtime.CompilerServices.TaskAwaiter`1[TResult].GetResult () (at <eae584ce26bc40229c1b1aa476bfa589>:0)
KRT.VRCQuestTools.Automators.UpdateCheckerAutomator+<>c__DisplayClass4_0+<<CheckForUpdates>b__0>d.MoveNext () (at Packages/com.github.kurotu.vrc-quest-tools/Editor/Automators/UpdateCheckerAutomator.cs:49)
UnityEngine.Debug:LogException(Exception)
KRT.VRCQuestTools.Automators.<<CheckForUpdates>b__0>d:MoveNext() (at Packages/com.github.kurotu.vrc-quest-tools/Editor/Automators/UpdateCheckerAutomator.cs:98)
System.Runtime.CompilerServices.AsyncTaskMethodBuilder`1:SetException(Exception)
KRT.VRCQuestTools.Services.<GetLatestRelease>d__3:MoveNext() (at Packages/com.github.kurotu.vrc-quest-tools/Editor/Services/GitHubService.cs:42)
System.Runtime.CompilerServices.AsyncTaskMethodBuilder`1:SetResult(Int32)
Mono.Net.Security.<StartOperation>d__58:MoveNext()
System.Runtime.CompilerServices.AsyncTaskMethodBuilder`1:SetResult(AsyncProtocolResult)
Mono.Net.Security.<StartOperation>d__23:MoveNext()
System.Runtime.CompilerServices.AsyncTaskMethodBuilder:SetResult()
Mono.Net.Security.<ProcessOperation>d__24:MoveNext()
System.Runtime.CompilerServices.AsyncTaskMethodBuilder`1:SetResult(Nullable`1)
Mono.Net.Security.<InnerRead>d__25:MoveNext()
System.Runtime.CompilerServices.AsyncTaskMethodBuilder`1:SetResult(Int32)
Mono.Net.Security.<InnerRead>d__66:MoveNext()
System.Threading._ThreadPoolWaitCallback:PerformWaitCallback()

使用アセット 使用したアバター/衣装/ツールなどのURLを記載してください。

環境情報

追加情報 ほかに必要な情報があれば記載してください。

kurotu commented 1 year ago

症状からすると最新バージョン取得時の例外処理が正しく動いていないように見えるので確認します。 (そもそも再生モードで更新確認をする必要はないので動作自体を止めてしまった方が早いような気もしますが。)

また、念のため確認させてください。テストができなくなるのは VRMSpringBone と VRCPhysBone のどちらでしょうか？

更新確認機能については、できれば残しておきたいと考えています。 VCCでパッケージの更新自体は楽になりましたが、更新が存在するかどうかは Manage Project を開かないとわからず更新に気付いてもらう機会としては弱いと思っているためです。 通常の開発ツールではここまでうるさいものは全く必要ないと思いますが、VRCの場合の多数のユーザーからするとわざわざVCCを見てもらうのは難しいとも思っています。

Narazaka commented 1 year ago

VRMSpringBone ですね。失礼しました。 全体で止めないにしても再生モードでやる必要は無いと思うので、止めてしまうのが良いと思います。

Narazaka commented 1 year ago

また勝手にGithub APIのquotaが減るのもどうかと思う（普通にテストしてる回数で403も出る）ので、やるにしても出来れば別のURLで判定して欲しいというのもありそうです。

kurotu commented 1 year ago

必ず403が変える状況を用意してみましたが、VRMSpringBoneのテストができない点は再現できませんでした。 一方で指摘いただいた点自体は改善できる部分なので、v1.12.0 で対応しました。