Skip to content

Feat/bug fixes - #64

Merged
nil-malh merged 39 commits into
mainfrom
feat/bug-fixes
Oct 6, 2026
Merged

nil-malh merged 39 commits into
mainfrom
feat/bug-fixes

Conversation

@nil-malh

@nil-malh nil-malh commented Oct 6, 2026

Copy link
Copy Markdown
Contributor

📝 Description

Resolves #


🔄 Type of Change

  • 🐛 Bug fix — non-breaking change that fixes an issue
  • ✨ New feature — non-breaking change that adds functionality
  • 💥 Breaking change — fix or feature that changes existing behaviour
  • ♻️ Refactoring / technical debt — no functional change
  • 📖 Documentation — doc-only changes
  • ⬆️ Dependency update — bumped library/plugin version(s)
  • 🔧 CI / build — changes to workflows, pom.xml, tooling

✅ Pre-Merge Checklist

Code Quality

  • My code follows the Palantir Java Format style (passes mvn spotless:check)
  • I have added or updated Javadoc on public types and methods

Tests

  • I have added unit tests that cover the new / changed behaviour
  • All existing and new tests pass locally (mvn verify)
  • JaCoCo line coverage ≥ 70 % is maintained
  • Tests using Testcontainers are isolated and do not leave containers running

Licensing

  • Apache 2.0 license header has been added to every new source file (using spotless)

🧪 How to Test Locally

# 1. Build and run all tests (requires Docker for Testcontainers)
mvn verify

# 2. Check code style
mvn spotless:check

📸 Screenshots / Logs


📚 Additional Context

@github-actions

github-actions Bot commented Oct 6, 2026 •

Copy link
Copy Markdown
Contributor

📋 Unreleased Changelog Preview

This is what the next release notes will look like based on commits in this PR.

Changelog

All notable changes to this project will be documented in this file.

[Unreleased]

♻️ Refactoring

  • Enhance plugin loading and shutdown logic

  • Enhance timeout handling and improve documentation in KafkaRecordFetcher

  • Extract null value message to constant in MatchResult

⬆️ Dependency Updates

  • Bump ktestify-parent version from 1.0.4 to 1.0.5

🐛 Bug Fixes

  • Fixed an issue where a NullPointerException was thrown on unset environment variables

  • Close URLClassLoader in PluginRegistry to prevent file handle leak

  • Add missing schemas directory default to reference.conf

  • Harden XML parsers against XXE attacks in XMLUtils

  • Enforce error taxonomy and add missing exception constructors

  • Correct over-suppression of CHILD_NODELIST_LENGTH in XMLUtils

  • Fixed an issue where a SchemaRegistryClient was instantiated at each call instead of being cached

  • Improve byte buffer handling in AvroDeserializer

  • Include cause in ConsumerException for better error handling

  • Handle null values in record matchers and improve logging

  • Enhance security in SAXParserFactory and improve documentation

  • Improve handling of empty records in AttributeRecordMatcher

  • Improve getLine method to handle null content and out-of-bounds indices

  • Null pointer exception on empty TimestampVariable when format is null

  • Update dependency review workflow path

🔧 Miscellaneous

  • Fixed some tests

🧪 Tests

  • Add ClasspathTestPlugin for lifecycle call assertions

  • Add TombstoneMatcherTest to verify tombstone handling across matchers

  • Rename integration tests with *ITTests

  • Rename integration tests with *ITTests


Generated by git-cliff


🔄 Run #167 · Tue, 06 Oct 2026 19:11:38 GMT

@nil-malh
nil-malh merged commit 7941df8 into main Oct 6, 2026
8 checks passed
@github-actions github-actions Bot mentioned this pull request Oct 6, 2026
8 tasks done
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