chef / omnibus

Easily create full-stack installers for your project across a variety of platforms.
Apache License 2.0
1.29k stars 296 forks source link

Remove authorization from version manifest hash #1094

Closed gcs-devel closed 1 year ago

gcs-devel commented 1 year ago

Signed-off-by: Gregory Schofield grschofi@progress.com

Description

Removes the authorization: key from the manifest because it exposes the artifactory token used. This may cause omnibus packages built with authorization headers to be unable to rebuild from their version manifest. Potentially this could be made possible with a new flag when building from a manifest?

How it looks before this PR

"libxslt": {
      "locked_version": "1.1.35",
      "locked_source": {
        "sha256": "8247f33e9a872c6ac859aa45018bc4c4d00b97e2feac9eebc10c93ce1f34dd79",
        "url": "https://artifactory-internal.ps.chef.co/artifactory/omnibus-software-local/libxslt/libxslt-1.1.35.tar.xz",
        "authorization": "X-JFrog-Art-Api:<redacted>",
        "internal": true
      },
      "source_type": "url",
      "described_version": "1.1.35",
      "license": "MIT"
    },

How it looks after this PR

"libxslt": {
      "locked_version": "1.1.35",
      "locked_source": {
        "sha256": "8247f33e9a872c6ac859aa45018bc4c4d00b97e2feac9eebc10c93ce1f34dd79",
        "url": "https://artifactory-internal.ps.chef.co/artifactory/omnibus-software-local/libxslt/libxslt-1.1.35.tar.xz",
        "internal": true
      },
      "source_type": "url",
      "described_version": "1.1.35",
      "license": "MIT"
    },

Maintainers

Please ensure that you check for:

sonarcloud[bot] commented 1 year ago

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication