Skip to content

Add utilities to detect and replace broken links. #1366

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

Open
barsh404error wants to merge 11 commits into
base: develop
Choose a base branch
from
Open

Add utilities to detect and replace broken links. #1366

Changes from 1 commit
Commits
Show all changes
11 commits
Select commit Hold shift + click to select a range
9004e6b
Add utilities to detect and replace broken links.
barsh404error Dec 29, 2025
7153833
style: run spotlessApply
barsh404error Dec 31, 2025
d205074
Add utilities to detect and replace broken links V2
barsh404error Jan 2, 2026
c3a64e3
Add utilities to detect and replace broken links V2
barsh404error Jan 2, 2026
48c23ab
Add utilities to detect and replace broken links V2
barsh404error Jan 3, 2026
d556a33
Add utilities to detect and replace broken links V2
barsh404error Jan 6, 2026
9414ead
Fixed link detection to handle 3xx redirects properly
barsh404error Jan 30, 2026
ef38986
Apply Spotless formatting and regenerate jOOQ sources
barsh404error Jan 30, 2026
7d3bf57
commit:Broken links resolve requested changes
barsh404error Feb 14, 2026
97ef9ce
refactor: apply review feedback from @Zabuzard
barsh404error Feb 17, 2026
7118454
New fixed and changes
barsh404error Feb 18, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Add utilities to detect and replace broken links V2
  • Loading branch information
@barsh404error
barsh404error committed Jan 2, 2026
commit c3a64e39a4bfb2e1891896905d63e0b18659d855
15 changes: 5 additions & 10 deletions application/src/main/java/org/togetherjava/tjbot/features/utils/LinkDetection.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -68,7 +68,6 @@ public static boolean containsLink(String content) {
return !(new UrlDetector(content, UrlDetectorOptions.BRACKET_MATCH).detect().isEmpty());
}

@SuppressWarnings("java:S2095")
public static CompletableFuture<Boolean> isLinkBroken(String url) {
HttpRequest headRequest = HttpRequest.newBuilder(URI.create(url))
.method("HEAD", HttpRequest.BodyPublishers.noBody())
Expand All @@ -77,21 +76,17 @@ public static CompletableFuture<Boolean> isLinkBroken(String url) {
return HTTP_CLIENT.sendAsync(headRequest, HttpResponse.BodyHandlers.discarding())
.thenApply(response -> {
int status = response.statusCode();
if (status >= 200 && status < 400) {
return false;
}
return status >= 400 ? true : null;
return status < 200 || status >= 400;
})
.exceptionally(ignored -> null)
.exceptionally(ignored -> true)
.thenCompose(result -> {
if (result != null) {
return CompletableFuture.completedFuture(result);
if (!result) {
return CompletableFuture.completedFuture(false);
}
HttpRequest getRequest = HttpRequest.newBuilder(URI.create(url)).GET().build();

return HTTP_CLIENT.sendAsync(getRequest, HttpResponse.BodyHandlers.discarding())
.thenApply(resp -> resp.statusCode() >= 400)
.exceptionally(ignored -> true);
.exceptionally(ignored -> true); // still never null
});
}

Expand Down