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

Emoji in logs not parsed correctly #1879

Closed
5 tasks done
Sebclem opened this issue Jun 21, 2023 · 4 comments · Fixed by #1874
Closed
5 tasks done

Emoji in logs not parsed correctly #1879

Sebclem opened this issue Jun 21, 2023 · 4 comments · Fixed by #1874
Labels
bug Something isn't working

Comments

@Sebclem
Copy link

Sebclem commented Jun 21, 2023

Component

server, web-ui

Describe the bug

Since the migration to the new login system, all emoji en logs are not parsed correctly :
image
Same issue with text generated by toilet:
image

Here the entrypoint of my plugin if you need to check what it does : https://git.sebclem.fr/sebclem/woodpecker-ansible-runner/src/branch/master/entrypoint

System Info

{
  "source": "https://github.com/woodpecker-ci/woodpecker",
  "version": "next-6ab9f802"
}

Additional context

No response

Validations

  • Read the Contributing Guidelines.
  • Read the docs.
  • Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
  • Checked that the bug isn't fixed in the next version already [https://woodpecker-ci.org/faq#which-version-of-woodpecker-should-i-use]
  • Check that this is a concrete bug. For Q&A join our Discord Chat Server or the Matrix room.
@Sebclem Sebclem added the bug Something isn't working label Jun 21, 2023
@lafriks
Copy link
Contributor

lafriks commented Jun 21, 2023

was it working correctly before?

@Sebclem
Copy link
Author

Sebclem commented Jun 21, 2023

I don't have screenshots but yes was working before, i think it's related to #1802 but i'm not sure

@anbraten
Copy link
Member

Unicode chars are currently not working: #1874 fixes it

@Sebclem
Copy link
Author

Sebclem commented Jun 21, 2023

Sorry, didn't see this PR, I will close this issue when merged 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants