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

Feature/fix msgpack exception formatter #2963

Conversation

eugene-doobu
Copy link
Member

@eugene-doobu eugene-doobu commented Oct 29, 2024

  • 닷넷 업그레이드 대처 이후 고장난 exception formatter를 고칩니다
  • Libplanet Exception에 대한 테스트 코드가 없어 임시 테스트 코드 하나(Libplanet_Exception_Serializable) 추가하였습니다. 관련해서 테스트 제대로 작성하려면 Libplaent의 Exception수정 필요해 보입니다
  • Libplanet Exception을 Serialize하는 기준을 모르겠습니다 혹시 아시는 분 있음 알려주세요
  • Exception Serialize시 직렬화 할 데이터 타입을 NineChroniclesResolverGetFormatterHelper의 FormatterMap에 추가해줘야 합니다.

@boscohyun 님의 커밋 c53349a 체리픽하여 추가하였습니다

#2803
https://github.com/dotnet/designs/blob/main/accepted/2020/better-obsoletion/binaryformatter-obsoletion.md

@eugene-doobu eugene-doobu added this to the v250.0.0 milestone Oct 29, 2024
@eugene-doobu eugene-doobu self-assigned this Oct 29, 2024
@eugene-doobu eugene-doobu linked an issue Oct 29, 2024 that may be closed by this pull request
@eugene-doobu
Copy link
Member Author

build-for-unity 에서 실패하고있는데 이거 job자체를 확인 부탁드려도 될까요?
is not 연산자 쓰면 에러나는 것도 그렇고.. 유니티에서 사용해도 문제 없는데 뭔가 버전이 잘못 셋팅 된 느낌이 있습니다
해당 lib9c 적용하고 유니티 빌드시 문제없이 잘 됩니다

Comment on lines 14 to 16
private static readonly Dictionary<Type, object> FormatterMap = new Dictionary<Type, object>()
private static readonly Dictionary<Type, object> FormatterMap = new()
Copy link
Contributor

@boscohyun boscohyun Oct 29, 2024

Choose a reason for hiding this comment

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

지금 lib9c를 사용하는 NineChronicles의 유니티 엔진 버전은 2021.3.37f1C# 9.0 버전을 지원하고 있습니다.
하지만 본 저장소에서 build-for-unity 스크립트가 .NET Standard 2.1 로 동작하고 있는 점 때문에 기본 언어 버전이 C# 8.0으로 동작하게 되는 데 원인이 있습니다.

main.yml 스크립트를 개선해서 <LangVersion>9</LangVersion> 설정을 추가해보는 게 좋겠어요.

Copy link
Contributor

Choose a reason for hiding this comment

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

이슈로 남겨뒀어요. #2964

@eugene-doobu
Copy link
Member Author

관련 내용으로 추가 이슈 생성하여 해당 pr은 머지해놓겠습니다
#2966
#2967

@eugene-doobu eugene-doobu merged commit 448d741 into planetarium:development Nov 1, 2024
8 checks passed
@eugene-doobu eugene-doobu deleted the feature/fix-msgpack-exception-formatter branch November 1, 2024 01:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

ExceptionFormatter 직렬화 에러 수정
3 participants