Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update reportcommon.py #1516

Merged
merged 1 commit into from
Jun 27, 2023
Merged

Update reportcommon.py #1516

merged 1 commit into from
Jun 27, 2023

Conversation

tuxudo
Copy link
Contributor

@tuxudo tuxudo commented Jun 26, 2023

Fix config bug discovered in issue #1515

Fix config bug discovered in issue munkireport#1515
@bochoven
Copy link
Contributor

Was this key always wrong? Or was there a reason it did not contain munki?

@tuxudo
Copy link
Contributor Author

tuxudo commented Jun 26, 2023

I changed this key in changing from Python 2 to 3 to a new incorrect key. Documentation, existing configs, and another script all reference it as being UseMunkiAdditionalHttpHeaders

@bochoven bochoven merged commit 12a41b3 into munkireport:5.x Jun 27, 2023
tuxudo added a commit to tuxudo/munkireport-php that referenced this pull request Jun 29, 2023
Additional fix for issue munkireport#1515 and PR munkireport#1516
@tuxudo tuxudo mentioned this pull request Jun 29, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants