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 not working properly? #366

Open
Harm10 opened this issue Nov 26, 2022 · 4 comments
Open

Disable warnings not working properly? #366

Harm10 opened this issue Nov 26, 2022 · 4 comments

Comments

@Harm10
Copy link

Harm10 commented Nov 26, 2022

I am using version 0.9.4 on RF 4.1.3.
I have weird behavior of disable_warnings=1.
I set this option on create session together with max_retries=5.
To check if I have a proper connection I execute a valid command on get on session.
If the connection is bad I get 5 warnings on console before the keyword fails.
I can suppress these warnings if I add disable_warnings=1 on get on session!
If the connection is OK I get an error:
TypeError: request() got an unexpected keyword argument 'disable_warnings'

Looking at the documentation disable_warnings is not allowed on get on session.
But I do not want those warnings displayed on console for a bad connection and can successfully suppress them by using this parm on get on session.
Following the documentation apparently the disable_warnings set on create session is not inherited for the get on session?

Is this a bug?

@Harm10
Copy link
Author

Harm10 commented Dec 10, 2022

Can I please get some response on my report?

@Harm10
Copy link
Author

Harm10 commented Jan 30, 2023

@lucagiove Can you please give me some reply on my issue report?

@lucagiove
Copy link
Member

To be honest I don't like how it has been implemented the disable_warnings parameter.
I think I'll remove it in next versions, you could reduce the logging level on your own directly in robot without the needs of this parameter that does also other not straight forward things.

@Harm10
Copy link
Author

Harm10 commented Feb 25, 2023

I totally missed your reply. I will experiment with log level of RF and report back.

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

No branches or pull requests

2 participants