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

fix https InsecureRequestWarning #34

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Ganofins
Copy link

@Ganofins Ganofins commented Feb 4, 2021

pull request to fix/disable InsecureRequestWarning message

@Ganofins
Copy link
Author

Hey @s0md3v , can you please review this?
Thanks

@s0md3v
Copy link
Owner

s0md3v commented Mar 19, 2021

Insecure HTTPS warnings are already disabled here.

If you think such warnings are still being displayed, feel free to submit a screenshot. https://badssl.com/ is good test bed to test it,

@Ganofins
Copy link
Author

Ganofins commented Mar 21, 2021

Insecure HTTPS warnings are already disabled here.

If you think such warnings are still being displayed, feel free to submit a screenshot. https://badssl.com/ is good test bed to test it,

I have created a POV for ya @s0md3v
https://drive.google.com/file/d/1KWhgsyYRUfso0rwTK0HDwBUfA3THw7Lq/view?usp=sharing

@s0md3v
Copy link
Owner

s0md3v commented Mar 25, 2021

Okay, I did exactly what you did and didn't get any warning. I am not sure what to make of this.

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.

None yet

2 participants