Skip to content

Create NOTICE#40

Merged
Mahesh-Binayak merged 1 commit into
mosip:developfrom
rajapandi1234:patch-6
Jan 6, 2026
Merged

Create NOTICE#40
Mahesh-Binayak merged 1 commit into
mosip:developfrom
rajapandi1234:patch-6

Conversation

@rajapandi1234

@rajapandi1234 rajapandi1234 commented Jan 6, 2026

Copy link
Copy Markdown

Summary by CodeRabbit

  • Chores
    • Added third-party license attribution documentation listing Apache-2.0, MIT, and MPL-2.0 licensed components included in the project.

✏️ Tip: You can customize this high-level summary in your review settings.

Signed-off-by: rajapandi1234 <138785181+rajapandi1234@users.noreply.github.com>
@coderabbitai

coderabbitai Bot commented Jan 6, 2026

Copy link
Copy Markdown

Walkthrough

A NOTICE file documenting third-party licenses used by the project has been added. The file lists Apache-2.0, MIT, and MPL-2.0 licensed components with their corresponding plugins and libraries, version inheritance notes from MOSIP Kernel BOM, and attribution requirements. No code changes.

Changes

Cohort / File(s) Summary
License Documentation
NOTICE
Added third-party license attribution document listing Apache-2.0, MIT, and MPL-2.0 licensed components with version and attribution details

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

  • license folder #38: Adds complementary repository license and NOTICE documentation with overlapping license file definitions.

Poem

🐰 A notice for all the code we use,
Apache, MIT, no need to fuss,
Attribution shared with care so true,
Third-party friends, we credit you! 📜

Pre-merge checks

✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'Create NOTICE' directly and clearly describes the main change in the pull request, which adds a NOTICE file documenting third-party licenses.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

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 and usage tips.

@Mahesh-Binayak

Copy link
Copy Markdown

@CodeRabbit review

@coderabbitai

coderabbitai Bot commented Jan 6, 2026

Copy link
Copy Markdown
✅ Actions performed

Review triggered.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

@Mahesh-Binayak Mahesh-Binayak merged commit 07084a1 into mosip:develop Jan 6, 2026
5 checks passed

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 1

🧹 Nitpick comments (1)
NOTICE (1)

46-47: Consider improving version specificity documentation.

Lines 46–47 use the placeholder phrase "version as used in project" for MOSIP Kernel Biometrics and ImageDecoder. While delegating to dependency management is reasonable, consider documenting the version constraint (e.g., range, BOM reference) for future maintainers and compliance audits.

📜 Review details

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 1e345f5 and 35869c8.

📒 Files selected for processing (1)
  • NOTICE

Comment thread NOTICE
Comment on lines +1 to +64
MOSIP
Copyright (c) 2022
Modular Open Source Identity Platform

