Recent

Author Topic: [SOLVED] TInetSocket connect timeout patch.  (Read 3215 times)

d.ioannidis

  • Full Member
  • ***
  • Posts: 221
    • Nephelae
[SOLVED] TInetSocket connect timeout patch.
« on: May 16, 2018, 05:49:36 pm »
Hi,

  I created a patch against fpc trunk for the TInetSocket which is used from TFPHTTTPClient to add support for timeout during connect. Also there is a patch for the TFPHTTPClient to use this functionality.

  The patches are in the bug tracker issue 33745 TInetSocket connect timeout .

  Tested in Windows 10 and Debian 9 both x86_64.

  As I didn't do any testing with 32 bit Windows/Linux or at all in FreeBSD, I'll appreciate any help to see if there is any problem or side effects on those platforms and inform me here.

Best regards,
« Last Edit: June 09, 2018, 09:20:22 pm by Dimitrios Chr. Ioannidis »

d.ioannidis

  • Full Member
  • ***
  • Posts: 221
    • Nephelae
Re: TInetSocket connect timeout patch.
« Reply #1 on: June 09, 2018, 09:19:51 pm »
Hi all,

  FYI, the patch is accepted in trunk rev: 39199.

engkin

  • Hero Member
  • *****
  • Posts: 3112
Re: [SOLVED] TInetSocket connect timeout patch.
« Reply #2 on: June 09, 2018, 09:37:25 pm »
Thank you for the patch and the bug report.

It seems that Michael forgot to remove unit math from the uses section. I can not write a note regarding that as he marked it "resolved", I believe you should be able to change the status and/or leave a note.

Thank you again.

d.ioannidis

  • Full Member
  • ***
  • Posts: 221
    • Nephelae
Re: [SOLVED] TInetSocket connect timeout patch.
« Reply #3 on: June 09, 2018, 09:48:27 pm »
Hi,

It seems that Michael forgot to remove unit math from the uses section. I can not write a note regarding that as he marked it "resolved", I believe you should be able to change the status and/or leave a note.

I already send him an email. I hadn't thought to use the tracker ....  :(

[EDIT]: I updated the tracker also ...

regards,
« Last Edit: June 09, 2018, 10:00:50 pm by Dimitrios Chr. Ioannidis »

engkin

  • Hero Member
  • *****
  • Posts: 3112
Re: [SOLVED] TInetSocket connect timeout patch.
« Reply #4 on: June 09, 2018, 10:21:36 pm »
And it got fixed.  :D

Thaddy

  • Hero Member
  • *****
  • Posts: 14204
  • Probably until I exterminate Putin.
Re: [SOLVED] TInetSocket connect timeout patch.
« Reply #5 on: June 09, 2018, 10:26:47 pm »
Yup! :)
Specialize a type, not a var.

 

TinyPortal © 2005-2018