Finished adding the ability to add headers during auth attempts

ztgrace / changeme

A default credential scanner.

GNU General Public License v3.0

1.44k stars 248 forks source link

Finished adding the ability to add headers during auth attempts #10

Closed Graph-X closed 7 years ago

Graph-X commented 7 years ago

The yaml schema already allowed for auth > headers:, but the python did not have the functionality included. Updated the teleopti_wfm.yml as well becuase some servers will return 500 error if a content-type header does not specify application/json. This may be the case with other apps moving forward so I figured I would finish what looked like a near completion of that capability

ztgrace commented 7 years ago

Looks like some of the unit tests are failing. Make sure nosetests -v -s --with-coverage --cover-erase --cover-package=changeme tests/*.py" runs successfully. I've added that line to my .git/hooks/pre-commit file to make sure I don't break anything with my changes.

ztgrace commented 7 years ago

Thanks for the PR! Still had some build errors, but fixed them with this commit: https://github.com/ztgrace/changeme/commit/d6c036e75f83caa4cfcaea2f478fa16f02fa1a67

Graph-X commented 7 years ago

Glad I could help. I see what you mean about the key errors now too. Should have some new creds for you soon too.