fix: postman headers - Githubissues

eclipse-edc / MinimumViableDataspace

Guidance on documentation, scripts and integration steps on using the EDC project results

Apache License 2.0

58 stars 167 forks source link

Closed nikschul closed 1 month ago

nikschul commented 1 month ago

The headers Authorization is already defined on the collection-level, so adding them to each request is redundant on the individual requests.

Removing the headers on the individual requests, which should make no difference to the actual requests send out by Postman.

Closes #347.

paullatzelsperger commented 1 month ago

@nikschul I self-requested a review, because I think you cannot request one. I'll take a look once you have the PR ready.

nikschul commented 1 month ago

Hi Paul, thanks for reviewing my PR. I have updated the following things inside the Postman collection:

all X-Api-Key headers are removed except for the collection-wide auth definition and the one request where {{IH_API_TOKEN}} is used
Content-Type: application/json is added to each request, which has a non-empty body