Skip to content

Conversation

@manovotn
Copy link
Member

@manovotn manovotn commented Oct 7, 2025

No description provided.

@manovotn manovotn merged commit be2a16a into master Oct 7, 2025
2 checks passed
@manovotn manovotn deleted the release503Final branch October 7, 2025 21:04
@manovotn
Copy link
Member Author

manovotn commented Oct 7, 2025

@Vampire do you happen to remember why we added the following javadoc config? It came in the PR with Spock although it applies to whole project.

It is now causing an error in release build and I can't figure out why we need it to begin with.

The URL response time is currently pretty horrendous although I don't get the error locally, it just takes a looong time 🤔

@Vampire
Copy link
Contributor

Vampire commented Oct 8, 2025

image

@manovotn
Copy link
Member Author

manovotn commented Oct 8, 2025

Ah, I totally missed that when comparing the two outputs, thank you.
It is indeed nice but if the links don't work it's not much use.
I'll probably need to cut it out to perform the release.

@Vampire
Copy link
Contributor

Vampire commented Oct 9, 2025

The links usually work, especially for the viewer.
Seems "just" the Cloudflare protection got a bit aggressive these days or misconfgured: maxcellent/javadoc.io#222

But in another project where I have the same problem one or two retries helped. 🤷‍♂️

@manovotn
Copy link
Member Author

But in another project where I have the same problem one or two retries helped. 🤷‍♂️

I did at least three tries in total and within a window a about ~12h and still saw it.
Only after that I removed it did the release again.

Thanks for the link though, I will keep an eye on it.

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

Successfully merging this pull request may close these issues.

3 participants