Skip to content
New issue

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

#

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

Already on GitHub? # to your account

Jump to bottom

CVE-2018-14643 Downstream security fix #53

Closed
wants to merge 1 commit into from
Closed

CVE-2018-14643 Downstream security fix #53

wants to merge 1 commit into from

Conversation

Klaas-
Copy link

@Klaas- Klaas- commented Sep 20, 2018

@theforeman-bot
Copy link
Member

There were the following issues with the commit message:

  • 0f3f5b6 must be in the format fixes #redmine_number - brief description

If you don't have a ticket number, please create an issue in Redmine.

More guidelines are available in Coding Standards or on the Foreman wiki.

This message was auto-generated by Foreman's prprocessor

@iNecas
Copy link
Member

iNecas commented Sep 20, 2018

Hi, I will push a patch with tests in a minute

@Klaas-
Copy link
Author

Klaas- commented Sep 20, 2018

okay @iNecas

@Klaas- Klaas- closed this Sep 20, 2018
@iNecas
Copy link
Member

iNecas commented Sep 20, 2018

Closing in favor of #54

@adamruzicka
Copy link
Contributor

@Klaas- thank you for picking this up so quickly, @iNecas's version was just merged

# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
Needs testing Not yet reviewed Waiting on contributor
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@Klaas- @theforeman-bot @iNecas @adamruzicka