Add GitHub workflow and Update dependency by alikemalocalan · Pull Request #10 · stitchdata/java-stitch-client

alikemalocalan · 2026-04-21T08:12:45Z

Description of change

This PR includes Maven project configuration improvements and a migration to Java's built-in HTTP client.

What changed

Runtime HTTP client migration: Replaced Apache HttpClient with java.net.http.HttpClient for all network calls to Stitch
Build/release configuration:
- Updated build/release plugin setup in pom.xml (source, javadoc, GPG signing, Nexus staging, and release flow)
- Refined publishing-related configuration for OSSRH deployment
- Updated/maintained dependency and plugin versions

Why

HTTP client migration: Simplifies dependencies by using Java's built-in HTTP client (available since Java 11), reducing external dependency footprint and improving maintainability
Configuration updates: Improve release reliability and maintainability of the Java Stitch Client build pipeline

Runtime behavior changes

This change modifies runtime HTTP behavior including:

TLS/SSL defaults and cipher suite handling
HTTP redirect handling semantics
Connection timeout and retry behavior
Header processing and validation
Potential differences in error handling and edge cases

QA steps

automated tests passing
manual qa steps passing (list below)

Manual QA steps:

Run a clean build and verify packaging succeeds
Run the full test suite and confirm all tests pass
Verify source and javadoc artifacts are generated
Verify release-related Maven configuration is resolved correctly (without performing an actual production release)
Integration testing: Test HTTP connectivity and batch submissions to verify the new HTTP client behaves correctly with Stitch API endpoints

Risks

HTTP client behavior differences: Switching HTTP clients may expose edge cases or behavioral differences in TLS negotiation, redirects, timeouts, or error handling
Release/publish workflow risk: misconfiguration could affect artifact signing or staging
Build-time compatibility risk: plugin version changes may behave differently across local/CI environments
Integration risk: Network communication changes must be validated against actual Stitch API

Rollback steps

Revert this branch
Re-run CI to confirm previous build and HTTP behavior is restored

spendito-peter · 2026-05-04T07:44:33Z

@psantacl @cmerrick @mdelaurentis

Would it be possible for you to review this request? Due to severe vulnerabilities a release with updated dependencies would be very desirable.

Thank You!

@alikemalocalan fyi

Copilot

Pull request overview

This PR modernizes the project’s build/release setup (Maven plugins, Java level, dependencies) and adds GitHub automation, while also updating the client implementation to use Java 11’s java.net.http.HttpClient and Jakarta JSON.

Changes:

Update pom.xml for Java 11, newer Maven release/publish plugins, and dependency refresh (including removing Apache HttpClient in favor of JDK HTTP client).
Add GitHub Actions CI workflows and Dependabot configuration for Maven dependencies.
Replace previously “example” main classes with JUnit tests that exercise the same flows; add a test asserting HTTP/1.1 usage.

Reviewed changes

Copilot reviewed 14 out of 14 changed files in this pull request and generated 4 comments.

Show a summary per file

File	Description
`src/main/java/com/stitchdata/client/StitchClient.java`	Switch HTTP transport to `java.net.http.HttpClient`, update JSON parsing, add helper for JSON content-type detection.
`src/main/java/com/stitchdata/client/StitchResponse.java`	Migrate JSON type to `jakarta.json.JsonObject`.
`src/main/java/com/stitchdata/client/StitchClientBuilder.java`	Formatting/type tweaks around Transit write handlers and builder `build()` formatting.
`src/main/java/com/stitchdata/client/Buffer.java`	Import cleanup and formatting-only changes.
`src/test/java/com/stitchdata/client/StitchClientTest.java`	Import cleanup/formatting; add test asserting client uses HTTP/1.1.
`src/test/java/com/stitchdata/client/BufferTest.java`	Import cleanup/formatting.
`src/test/java/com/stitchdata/client/SimpleExampleTest.java`	New test replacing the removed “simple example” main program flow.
`src/test/java/com/stitchdata/client/CallbackExampleTest.java`	New test replacing the removed “callback example” main program flow.
`src/main/java/com/stitchdata/client/examples/SimpleExample.java`	Removed example main program (superseded by tests).
`src/main/java/com/stitchdata/client/examples/CallbackExample.java`	Removed example main program (superseded by tests).
`pom.xml`	Update compiler level to 11; update Maven plugins; refresh dependencies (Transit, Jakarta JSON, JUnit), remove Apache HttpClient dependency.
`.github/workflows/maven.yml`	New Maven build workflow (currently restricted to `master`).
`.github/workflows/maven-unit-tests.yml`	New unit test workflow for Java 11/17 (currently `push` only).
`.github/dependabot.yml`	New Dependabot configuration for Maven.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

Copilot

Pull request overview

Copilot reviewed 15 out of 15 changed files in this pull request and generated 6 comments.

