Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feature/verification backend: Added Verification Add-on + Modularized Backend to Core #342

Merged
merged 19 commits into from
Jul 3, 2024

Conversation

matbmoser
Copy link
Contributor

@matbmoser matbmoser commented Jun 21, 2024

Why we create this PR?

The feature eclipse-tractusx/sig-release#655 requested us to create the digital product pass verification for the consumer backend.

What we want to achieve with this PR?

Be able to verify the data incoming from the digital twin behind an EDC against a wallet that issued the credential.

What is new?

Added

  • Added the Verification Module with all the necessary changes to verify a digital product pass Certified Data Credential
  • Added wallet service and configuration so that the backend can communicate with the Simple Wallet

Updated

  • Moved all the "core" backend components to the "core" folder so that the application would be modular.
  • Updated configurations and implemented verification configuration
  • Updated licenses from charts

PR Linked to:

Closes eclipse-tractusx/sig-release#655

Mathias Brunkow Moser and others added 14 commits June 17, 2024 16:01
Co-authored-by: Mathias Brunkow Moser <[email protected]>
Co-authored-by: Muhammad Saud Khan <[email protected]>
Co-authored-by: Mathias Moser <[email protected]>
Co-authored-by: Saud Khan <[email protected]>
Co-authored-by: Mathias Brunkow Moser <[email protected]>
Co-authored-by: Muhammad Saud Khan <[email protected]>
Co-authored-by: Mathias Moser <[email protected]>
Co-authored-by: Saud Khan <[email protected]>
…ation

Co-authored-by: Mathias Brunkow Moser <[email protected]>
Co-authored-by: Muhammad Saud Khan <[email protected]>
Co-authored-by: Mathias Moser <[email protected]>
Co-authored-by: Saud Khan <[email protected]>
Co-authored-by: Mathias Brunkow Moser <[email protected]>
@matbmoser matbmoser added backend Issue or PR connected to the backend application DPP Verification Digital Product Pass Verification and Certification of Assets add-on labels Jun 21, 2024
@matbmoser matbmoser added this to the v4.0.0 milestone Jun 21, 2024
@matbmoser matbmoser requested a review from saudkhan116 June 21, 2024 17:05
@matbmoser matbmoser self-assigned this Jun 21, 2024
@matbmoser
Copy link
Contributor Author

I have move the complete backend files to another folder, because of this there is so many lines of code. All the code has been done by me.

/**
* Initiates the main needed variables for Data Transfer Service by loading from the environment variables and Vault.
**/
public void init(Environment env) {

Check notice

Code scanning / CodeQL

Useless parameter Note

The parameter 'env' is never used.
Copy link
Contributor

@saudkhan116 saudkhan116 left a comment

Choose a reason for hiding this comment

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

The license headers are not correct in verification source code, please correct them.

The correct one is:

"Tractus-X - Digital Product Passport Verification Add-on

Copyright (c) 2023 BMW AG
Copyright (c) 2023 CGI Deutschland B.V. & Co. KG
Copyright (c) 2024 Contributors to the Eclipse Foundation

See the NOTICE file(s) distributed with this work for additional
information regarding copyright ownership.

This program and the accompanying materials are made available under the
terms of the Apache License, Version 2.0 which is available at
https://www.apache.org/licenses/LICENSE-2.0.

Unless required by applicable law or agreed to in writing, software
distributed under the License is distributed on an "AS IS" BASIS
WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND,
either express or implied. See the
License for the specific language govern in permissions and limitations
under the License.

SPDX-License-Identifier: Apache-2.0"

@@ -0,0 +1,194 @@
/*********************************************************************************
*
* Tractus-X - Digital Product Pass Application
Copy link
Contributor

Choose a reason for hiding this comment

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

Is it not a verification add-on in license header?

@@ -0,0 +1,69 @@
/*********************************************************************************
*
* Tractus-X - Digital Product Pass Application
Copy link
Contributor

Choose a reason for hiding this comment

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

same here

* Tractus-X - Digital Product Pass Application
*
* Copyright (c) 2022, 2024 BMW AG, Henkel AG & Co. KGaA
* Copyright (c) 2022, 2024 CGI Deutschland B.V. & Co. KG
Copy link
Contributor

Choose a reason for hiding this comment

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

Since the working on dpp-verification was started in year 2023, no?
Copyright (c) 2022, 2024 CGI Deutschland B.V. & Co. KG -> Copyright (c) 2023 CGI Deutschland B.V. & Co. KG?

Also, the license headers are not aligned as in other files/resources from verification.

Copy link
Contributor

@saudkhan116 saudkhan116 left a comment

Choose a reason for hiding this comment

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

All copyright license headers are corrected and updated in PR #351
No IP issues found
LGTM

@matbmoser
Copy link
Contributor Author

Ok all set to do the merge

@matbmoser matbmoser merged commit 89cc92d into main Jul 3, 2024
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
add-on backend Issue or PR connected to the backend application DPP Verification Digital Product Pass Verification and Certification of Assets
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

[DPP] Implementation of Digital Product Pass Verification Concept released in v24.05
2 participants