Skip to content
GitLab
Projects Groups Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in / Register
  • A aem-hacker
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 12
    • Issues 12
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 2
    • Merge requests 2
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Packages and registries
    • Packages and registries
    • Package Registry
    • Infrastructure Registry
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • 0ang3el
  • aem-hacker
  • Merge requests
  • !8

Prevent extra headers to override script defined headers

  • Review changes

  • Download
  • Email patches
  • Plain diff
Closed Administrator requested to merge github/fork/NotBikappa/master into master Sep 27, 2019
  • Overview 0
  • Commits 4
  • Pipelines 0
  • Changes 1

Created by: NotBikappa

The goal of this PR is to give precedence to the headers defined by the aem_hacker script over the extra headers passed as command line arguments.

In some scenario the user might need to configure the 'Authorization' header because his target environment is protected with basic authentication, for example.

The aem_hacker scripts also need to configure that header for some checks, aiming to test the configuration with default credentials.

If the user defined headers are used those checks are not effective

Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: github/fork/NotBikappa/master