feat(rest): Added new endpoints to update attachment info of Project, Component, Release

[JaideepPalit]

Added new endpoints to update attachment info of Project, Component, Release.

• Which issue is this pull request belonging to and how is it solving it? (REST API: allow modification of attachment meta info (checkStatus) #1189)
• Did you add or update any new dependencies that are required for your change? - No

Issue: closes #1189

How To Test?

Refer Rest Documentation
http://localhost:8080/resource/docs/api-guide.html#resources-project-attachment-info-update
Example Request -

curl 'https://sw360.org/api/projects/98745/attachment/1234' -i -X PATCH 
-H 'Content-Type: application/hal+json' -H 'Authorization: Bearer **********' -H 'Accept: application/hal+json' 
-d '{
  "attachmentType" : "BINARY",
  "createdComment" : "Created Comment",
  "checkedComment" : "Checked Comment",
  "checkStatus" : "ACCEPTED"
}'

http://localhost:8080/resource/docs/api-guide.html#resources-component-attachment-info-update
Example Request -

curl 'https://sw360.org/api/components/98745/attachment/1234' -i -X PATCH 
-H 'Content-Type: application/hal+json' -H 'Authorization: Bearer *******' -H 'Accept: application/hal+json' 
-d '{
  "attachmentType" : "BINARY",
  "createdComment" : "Created Comment",
  "checkedComment" : "Checked Comment",
  "checkStatus" : "ACCEPTED"
}'

http://localhost:8080/resource/docs/api-guide.html#resources-release-attachment-info-update
Example Request -

curl 'https://sw360.org/api/releases/98745/attachment/1234' -i -X PATCH 
-H 'Content-Type: application/hal+json' -H 'Authorization: Bearer *************' -H 'Accept: application/hal+json' 
-d '{
  "attachmentType" : "BINARY",
  "createdComment" : "Created Comment",
  "checkedComment" : "Checked Comment",
  "checkStatus" : "ACCEPTED"
}'

Have you implemented any additional tests? - No

Checklist

Must:

• All related issues are referenced in commit messages and in PR

Signed-off-by: Jaideep Palit jaideep.palit@siemens.com

[akapti]

Reviewed the code, suggested some minor changes.
Tested the changes, and all the 3 endpoints working as expected.

[JaideepPalit]

Thanks for review and test.
Incorporated changes based on review comments

[akapti]

Code looks good.
Tested the PR again after changes, and is working fine.

[Lythenas]

Is there a reason that the patch url is different from the get url?

To get the attachment infos you need to use https://sw360.org/api/releases/1234/attachments/1234
but to patch the same infos you need to use https://sw360.org/api/releases/1234/attachment/1234. I think this is really inconvenient.

It would also be nice if attachment infos could be updated through the self link (i.e. https://sw360.org/api/attachments/1234)

[mcjaeger]

so, it is about the "s" ... I agree, but I am not sure how much trouble we cause will there be in case we change it