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

warning JSF1064 is logged when page refresh is forced #5527

Open
EugenFischer opened this issue Nov 6, 2024 · 2 comments
Open

warning JSF1064 is logged when page refresh is forced #5527

EugenFischer opened this issue Nov 6, 2024 · 2 comments

Comments

@EugenFischer
Copy link

EugenFischer commented Nov 6, 2024

If an impatient user frequently reloads the page, the error is logged more frequently

The guard that is supposed to prevent this is inadequate.
ResourceHandlerImpl isConnectionAbort.
In my case the system language is set to german so the check doesn't work.

To Reproduce

Steps to reproduce the behavior:
configure system language to any other than english

  1. open any JSF page where several web resorces get loaded
  2. execute F5 or Strg + F5 several times
  3. warining occurs
    WARNING [jakarta.enterprise.resource.webcontainer.faces.application] (default task-14) JSF1064: Ressource theme.css aus Bibliothek primefaces kann nicht gefunden oder bedient werden. WARNING [jakarta.enterprise.resource.webcontainer.faces.application] (default task-14) : java.io.IOException: Eine bestehende Verbindung wurde softwaregesteuert durch den Hostcomputer abgebrochen

Expected behavior
no warning get logged

resourceHandlerImpl

@BalusC
Copy link
Contributor

BalusC commented Nov 7, 2024

Yeah, that's hacky as it hasn't been standardized in Servlet API yet. Ideally the underlying I/O handler should have thrown ClosedChannelException or even InterruptedIOException or EOFException instead. Try reporting an issue to Undertow to have them to proactively improve that in their impl. In the meanwhile all we can do is to detect German translation as well but then one would wonder how about all these other languages .. English is the de facto standard/default and hence it was the only one being checked for.

@pizzi80
Copy link
Contributor

pizzi80 commented Nov 8, 2024

+1 for standardize this

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

No branches or pull requests

3 participants