Skip to content
This repository has been archived by the owner on Jan 7, 2020. It is now read-only.

Do not serialize the TLS config #753

Merged
merged 1 commit into from
Jan 10, 2018
Merged

Do not serialize the TLS config #753

merged 1 commit into from
Jan 10, 2018

Conversation

palourde
Copy link
Contributor

Signed-off-by: Simon Plourde simon.plourde@gmail.com

Description

It prevents the unsupported type: func() time.Time error when serializing the config to JSON.

Related Issue

N/A

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING document.
  • I have added tests to cover my changes.
  • All new and existing tests passed.

@palourde
Do not serialize the TLS config
59df5f9 
Signed-off-by: Simon Plourde <simon.plourde@gmail.com>
@palourde palourde added this to the 1.1.1 milestone Jan 10, 2018
@palourde palourde merged commit 846b273 into sensu:master Jan 10, 2018
@palourde palourde deleted the bugfix/config-info branch January 10, 2018 16:26
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
1.1.1
Development

Successfully merging this pull request may close these issues.
1 participant
@palourde