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

ipmi_sensor: allow @ symbol in password #2633

Merged
merged 2 commits into from
Apr 6, 2017

Conversation

vtg
Copy link
Contributor

@vtg vtg commented Apr 6, 2017

Required for all PRs:

  • CHANGELOG.md updated
  • Sign CLA

@danielnelson
Copy link
Contributor

Looks great, and special thanks for adding the unittest. Can you sign the CLA and add this to changelog and we can merge.

@vtg
Copy link
Contributor Author

vtg commented Apr 6, 2017

No problem. Should be signed now

@danielnelson
Copy link
Contributor

I still not seeing you in our CLA list, can you resign it?

@vtg
Copy link
Contributor Author

vtg commented Apr 6, 2017

done

@danielnelson danielnelson merged commit 92fa20c into influxdata:master Apr 6, 2017
@danielnelson
Copy link
Contributor

Thanks!

@vtg vtg deleted the impi_sensors_password_fix branch April 6, 2017 21:53
vlamug pushed a commit to vlamug/telegraf that referenced this pull request May 30, 2017
maxunt pushed a commit that referenced this pull request Jun 26, 2018
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