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

feat | fix : New pcf submodel v 7.0.0 support changes and security file update #201

Merged
merged 4 commits into from
Jul 17, 2024

Conversation

ChetanT-System
Copy link
Contributor

@ChetanT-System ChetanT-System commented Jul 11, 2024

Description

  • New changes for PCF v7.0.0 support.
  • Security config changes.
  • Updated change log and dependency file.

Pre-review checks

Please ensure to do as many of the following checks as possible, before asking for committer review:

return objectNode;
}

public void delete(Integer rowIndex, JsonObject jsonObject, String delProcessId, String refProcessId) {

Check notice

Code scanning / CodeQL

Missing Override annotation Note

This method overrides
EDCUsecaseStep.delete
; it is advisable to add an Override annotation.
public class PCFEDCUsecaseHandler extends Step implements EDCUsecaseStep {

@SneakyThrows
public ObjectNode run(Integer rowNumber, ObjectNode objectNode, String processId, PolicyModel policy) {

Check notice

Code scanning / CodeQL

Missing Override annotation Note

This method overrides
EDCUsecaseStep.run
; it is advisable to add an Override annotation.

String resource = "pcf-v7.0.0.json";
// this is the path within the jar file
InputStream input = this.getClass().getResourceAsStream("/resources/" + resource);

Check warning

Code scanning / CodeQL

Unsafe use of getResource Warning

The idiom getClass().getResource() is unsafe for classes that may be extended.
@ChetanT-System
Copy link
Contributor Author

@sachinargade123, @adityagajbhiye9 Please review and approve the PR.

@almadigabor almadigabor merged commit 38ee6d4 into eclipse-tractusx:main Jul 17, 2024
6 checks passed
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