-
Notifications
You must be signed in to change notification settings - Fork 306
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
problems with static resources with HTTP/2 enabled /FISH-61 #2625
Comments
I've seen this too, and it's blocking us from moving to Payara5, but now that I've seen I'm not the only one, I think it may be worth the time to try to create a reproducer. In my case, the problem doesn't occur on every single request so it's a little difficult to reproduce. |
This looks like an issue with server push in HTTP/2. Can you confirm this? |
Also looks like a Grizzly issue |
I don't know about server push, but there's definitely lots of mention of grizzly in the stack traces. I'll try to setup another deployment of my project with payara 5.181 and gather some stack traces for reference. |
I think I can manage to provide a reproducer of https://bitbucket.org/fryske-akademy/superlemma-list:
Maybe tomorrow, but I think later.... |
Try disabling http/2 on the listener and see if the issue goes away. |
Great idea |
just tested, problem gone |
Ok looks like an issue with server push in http/2 as Lenny suggests. |
Sorry Lenny my phone called you Kenny. |
Disabling HTTP/2 on the listeners worked for me too... I receive no errors after that. |
shall I close this issue? |
|
no leave open as it looks like a genuine issue |
Reproducer
|
Hi @eduarddrenth, The scope of that example is rather large, and I'm having trouble trying to narrow down the issue. Are you able to produce a concise reproducer? Kind regards, Matt |
Hi, I am aware...I'll have to find some time somewhere.... |
Just tried fresh quarterly, layout isn't messed up anymore with http 2 on, but sometimes a page remains blank, after F5 I see the page. without http 2 all is well. See if I can give you a mvce anywhere soon |
I see in the log:
and later:
|
maybe this helps: square/okhttp#2506 |
enabling http2, but disabling push solves the problem |
Hi @eduarddrenth, Thank you for your clarification. Unfortunately, I still need that reproducer. I can't seem to build the app you posted since it relies on some internal dependencies, and the setup is rather long which doesn't help me narrow down the issue. I've attempted my own reproducer here, but can't find the issue. Do you have a shorter, reproducible application, or a tweak to my current application to help me reproduce it? Cheers, Matt |
I'll see what I can do, the app isn't realy that complicated, so I should be able to create a simple reproducer for you....may some time though. |
For now we will not do http2, no functional need, apache not quite ready yet, a bit too early. I'll still see if I can find some time to provide a reproducer. |
Hi @eduarddrenth, There's an issue raised with PrimeFaces that may be related. Are you able to reproduce the issue without using PrimeFaces? Kind regards, Matt |
Thanks, that looks very similar indeed... From the discussion I understand there is no clarity whether it is a pf or a payara issue. I'm on holiday now. Reproducing without using pf is I suspect more of a hassle for me than providing a standalone reproducer.... |
I've tried to reproduce the problem without primefaces but thus far have been unable to do so. I've not had time to make an isolated reproducer but would be willing to dedicate some time if this issue is going to get some attention. |
Assigned |
Hi, here one complet Log. It' exist a lot of same error for each resources files images, css, js ... than it's not only on PF graphicimage component also in java.io
|
chromium shows http1.1 as well |
Tracked now as |
The problem persists in 5.194: [2019-12-31T10:16:16.236-0500] [Payara 5.194] [WARNING] [jsf.application.resource.unable_to_serve_from_library] [javax.enterprise.resource.webcontainer.jsf.application] [tid: _ThreadID=1186 _ThreadName=http-thread-pool::http-listener-2(24)] [timeMillis: 1577805376236] [levelValue: 900] [[ the server stops responding after a certain number of requests. |
Just tested again, with and without push, on 5.194 this: #2625 (comment) is still valid |
I've bumped the priority on this. |
Also getting this issue on 5.194 + SSL + HTTP/2. Disabling HTTP/2 for http-listener-2 (https) resolves the issue. |
Moving from 4 to 5.194 build 327 + SSL + HTTP/2 we sometimes see HTML/images loading but not CSS/JS. We also have 2 websites on 5.192 + SSL + HTTP/2 which sometimes display a white screen / or partial loading as 5.194. Both seem related to this issue. We have now disabled HTTP/2 so far all good. We have also seen a class cast exception: [2020-02-29T23:15:53.175+0000] [Payara 5.194] [WARNING] [] [org.glassfish.grizzly.filterchain.DefaultFilterChain] [tid: _ThreadID=47 _ThreadName=http-thread-pool::http-listener-2(2)] [timeMillis: 1583018153175] [levelValue: 900] [[ Thanks |
Hi all, Thank you all for your input on this issue. This has now spread into several issues, which the work on For each of the issues I've been using the reproducer built from https://bitbucket.org/luminositylabs/payara-2625.git. Issue 1Since these problems are all caused by HTTP/2 push, we advised that turning off HTTP/2 push would fix the issue. Despite this, the following errors could still be seen in the logs:
This shows that HTTP/2 push is still, in some capacity, being used. CUSTCOM-22 has now been tagged and will be closed with the linked PR above. This fixes the help text of the admin console, and prevents these errors appearing when HTTP/2 push is disabled, therefore meaning that disabling HTTP/2 push should prevent these errors entirely. Issue 2The second issue is that when reloading the reproducer page repeatedly with push enabled, the following would occasionally happen: A resource intended to be pushed is instead only partially pushed. A thread dump reveals that the server thinks it has finished processing the resource, but the browser is waiting on more. This will need a more complex solution, so Issue 3The third issue is the above error itself. As stated briefly above, this could not be a problem at all if the errors are due to the previous connection being cut by a page reload (essentially the HTTP/2 push version of https://stackoverflow.com/questions/642061/error-commiting-response-java-io-ioexception-broken-pipe-at-sun-nio-ch-filedisp). Once CUSTCOM-277 has been resolved, we can then begin to investigate this properly. Thank you all for your patience with this issue. If the first issue I outlined is solved by the linked PR, I'll close this issue and reopen issues outlining the remaining HTTP/2 push problems. |
CUSTCOM-22 is part of release 5.2020.2. More work around HTTP/2 and HTTP Push is planned for next release. |
Hi, the issue seems to have been resolved in release 5.2020.2 and 5.20. I will be closing this issue, however feel free to raise a new issue if any bugs are spotted in the latest release. Thank you. |
Just tested, I can confirm http2 is active and application runs fine. Thanks! |
Congratulations everyone! This one is near and dear to my heart. |
Sorry, but under Payara 5.2022.5 Community I still have the problem as soon as I switch to https and HTTP/2 (not push) is activated. After a short time, resources are simply not reloaded and the display of the pages is incorrect and the behavior is incorrect because the Javascripts sometimes do not work properly. |
Description
I have a simple jsf 2 application with resources under javax.faces.resource. On and of there is a problem with downloading these, it either takes very long or fails, messing up the layout. Pressing F5 several times will finaly result in a correct page.
Expected Outcome
no problems with downloading static resources
Current Outcome
non responsive application with layout messed up
In the log I see:
Steps to reproduce (Only for bug reports)
Perhaps I'll spend time on reproducing, but I've just spent a few days already trying to solve this. My decision is to stick with 4.1.2.181 which is very stable.
Environment
The text was updated successfully, but these errors were encountered: