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

Close resources to stop Apache lib from pending #3524

Merged
merged 4 commits into from
Dec 19, 2024
Merged

Conversation

sebr72
Copy link
Contributor

@sebr72 sebr72 commented Dec 17, 2024

Fix issue where you no longer get a reply after 5 requests with:
` try (CloseableHttpClient client = HttpClients.createDefault()) {
HttpUriRequest request = new HttpGet("https://map.geo.ti.ch/oereb2/image/symbol/ch.Nutzungsplanung/legend_entry.png?identifier=423441325078");
HttpContext context = null;

        for (int i = 1; i < 7; i++) {
            System.out.println("i = " + i);
            HttpResponse response = client.execute(request, context);
            System.out.println(response.getReasonPhrase());
        }
    }

`

See JIRA issue: ISSUE-36.

@sebr72 sebr72 force-pushed the oereb-issue-145 branch 3 times, most recently from 2aec670 to fcac103 Compare December 18, 2024 11:39
@sebr72 sebr72 marked this pull request as ready for review December 19, 2024 17:05
@sebr72 sebr72 requested a review from sbrunner December 19, 2024 17:06
@sbrunner
Copy link
Member

Thanks :-)

@sebr72
Copy link
Contributor Author

sebr72 commented Dec 19, 2024

This PR is followed by issue 39

@sebr72 sebr72 merged commit 3f8a874 into master Dec 19, 2024
11 checks passed
@sebr72 sebr72 deleted the oereb-issue-145 branch December 19, 2024 21:03
@sbrunner sbrunner added the backport 3.31 Backport the pull request to the '3.31' branch label Dec 20, 2024
@c2c-bot-gis-ci-2 c2c-bot-gis-ci-2 removed the backport 3.31 Backport the pull request to the '3.31' branch label Dec 20, 2024
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