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

Disable warnings possible? #25

Closed
dahankzter opened this issue Apr 13, 2016 · 7 comments
Closed

Disable warnings possible? #25

dahankzter opened this issue Apr 13, 2016 · 7 comments
Assignees

Comments

@dahankzter
Copy link

I get this continously:

RESTY 2016/04/13 16:28:18 WARNING - Using Basic Auth in HTTP mode is not secure.

It would be nice if it was (perhaps it already is!) possible to disable warnings.

@dahankzter
Copy link
Author

Thanks for a nice package by the way! ;)

@jeevatkm jeevatkm self-assigned this Apr 14, 2016
@jeevatkm
Copy link
Member

Thanks @dahankzter

This warning was added to recommend best practice. You know it already; exchange credentials in plain text exposes request to vulnerability.

Will provide option disable these warnings, however resty will log warning once. Sounds good?

@jeevatkm jeevatkm added this to the v0.7 Milestone milestone Apr 14, 2016
@dahankzter
Copy link
Author

Sounds perfect, thank you!

@jeevatkm
Copy link
Member

Added method SetDisableWarn at client level, please try it out and let me know.

@jeevatkm jeevatkm closed this as completed May 2, 2016
@dahankzter
Copy link
Author

I will try it as soon as possible, thank you!

@dahankzter
Copy link
Author

Seems to work just fine on a cursory test thank!

@jeevatkm
Copy link
Member

jeevatkm commented May 2, 2016

Thanks for your feedback.

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

No branches or pull requests

2 participants