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

feat: ✨ add stacktrace and system infomation collector #94

Open
wants to merge 6 commits into
base: dev
Choose a base branch
from

Conversation

AstroAir
Copy link
Collaborator

The initial capture of anomalies and collection of information have been completed, but there are still the following issues:

  • Firstly, in terms of signal capture, I only dealt with the two most basic types and did not consider many more types
  • There is a certain degree of error in obtaining CPU usage, which seems to be a problem with the Windows API
  • The automatic installation of some libraries has not yet been implemented, but most platforms should come with this library or additional installation logic. I am not sure how to handle this more appropriately
  • The submitted code was only tested on the Windows platform, and the mingw64 environment exploded due to the use of a very special API for win32
  • There are still many functions that can be done, such as the packaging and collection of logs, which have not been implemented yet

@AstroAir
Copy link
Collaborator Author

There is still a part of the code that I haven't ported, which has a strong sense of rogue behavior and may be classified as malicious software

Copy link
Member

Choose a reason for hiding this comment

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

delete this file

修复了Crash.cc文件中的时间处理错误,使用了平台特定的时间处理函数来获取本地时间。
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.

2 participants