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.

Sign up for GitHub

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

Jump to bottom

Send dev server output to normal log output #12281

Closed
Artur- opened this issue Nov 5, 2021 · 1 comment · Fixed by #12321
Closed

Send dev server output to normal log output #12281

Artur- opened this issue Nov 5, 2021 · 1 comment · Fixed by #12321
Labels
prerelease version for platform 22.0.0 Released with Platform 22.0.0.beta2 vite Tickets related to vite support
Projects
Frontend build optimization

Comments

@Artur-
Copy link
Member

Artur- commented Nov 5, 2021

The dev server process currently writes its output character by character to stdout. This doesn't work very well when something is logging at the same time. It is a tad hard to see what the problem is e.g. in this case

u[r1rmentC.oCmopmplleetatbaleFbultuerFe$uUtnuiWrhee.njCaovmap:l4ete8.3tyF[imre()C
omp le t ab leFuture[.jav1am:8a41t)
	at  joarvga..ebascel/jaivpas.uteil..cjoentcurtryent..Csomeprlveteablre.FuHtturet$pCComhpalentnieon.le.xecl(Caommpbletdaable$Fuhturae.jnadva:l48e3)
$1	at ava[.b1asema/tjav[.utmil.c:oncu rrent.Fr[kJo1i;nTask3.1dmoExjeac(vForakxJoi.nTsaesk.rjavva:l37e3)
t	.at Sjeavra.bavslee/tjEavax.cutielp.tcoinocnu:r rceonmt..vFaoardkiJno.ifnlPoowo.ls$eWrovrekrQ.uSeeurev.toipcLeevelEExxecc(FeorkpJtoiinoPoonl:.ja vaj:11a8v2)a
.	lat ajanva.bga.seI/jlalva.eutgil.acolncSutrartente.FEoxrkJcoienPopotl.sicaon(Fno:rk JoiVniPootle. jaevxa:i1t65e5d)
 p	arte jmaavat.ubasree/ljayv.u[tmil.
conc urrjeanvt.aF.or k J oi nPol[.1rumnaWto[kmer o(rFgo.rekclJiopisneP.ojoelt.tjya.vsa:e1r6v2e2r)
.ha	antd ljearv.aHa.nbdalseer/Cjoalvlaec.tuitoinl..hcaonncdulrerent. For(ko[i1nWmoHrtketrpTChhraenadn.erlu.nj(aFvoar:k3J8o8i[Wmo)rke rTh(r[a1dm.Hjaanvdal:e1r6C5o)ll
ection.java:162)

The dev server should instead output line by line through the normal logger
@Artur- Artur- added this to To do in Frontend build optimization via automation Nov 5, 2021
@Artur- Artur- added the vite Tickets related to vite support label Nov 5, 2021
@caalador caalador mentioned this issue Nov 5, 2021
Closed
Artur- added a commit that referenced this issue Nov 9, 2021
@Artur-
fix: Log dev server output using standard logger
11237f7 
Fixes #12281
@Artur- Artur- mentioned this issue Nov 9, 2021
Merged
Frontend build optimization automation moved this from To do to Done Nov 10, 2021
Artur- added a commit that referenced this issue Nov 10, 2021
@Artur-
fix: Log dev server output using standard logger (#12321)
36307ca 
Rename DevServerOutputFinder to the more descriptive DevServerOutputTracker

Fixes #12281
@vaadin-bot
Copy link
Collaborator

This ticket/PR has been released with platform 22.0.0.beta2 and is also targeting the upcoming stable 22.0.0 version.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
prerelease version for platform 22.0.0 Released with Platform 22.0.0.beta2 vite Tickets related to vite support
Projects
Frontend build optimization
Done
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix: Log dev server output using standard logger vaadin/flow
2 participants
@Artur- @vaadin-bot