0037249: For server to server requests set read and connection time out

Openbravo Issue Tracking System - Retail Modules

View Issue Details

Project

Category

View Status

Date Submitted

Last Update

0037249

Retail Modules

StoreServer

public

2017-11-07 13:37

2017-11-08 11:44

Reporter

mtaal

Assigned To

mtaal

Priority

normal

Severity

minor

Reproducibility

have not tried

Status

closed

Resolution

fixed

Platform

OS Version

Product Version

Target Version

Fixed in Version

RR18Q1

Merge Request Status

Review Assigned To

AugustoMauch

OBNetwork customer

Support ticket

Regression level

Regression date

Regression introduced in release

Regression introduced by commit

Triggers an Emergency Pack

Summary

0037249: For server to server requests set read and connection time out

Description

The case is described in this [1] document. When doing server to server requests [2] no time out is set. This means that the system uses the default (infinite wait) for 2 timeouts:
- connect timeout: timeout for creating the request connection between servers
- read timeout: timeout in which the calling server waits for the receiving server to process the request. The read timeout applies after the connection to the other server has been made.

The system default is infinite wait.

[1]
https://docs.google.com/document/d/1ZNgmx_AWu0t8zBuXwgkeHH6sFsb_X-G8k7KyWxkAaHQ/edit?ts=5a018088# [^]

Steps To Reproduce

Install multi-server
Put a breakpoint in the request executor when it catches an exception [1]
Put a breakpoint in the central server OrderLoader
Goto webpos, enter ticket and submit
skip the breakpoint in the store server, let it wait on the breakpoint in the CS.

The request from SS to CS does not timeout. If it would timeout a timeout exception should be caught at [1]

[2]
https://code.openbravo.com/erp/pmods/org.openbravo.mobile.core/file/ca39e197ccae/src/org/openbravo/mobile/core/servercontroller/MobileServerRequestExecutor.java#l197 [^]

Proposed Solution

Default connect timeout at 5 seconds and the readtimeout at 15 minutes. Do the change here [1]. Make the timeouts configurable if possible.

[1]
https://code.openbravo.com/erp/pmods/org.openbravo.mobile.core/file/ca39e197ccae/src/org/openbravo/mobile/core/servercontroller/MobileServerRequestExecutor.java#l70 [^]

Additional Information

Tags

No tags attached.

Relationships

related to	defect	0036884		closed	mtaal	[SERQA 3289] If we stop the instance instead of tomcat we get a timeout paying a ticket
blocks	feature request	0037379		closed	mtaal	In mobile server request executor when read time out occurs then do not go offline