Copilot

Pull request overview

Copilot reviewed 15 out of 15 changed files in this pull request and generated 5 comments.

Copilot

Pull request overview

Copilot reviewed 15 out of 15 changed files in this pull request and generated 8 comments.

            ByteArrayInputStream bais = new ByteArrayInputStream(entry.bytes);
            Reader reader = TransitFactory.reader(TransitFactory.Format.JSON, bais);
-            messages.add((Map)reader.read());
+            messages.add(reader.read());


+        // Try to extract more context from JSON response body if available
+        if (jsonContent != null && jsonContent.containsKey("message")) {
+            String message = jsonContent.getString("message");
+            return standardPhrase + " - " + message;
+        }


+     *                         Stitch
     */
-    public void push(StitchMessage message, Object callbackArg) throws StitchException, IOException {
+    public void push(StitchMessage message, Object callbackArg) throws StitchException, IOException, IllegalArgumentException {


+      <plugin>
+        <groupId>org.apache.maven.plugins</groupId>
+        <artifactId>maven-compiler-plugin</artifactId>
+        <version>3.15.0</version>
+        <configuration>
+          <source>11</source>
+          <target>11</target>
+        </configuration>
+      </plugin>


-import java.util.List;
-import java.util.ArrayList;
-import java.util.concurrent.atomic.AtomicInteger;
+import java.util.*;


+import java.io.ByteArrayInputStream;
+import java.io.IOException;
+import java.nio.charset.StandardCharsets;
+import java.util.*;


+import java.io.ByteArrayInputStream;
+import java.io.IOException;
+import java.nio.charset.StandardCharsets;
+import java.util.*;


+public class SimpleExampleTest {
+
+    private static Map<String, Object> makePerson(int id, String name) {
+        Map<String, Object> result = new HashMap<>();
+        result.put("id", id);
+        result.put("name", name);
+        return result;
+    }
+
+    private static class DummyStitchClient extends StitchClient {
+        final List<Map<String, Object>> receivedPeople = new ArrayList<Map<String, Object>>();
+
+        DummyStitchClient() {
+            super("", 1, "token", "namespace", "people", Arrays.asList("id"),
+                    StitchClientBuilder.DEFAULT_BATCH_SIZE_BYTES,


alikemalocalan added 4 commits March 16, 2026 16:45

Add workflows

6af65ae

Update dependency

0fa8eb9

Remove apache fluent hc and use Java 11 native http client

541cea6

Fix maven repo url issue

c703cca

alikemalocalan force-pushed the master branch from 5b0e1e1 to c703cca Compare May 4, 2026 07:41

sgandhi1311 requested a review from Copilot May 5, 2026 14:22

Copilot started reviewing on behalf of sgandhi1311 May 5, 2026 14:23 View session

Copilot AI reviewed May 5, 2026

View reviewed changes

Comment thread src/main/java/com/stitchdata/client/StitchClient.java

Comment thread src/main/java/com/stitchdata/client/StitchClient.java

Comment thread .github/workflows/maven.yml Outdated

Comment thread .github/workflows/maven-unit-tests.yml

alikemalocalan and others added 2 commits May 6, 2026 14:07

Potential fix for pull request finding

5a5335f

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

feat: Reformat code

5c6d87b

alikemalocalan requested a review from Copilot May 6, 2026 12:58

Copilot started reviewing on behalf of alikemalocalan May 6, 2026 12:59 View session

Copilot AI reviewed May 6, 2026

View reviewed changes

feat Fix Ttypos

8e4ecbd

alikemalocalan requested a review from Copilot May 6, 2026 13:16

Copilot started reviewing on behalf of alikemalocalan May 6, 2026 13:17 View session

Copilot AI reviewed May 6, 2026

View reviewed changes

feat Fix typos

7779222

alikemalocalan requested a review from Copilot May 6, 2026 14:41

Copilot started reviewing on behalf of alikemalocalan May 6, 2026 14:42 View session

Copilot AI reviewed May 6, 2026

View reviewed changes

alikemalocalan requested a review from Copilot May 6, 2026 14:51

Copilot started reviewing on behalf of alikemalocalan May 6, 2026 14:51 View session

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Add GitHub workflow and Update dependency#10

Add GitHub workflow and Update dependency#10
alikemalocalan wants to merge 8 commits intostitchdata:masterfrom
Spendit-AG:master

alikemalocalan commented Apr 21, 2026 •

edited

Loading

Uh oh!

spendito-peter commented May 4, 2026

Uh oh!

Copilot AI left a comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

Conversation

alikemalocalan commented Apr 21, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Description of change

What changed

Why

Runtime behavior changes

QA steps

Manual QA steps:

Risks

Rollback steps

Uh oh!

spendito-peter commented May 4, 2026

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

alikemalocalan commented Apr 21, 2026 •

edited

Loading