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

Heartbeat ports exported as string #5696

Closed
urso opened this issue Nov 23, 2017 · 3 comments
Closed

Heartbeat ports exported as string #5696

urso opened this issue Nov 23, 2017 · 3 comments
Labels
bug Heartbeat Team:obs-ds-hosted-services Label for the Observability Hosted Services team

Comments

@urso
Copy link

urso commented Nov 23, 2017

Just noticed port numbers in the json document exported by heartbeat are string, not numbers. See sample output from event refactoring: #4091

The fields.yml file already configures the port number to be an integer.

@ruflin
Copy link
Member

ruflin commented Nov 28, 2017

I just tested this on master and it seems to be integer there now. Must be a change between 5.x and 6.x where we fixed this?

The initial thread where this popped up: https://discuss.elastic.co/t/what-type-is-the-port-field-in-heartbeat-output/108586/11

@ruflin ruflin added Team:obs-ds-hosted-services Label for the Observability Hosted Services team and removed Team:obs-ds-hosted-services Label for the Observability Hosted Services team labels Dec 3, 2018
@elasticmachine
Copy link
Collaborator

Pinging @elastic/uptime

@andrewvc
Copy link
Contributor

Given that it's fixed in 6.x I'm going to close this since 5.x isn't supported.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Heartbeat Team:obs-ds-hosted-services Label for the Observability Hosted Services team
Projects
None yet
Development

No branches or pull requests

4 participants