Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

feat: split long logs into multiple instead of truncating #177

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

Draft
rigor789 wants to merge 2 commits into main
base: main
Choose a base branch
Loading
from feat/large-log-chunking

Conversation

Copy link
Member

@rigor789 rigor789 commented Jul 25, 2022
edited
Loading

NSLog has a 1024 character limit, and anything beyond is truncated with <...>. This PR aims to split longer logs into multiple NSLog calls when necessary.

This is already possible on Android by setting the maxLogcatObjectSize boolean under the android key in the config.

The goal is to add a similar config option for iOS as well.

Todos:

  • make the max length configurable
  • add the new config to the type definitions
  • check if we need to use ~1000 or if we can bump the max length to 1024 or 1023 before it actually truncates...

closes NativeScript/NativeScript#8657

Copy link
Contributor

@rigor789 will the result of the splitted log be on the same line in the console? Like let s say it is a JSON.stringify which is splitted. Will you be able to copy it, paste it as a correct JSON string?

Copy link
Contributor

@rigor789 i am intersted in testing this. Any chance to have an alpha runtime with this?

Copy link
Collaborator

@farfromrefug once the build finishes you can get the .tgz from the artifacts here: https://github.com/NativeScript/ios/actions/runs/5456662035

farfromrefug and rigor789 reacted with thumbs up emoji

Copy link
Member Author

rigor789 commented Jul 4, 2023

Thanks @edusperoni 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Add an option to print full log.

AltStyle によって変換されたページ (->オリジナル) /