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

Sensu Enterprise API SSL attributes are incorrectly configured #784

Closed
cwjohnston opened this issue Jul 27, 2017 · 1 comment
Closed

Sensu Enterprise API SSL attributes are incorrectly configured #784

cwjohnston opened this issue Jul 27, 2017 · 1 comment
Assignees

Comments

@cwjohnston
Copy link
Contributor

cwjohnston commented Jul 27, 2017

Description of problem

  • What did you do?

Configured api_ssl_port, api_ssl_keystore_file and api_ssl_keystore_password parameters

  • What did you expect to happen?

Expected rendered configuration to match documented Sensu Enterprise SSL attributes. e.g.

{
  "api": {
    "ssl": {
      "port": 4568,
      "keystore_file": "/etc/sensu/api.keystore",
      "keystore_password": "secret"
    }
  }
}
  • What happened?

API SSL parameter values were rendered under api scope but do not match documented configuration attributes:

{
  "api": {
    "ssl_port": 4568,
    "ssl_keystore_file": "/etc/sensu/api.keystore",
    "ssl_keystore_password": "secret"
  }
}
  • How can someone reproduce the problem?

Use the module to configure Sensu Enterprise with api_ssl_port, api_ssl_keystore_file and api_ssl_keystore_password parameters

Anything else to add that you think will be helpful?

It appears that the specs for this module feature don't align with the documented Sensu Enterprise API configuration specifications.

@jeffmccune jeffmccune self-assigned this Jul 31, 2017
jeffmccune added a commit to jeffmccune/sensu-puppet that referenced this issue Jul 31, 2017
Without this path the Puppet module incorrectly configures Sensu Enterprise API
SSL parameters.  This patch corrects the configuration scope to match
https://sensuapp.org/docs/1.0/enterprise/api.html#ssl-attributes-example

Resolves sensu#784
jeffmccune added a commit to jeffmccune/sensu-puppet that referenced this issue Jul 31, 2017
Without this path the Puppet module incorrectly configures Sensu Enterprise API
SSL parameters.  This patch corrects the configuration scope to match
https://sensuapp.org/docs/1.0/enterprise/api.html#ssl-attributes-example

Resolves sensu#784
jeffmccune added a commit to jeffmccune/sensu-puppet that referenced this issue Jul 31, 2017
Without this path the Puppet module incorrectly configures Sensu Enterprise API
SSL parameters.  This patch corrects the configuration scope to match
https://sensuapp.org/docs/1.0/enterprise/api.html#ssl-attributes-example

Resolves sensu#784
ghoneycutt added a commit that referenced this issue Jul 31, 2017
(#784) Fix Sensu Enterprise API SSL configuration scope
@ghoneycutt
Copy link
Collaborator

Thanks @cwjohnston for the great bug report.

Released in v2.30.1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants