-
Notifications
You must be signed in to change notification settings - Fork 47
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
Provide more details about http failures #270
Comments
right u are. Will look into it asap (might take a few days, as II'm currently on holiday) |
(and I feel deeply honored that you care for my little library... although it by no means matches your high software quality standards...) |
@mernst - could you please provide the context of the error message (at best the complete report)? In the corresponding Checker subclass, the http status code is included in the output:
And I verified that the URL you mentioned above returns status code 200... Therefore, I need some additional info for debugging... regards, Gernot |
Thanks for the quick response! To reproduce the error, please run:
There is just one error -- the one I quoted above. Please let me know if you need any more information or if I can do anything to help. Thanks for your help! |
had to remove the html5Validator requirement - as the (normal) build requires that prerequisite... actually there are two distinct problems here:
I'm investigating... |
Currently, the Randoop build requires installation of I'm investigating replacing that by htmlSanityCheck. |
Thanx to the hint from @double16 I'll try to come up with a more relaxed HttpsUrlConnection class, that does not try to validate the certificates... |
Hi First we have tried establishing a DNS cache in the pipeline because we suspected an intermittent DNS resolving issue. In the linked issue there was a problem with certificate verification. Additional error details for this kind of error would be much appreciated! |
Thanks for adding your comment, @PirMei.
Is this a requirement for the component, that it should show more context information when the error occurs? BTW: Which version of HSC are you using? Did you see that we come up with a version 2, which currently is more like a modularisation of the project? Find more info on https://hsc.aim42.org. |
And one additional question @PirMei : How do you run it in your Gitlab CI pipeline? Do you use some container? If yes, what is the base (OS, JDK, Gradle, etc.)? |
@ascheman Thanks for the quick reply 👍 We're currently using htmlSanityCheck gradle plugin version 1.1.6 (as the latest stable version) in a self-created container based on Ubuntu 24.04 with JDK 17.0.12-tem (installed via sdkman.io).
More details for HTTP errors would just make troubleshooting a whole lot easier. |
as we would like to be able to check more detailed error conditions when the problems occur (#270).
Meanwhile, I could assemble a first approach to improve this
Please give it a try, @PirMei: You have to either clone the repo and build locally, or refer to https://jitpack.io/#aim42/htmlSanityCheck/bugfix~270-improve-exception-handling-SNAPSHOT, i.e., Add JitPack repo to pluginManagement {
repositories {
maven { url 'https://jitpack.io' } // Make this the first entry!
...
gradlePluginPortal()
}
}
... Choose plugin version by changing the version to the branch, i.e., plugins {
id 'org.aim42.htmlSanityCheck' version "bugfix~270-improve-exception-handling-SNAPSHOT"
|
"Unfortunately" the |
Thanks a lot for the information, @PirMei. I was already guessing (and hoping) that it will turn out to be an infrastructure problem. Such flaky behaviour is hard to track (and even harder to circumvent) from the application layer. However, as I have spend some time on it, I will soon merge the improved error handling (but currently have other priorities). |
After reading your replies again, I would like to add one more bit of information. Since we only got the unknown host errors for a handful of hrefs but not for the vast majority of hrefs pointing to the same host even after establishing the DNS cache, I would specifically eliminate DNS problems. |
In a message like this one:
it would be helpful to give more details about the failure, such as the exact error code.
I am able to browse to that URL, and a HEAD request also seems works for me using curl, so I don't know exactly to diagnose the problem that htmlSanityCheck is suffering.
The text was updated successfully, but these errors were encountered: