Fixed: Add license header to ServiceContextCleanupFilter.java

[golja]

In the output of the latest run of the buildbot builder ofbizTrunkFrameworkRat, it is reported that the file

rest-api/src/main/java/org/apache/ofbiz/ws/rs/filters/ServiceContextCleanupFilter.java

has an unapproved license. The reason is that the license header is missing.

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[JacquesLeRoux]

Thanks Anahita,

There is also UtilCacheTest.groovy, UtilCacheTestTools.groovy and ServiceContextCleanupFilter.java. I have fixed them directly (easier than adding to your PR) before pushing this PR.

There is also pdftest2.pdf. I have asked Infra about that.

[JacquesLeRoux]

Oops, actually I did also ServiceContextCleanupFilter.java. I close here, sorry and thanks for he effort.

[golja]

Thanks Anahita,

There is also UtilCacheTest.groovy, UtilCacheTestTools.groovy and ServiceContextCleanupFilter.java. I have fixed them directly (easier than adding to your PR) before pushing this PR.

There is also pdftest2.pdf. I have asked Infra about that.

Hi @JacquesLeRoux,
changes to ServiceContextCleanupFilter.java are included in this PR. The other two files you mentioned (UtilCacheTest.groovy, UtilCacheTestTools.groovy) are in the ofbiz-framework repository, not in the ofbiz-plugins repository. That’s why this PR only includes changes to the ServiceContextCleanupFilter.

For your information, alongside this PR I’ve also submitted another PR to ofbiz-framework with changes to the other two files.

Anahita

[JacquesLeRoux]

Hi @golja ,

Yes I know, I added them all Saturday around 9AM, that's why I closed this PR. I don't remember for the framework PR.

[JacquesLeRoux]

Ah, Jacopo closed it: apache/ofbiz-framework#1122

[JacquesLeRoux]

You were right for me mixing framework and plugins, sorry.