This product includes software developed at
The Apache Software Foundation (https://www.apache.org/).

The following third-party components are licensed under the
Apache License 2.0 and require attribution. Their original NOTICE
contents are retained as required by the Apache License 2.0.


===============================================================================
Apache License 2.0 Components
===============================================================================

• Apache Maven Source Plugin (3.3.1)
• Apache Maven GPG Plugin (3.2.3)
• Apache Maven Assembly Plugin (3.1.1)
• Apache Maven Compiler Plugin (3.11.0)
• Apache Maven Javadoc Plugin (3.2.0)
• Apache Maven Surefire Plugin (2.22.0)
• Apache Maven JAR Plugin (3.0.2)
• Apache Maven Dependency Plugin (3.1.2)
• Apache Maven Deploy Plugin (3.1.1)
• Apache Maven Resources Plugin (3.0.2)
• Jackson Core (version managed through MOSIP Kernel BOM)
• Jackson Databind (version managed through MOSIP Kernel BOM)


===============================================================================
MIT License Components
===============================================================================

• SLF4J - jul-to-slf4j
• SLF4J - jcl-over-slf4j


===============================================================================
MPL-2.0 Components (MOSIP Kernel)
===============================================================================

• MOSIP Kernel BOM (1.3.0-SNAPSHOT)
• MOSIP Kernel Core (1.3.0-SNAPSHOT)
• MOSIP Kernel Logger Logback (1.3.0-SNAPSHOT)
• MOSIP Kernel Biometrics (version as used in project)
• MOSIP Kernel ImageDecoder (version as used in project)


===============================================================================
Notes
===============================================================================

This project uses components under the following licenses:

• MPL-2.0:
MOSIP Kernel modules including Kernel BOM, Core, Logger, Biometrics,
and ImageDecoder.

• MIT:
SLF4J bridging modules such as jul-to-slf4j and jcl-over-slf4j.

Jackson Core and Jackson Databind versions are inherited from the
MOSIP Kernel BOM (1.3.0-SNAPSHOT) and therefore not listed explicitly.

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ Potential issue | 🔴 Critical

🧩 Analysis chain

🏁 Script executed:

# Find pom.xml files and other build configuration
fd -n 'pom\.xml' -o 'build\.gradle' -o 'package\.json'

Repository: mosip/imagedecoder

Length of output: 292


🏁 Script executed:

# Get the root pom.xml and examine its structure
git ls-files | grep -E '(^pom\.xml|^[^/]*pom\.xml)$'

Repository: mosip/imagedecoder

Length of output: 44


🏁 Script executed:

# Look for any pom.xml in the repository
fd 'pom\.xml$'

Repository: mosip/imagedecoder

Length of output: 97


🏁 Script executed:

# Examine the main imagedecoder pom.xml
head -100 imagedecoder/pom.xml

Repository: mosip/imagedecoder

Length of output: 3739


🏁 Script executed:

# Check size and get the full main pom.xml
wc -l imagedecoder/pom.xml

Repository: mosip/imagedecoder

Length of output: 86


🏁 Script executed:

# Check sample pom.xml as well
wc -l sample/pom.xml

Repository: mosip/imagedecoder

Length of output: 80


🏁 Script executed:

# Read the rest of imagedecoder/pom.xml
tail -n +100 imagedecoder/pom.xml

Repository: mosip/imagedecoder

Length of output: 6857


🏁 Script executed:

# Read the full sample/pom.xml
cat sample/pom.xml

Repository: mosip/imagedecoder

Length of output: 6622


🏁 Script executed:

# Check if there's any LICENSE or related documentation in the repo
fd -n 'LICENSE|COPYING' --type f

Repository: mosip/imagedecoder

Length of output: 292


🏁 Script executed:

# Look for any documentation about licensing or dependencies
fd -n 'README|CONTRIBUTING|LICENSE' --type f -i

Repository: mosip/imagedecoder

Length of output: 292


🏁 Script executed:

# Find LICENSE and README files
fd 'LICENSE|README' --type f

Repository: mosip/imagedecoder

Length of output: 94


🏁 Script executed:

# Check what documentation files exist in the root
git ls-files | head -30

Repository: mosip/imagedecoder

Length of output: 2049


🌐 Web query:

jacoco-maven-plugin Apache license

💡 Result:

The jacoco-maven-plugin is distributed under the Eclipse Public License version 2.0 (EPL-2.0). [1][2]

Sources:

  • JaCoCo license page stating distribution under the Eclipse Public License 2.0. [1]
  • jacoco-maven-plugin source header (AgentMojo.java) showing SPDX: EPL-2.0. [2]

🌐 Web query:

central-publishing-maven-plugin license

💡 Result:

The central-publishing-maven-plugin is licensed under the Apache License 2.0. [1][2]

Sources:
[1] Sonatype Central publishing docs.
[2] Maven Repository listing for org.sonatype.central:central-publishing-maven-plugin.


🌐 Web query:

git-commit-id-plugin license

💡 Result:

The git-commit-id-maven-plugin is released under the GNU Lesser General Public License v3 (LGPL-3.0). [1][2]

Sources:

  • GitHub repository license section. [1]
  • Plugin documentation (project site). [2]

🌐 Web query:

sonar-maven-plugin license

💡 Result:

The Sonar Maven plugin (SonarScanner for Maven) is licensed under the GNU Lesser General Public License v3.0 (LGPL-3.0). (github.com)

Citations:


Update NOTICE file to match actual project dependencies and correct version discrepancies.

The NOTICE file has several critical inaccuracies when compared to the actual build configuration:

  1. MOSIP Kernel version is outdated: Both pom.xml files specify 1.2.1-SNAPSHOT, but NOTICE lists 1.3.0-SNAPSHOT for Kernel BOM, Core, and Logger Logback.

  2. Missing build plugins:

    • central-publishing-maven-plugin (0.7.0) – Apache License 2.0
    • maven-dependency-plugin (3.1.2) – Apache License 2.0 (used in sample/pom.xml)
    • maven-resources-plugin (3.0.2) – Apache License 2.0 (used in sample/pom.xml)
    • jacoco-maven-plugin (0.8.11) – Eclipse Public License 2.0 (EPL-2.0)
    • git-commit-id-plugin (3.0.1) – GNU Lesser General Public License 3.0 (LGPL-3.0)
    • sonar-maven-plugin (3.7.0.1746) – GNU Lesser General Public License 3.0 (LGPL-3.0)
  3. Phantom dependencies: "MOSIP Kernel Biometrics" and "MOSIP Kernel ImageDecoder" are listed but are not actual external dependencies in the pom.xml files.

  4. New license categories needed: Add sections for EPL-2.0 and LGPL-3.0 components to the NOTICE structure.

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.

2 participants