Welcome, Guest. Please login or register. Did you miss your activation email?

Author Topic: Broken HTTPS code stub  (Read 3217 times)

0 Members and 1 Guest are viewing this topic.

binary1248

  • SFML Team
  • Hero Member
  • *****
  • Posts: 1405
  • I am awesome.
    • View Profile
    • The server that really shouldn't be running
Broken HTTPS code stub
« on: September 09, 2013, 09:05:34 am »
I already had a feeling that SFML didn't support HTTPS due to the fact that it doesn't link in any SSL libraries, so I checked the source just for fun. I found some code that parses https:// URIs and connects to the server on port 443 to send a "normal" HTTP request. Is this really necessary? I don't think it's a good idea to pretend to the user that SFML has invisible HTTPS support and just fail every connection attempt after sending potentially sensitive data (yes... this might be a security risk). An error/warning message on the console should be enough when trying to do this.

Until you decide on real HTTPS support can't you just prevent that needless connection establishment from taking place and tell the user HTTPS support doesn't exist yet? You said yourself here that you would think about removing the code but until now it still exists.
SFGUI # SFNUL # GLS # Wyrm <- Why do I waste my time on such a useless project? Because I am awesome (first meaning).

Laurent

  • Administrator
  • Hero Member
  • *****
  • Posts: 32504
    • View Profile
    • SFML's website
    • Email
Re: Broken HTTPS code stub
« Reply #1 on: September 09, 2013, 09:32:38 am »
Yes, I will remove the corresponding code.

Thanks for reminding me.
Laurent Gomila - SFML developer

Laurent

  • Administrator
  • Hero Member
  • *****
  • Posts: 32504
    • View Profile
    • SFML's website
    • Email
Re: Broken HTTPS code stub
« Reply #2 on: September 16, 2013, 10:31:54 pm »
Done.
Laurent Gomila - SFML developer

FRex

  • Hero Member
  • *****
  • Posts: 1845
  • Back to C++ gamedev with SFML in May 2023
    • View Profile
    • Email
Re: Broken HTTPS code stub
« Reply #3 on: April 23, 2014, 07:16:26 pm »
Sorry for this idiotically big grave dig but since the https 'support' has been removed the comments for doxygen docs are wrong in Http.hpp because they say that if not specified the right port is chosen automatically 'according to the protocol used (80 for HTTP, 443 for HTTPS)' suggesting that https is supported.
Back to C++ gamedev with SFML in May 2023

Nexus

  • SFML Team
  • Hero Member
  • *****
  • Posts: 6286
  • Thor Developer
    • View Profile
    • Bromeon
Re: Broken HTTPS code stub
« Reply #4 on: April 23, 2014, 07:32:37 pm »
Indeed. I fixed it, thanks. I also added an explicit note that HTTPS is not supported.
Zloxx II: action platformer
Thor Library: particle systems, animations, dot products, ...
SFML Game Development:

 

anything