I think the reason why I converted this to lower case is to make sure that the protocol is correctly identified even if it's given in uppercase. Maybe would it be worth doing the protocol check with the lowercase version of the DSN.

Oh. I pinged the wrong guy. My bad :)

You are right, it would be a good idea convert the case to check for the protocol.

I'll give a try tomorrow.

@ReAzem No problem.
The scheme part(https+influxdb) should be converted to lower case to check, and other parts(like username, password) are case sensitive, in my opinion.

Looks like it already worked. urlparse does the work for us.

@ChristopherRabotin Any feedback? Good to merge?

@ReAzem looks good, nothing to add. Thanks for asking!

Thank you :)!