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

Add TLS disable check helper #103

Closed

Conversation

iongion
Copy link

@iongion iongion commented Feb 10, 2023

Inspiration from

I don't know what to do for Windows and Mac - this is just a shallow attempt

See tryphotino/photino.NET#65

@iongion iongion force-pushed the add-disable-tls-check-method branch 2 times, most recently from 9d379d5 to acf88eb Compare February 11, 2023 08:32
Implement as exemplified

TODO for Mac

Use latest headers
@iongion
Copy link
Author

iongion commented Feb 12, 2023

Closing this as it was just a wild exploration.
I forked all your repos into one big just due to my inability to bootstrap it properly.

I've added the TLS disable mechanism for both windows and linux and it actually works, it is more idiomatic to the project itself.
That is, using an attribute instead of a one time disable function.

https://github.com/iongion/photino-platform

Still, unable to develop and test on macs so can't really come with a valid PR.
But if you plan on adding this, just for the local development scenario, that would be quite helpful.

@iongion iongion closed this Feb 12, 2023
@ottodobretsberger ottodobretsberger added enhancement New feature or request help wanted Extra attention is needed All OS All operating systems labels Feb 16, 2023
@jammerxd
Copy link
Contributor

jammerxd commented Jan 4, 2024

@iongion if you look at #120 I did manage to find the solution for osx since I'm in the same boat as you and want to see this added.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
All OS All operating systems enhancement New feature or request help wanted Extra attention is needed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants