Skip to content

Multiple bug fixes - #61

Open
nil-malh wants to merge 26 commits into
mainfrom
feat/bug-fixes
Open

nil-malh wants to merge 26 commits into
mainfrom
feat/bug-fixes

Conversation

@nil-malh

@nil-malh nil-malh commented Oct 1, 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 1, 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

✨ Features

  • Add AvroJson and AvroLogicalTypesSerializationTest for JSON conversion of Avro logical types

  • Enhance DynamicVariableFactory for thread-safety and case-insensitivity

  • Added two new MatchResult noRecords and nullValue

🐛 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

🔧 Miscellaneous

  • Fixed some tests

🧪 Tests

  • Add ClasspathTestPlugin for lifecycle call assertions

  • Add TombstoneMatcherTest to verify tombstone handling across matchers


Generated by git-cliff


🔄 Run #151 · Fri, 02 Oct 2026 21:00:27 GMT

@github-actions

github-actions Bot commented Oct 1, 2026 •

Copy link
Copy Markdown
Contributor

✅ Test Results

Metric Count
✅ Passed 0
❌ Failed 0
⏭️ Skipped 0
📊 Total 0

⚠️ Coverage report not available


🔄 CI run #170 · Fri, 02 Oct 2026 21:00:26 GMT

This branch has not been deployed

No deployments
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