Skip to content

fix: honor system properties in Jira Cloud HTTP client - #135

Open
rakshitjain25 wants to merge 1 commit into
reportportal:developfrom
rakshitjain25:fix/jira-cloud-httpclient-system-properties
Open

rakshitjain25 wants to merge 1 commit into
reportportal:developfrom
rakshitjain25:fix/jira-cloud-httpclient-system-properties

Conversation

@rakshitjain25

Copy link
Copy Markdown

Summary

This PR introduces a minor fix to the ReportPortal Jira Cloud plugin to ensure that the Apache HTTP client respects JVM system properties, including configured proxy settings.

Background

We encountered an issue with the ReportPortal Jira Cloud plugin where Jira defect creation involving attachments was failing in our environment.

GitHub Issue: #133

Code Change

Added .useSystemProperties() to the HTTP client builder:

HttpClient client = HttpClientBuilder.create()
    .useSystemProperties()
    .build();

Validation

  • Rebuilt the plugin JAR with the proposed modification.
  • Deployed and tested the modified plugin in our ReportPortal environment.
  • Confirmed that the previously failing Jira Cloud integration functionality worked successfully.

External Contribution

Target Repository: https://github.com/reportportal/plugin-bts-jira-cloud

Purpose: Contribute a general-purpose fix to the upstream open-source project.

@coderabbitai

coderabbitai Bot commented Oct 14, 2026

Copy link
Copy Markdown

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration
  • Configuration used: Organization UI
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 84f1c5df-c16d-4f3e-97a5-eaa325a58819
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

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.

1 participant