Bug #14606: Gazillions of "InsecureRequestWarning: Unverified HTTPS request is being made" in logs - Rudder - Issue Tracker

Actions

Copy link

Bug #14606

closed

Gazillions of "InsecureRequestWarning: Unverified HTTPS request is being made" in logs

Added by Nicolas CHARLES over 5 years ago. Updated over 2 years ago.

Status:

Rejected

Priority:

N/A

Assignee:

Category:

Web - Technique editor

Target version:

6.0.10

Pull Request:

Severity:

Minor - inconvenience | misleading | easy workaround

UX impact:

User visibility:

Operational - other Techniques | Rudder settings | Plugins

Effort required:

Priority:

Name check:

Fix check:

Regression:

Description

each entry are validated by the technique editor when we type them
if certificate is not defined, we get a lot of


[Mon Apr 08 13:14:22.581979 2019] [wsgi:error] [pid 7443] /usr/share/ncf-api-virtualenv/lib/python2.7/site-packages/urllib3/connectionpool.py:857: InsecureRequestWarning: Unverified HTTPS request is being made. Adding certificate verification is strongly advised. See: https://urllib3.readthedocs.io/en/latest/advanced-usage.html#ssl-warnings
[Mon Apr 08 13:14:22.582008 2019] [wsgi:error] [pid 7443]   InsecureRequestWarning)

this is fairly annoying

Actions

Copy link

Updated by Nicolas CHARLES over 5 years ago

Status changed from New to In progress
Assignee set to Nicolas CHARLES

Actions

Copy link

Updated by Nicolas CHARLES over 5 years ago

Status changed from In progress to New
Assignee deleted (~~Nicolas CHARLES~~)

if request >= 2.16.0, we should use import urllib3
urllib3.disable_warnings(urllib3.exceptions.InsecureRequestWarning)

https://stackoverflow.com/questions/27981545/suppress-insecurerequestwarning-unverified-https-request-is-being-made-in-pytho

as i'm not sure of which version of python we use everywhere, i'd rather leave it to more expert people

Actions

Copy link

Updated by Vincent MEMBRÉ over 5 years ago

Target version changed from 5.0.9 to 5.0.10

Actions

Copy link

Updated by François ARMAND over 5 years ago

I don't think we want to set that property, it would hide a potentially serious problem.
AFAIU, it only happens with self-signer certificate ? So it should not happen in a real rudder installation ?

Actions

Copy link

Updated by Vincent MEMBRÉ over 5 years ago

Target version changed from 5.0.10 to 5.0.11

Actions

Copy link

Updated by Vincent MEMBRÉ over 5 years ago

Target version changed from 5.0.11 to 5.0.12

Actions

Copy link

Updated by Vincent MEMBRÉ over 5 years ago

Target version changed from 5.0.12 to 5.0.13

Actions

Copy link

Updated by Vincent MEMBRÉ over 5 years ago

Target version changed from 5.0.13 to 5.0.14

Actions

Copy link

Updated by Vincent MEMBRÉ about 5 years ago

Target version changed from 5.0.14 to 5.0.15

Actions

Copy link

#10

Updated by Vincent MEMBRÉ about 5 years ago

Target version changed from 5.0.15 to 5.0.16

Actions

Copy link

#11

Updated by Alexis Mousset almost 5 years ago

Target version changed from 5.0.16 to 5.0.17

Actions

Copy link

#12

Updated by Vincent MEMBRÉ over 4 years ago

Target version changed from 5.0.17 to 5.0.18

Actions

Copy link

#13

Updated by François ARMAND over 4 years ago

Severity set to Minor - inconvenience | misleading | easy workaround
User visibility set to Operational - other Techniques | Rudder settings | Plugins
Priority changed from 0 to 27