Skip to content

Truncate log message in NonDarwinLogger at 10.000 characters #1186

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

Merged
merged 1 commit into from
Apr 20, 2024

Conversation

ahoppen
Copy link
Member

@ahoppen ahoppen commented Apr 18, 2024

Truncate log message after 10.000 characters to avoid flooding the log with huge log messages (eg. from a sourcekitd response). 10.000 characters wasn't chosen because it seems to fit the result of most sourcekitd responses that are not generated interface or global completion results (which are a lot bigger).

@ahoppen ahoppen requested a review from bnbarham April 18, 2024 21:41
@ahoppen ahoppen requested a review from benlangmuir as a code owner April 18, 2024 21:41
@ahoppen
Copy link
Member Author

ahoppen commented Apr 18, 2024

@swift-ci Please test

Truncate log message after 10.000 characters to avoid flooding the log with huge log messages (eg. from a sourcekitd response). 10.000 characters wasn't chosen because it seems to fit the result of most sourcekitd responses that are not generated interface or global completion results (which are a lot bigger).
@ahoppen ahoppen force-pushed the truncate-non-darwing-log-message branch from f0adb54 to 8f74261 Compare April 20, 2024 04:32
@ahoppen
Copy link
Member Author

ahoppen commented Apr 20, 2024

@swift-ci Please test

@ahoppen
Copy link
Member Author

ahoppen commented Apr 20, 2024

@swift-ci Please test Windows

@ahoppen ahoppen enabled auto-merge April 20, 2024 05:20
@ahoppen ahoppen merged commit ce83fcb into swiftlang:main Apr 20, 2024
3 checks passed
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