summaryrefslogtreecommitdiff
path: root/src/components/protocol_handler
Commit message (Collapse)AuthorAgeFilesLines
* Fix tests in resumption, policy and protocol_handlerKozoriz2016-04-151-1/+5
|
* Change magic numbers to const vars and rename some varsiAndrew52016-04-053-31/+33
| | | | | 1) Magic number replaced with constant variables 2) Renamed enums, variables according google coding style
* Change other unit-tests according new code changesiAndrew52016-04-044-283/+378
| | | | | | | | | | | | | 1) Changed header files in tests 2) Renamed transport_manager_mock to mok=ck)transport_manager 3) Renamed other mock classes to match naming rules 4) Deleted redundant headers and namespaces. 5) Changed year in Ford headers 6) Changed raw "new" operator using to MakeShared for creating shared_ptr 7) Corrected header guards 8) Added new method CheckRegularMatches to control_message_matcher Relates: APPLINK-22767
* Complete covering of Protocol handleriAndrew52016-04-042-461/+988
| | | | | | | | | 1) Modified existing tests cases 2) Added new tests 3) Changed using entire namespaces to using particular entities from those namespaves 4) Changed header files according new code changes. Relates: APPLINK-22767
* Changes after reviewKozoriz2016-03-311-1/+0
| | | | | | Fixed LastState tests SharedPtr changed to auto_ptr in some cases Corrected consts in some cases
* ProtocolHandler test correctivesKozoriz2016-03-313-138/+7
|
* ProtocolHandler correctives after singletons removingKozoriz2016-03-312-4/+15
|
* Merge pull request #414 from LuxoftSDL/fix/Possible_core_fixAGaliuzov2016-03-181-13/+7
|\ | | | | Possible fix for core dump seen by PASA
| * Possible fix for core dump seen by PASAAndrey Oleynik2016-03-111-13/+7
| | | | | | | | | | | | | | | | | | Couple checks have been added. One of these was indicated by Coverity as BAD_FREE error. Closes-bug: APPLINK-21552 Removes odd new/delete call
* | TimeTester refatctoring.Kozoriz2016-03-105-36/+96
| | | | | | | | | | | | | | | | | | | | Removed singletone using from TimeTester. TimeTester renamed -> TelemetryMonitor. All files and directories corrected. Tests corrected, all works. Disabled 2 tests(task created). Related to : APPLINK-21915
* | Review correctives and buildability correctives.Kozoriz2016-03-071-1/+0
| |
* | Remove singleton from ProtocolHandlerKozoriz2016-03-049-541/+340
| | | | | | | | | | | | | | | | Remove singletone. Corrected tests. Created new mocks and settings. Related to : APPLINK-21440
* | Fix dead locks if user register 30 and more applicationsdtrunov2016-02-031-6/+4
|/ | | | | | | | | | | | | | | | | | | | | | | | | | | | | During processing this defect I found 3 dead locks in components: conection_handler, hearbeat_monitor. First deadlock: First thread: Calls KeepConnectionAlive() from connection_handler in this method thread locked connection_list_lock_, after that calls method KeepAlive() from heartbeat_monitor and thread tries to lock sessions_list_lock_ wich was locked by second thread Second thread: Blocks sessions_list_lock_ in method Process() from heartbeat_monitor after that sends hearbeat to mobile and calls method SendHeartBeat() from protocol_handler. In this method calls ProtocolVersionUsed() where thread tries to lock connection_list_lock_ wich was locked by first thread -> DEAD LOCK I fixed this problem using RWLock(). Second and third dead locks were associated with time of action of sessions_list_lock in method Process() from heartbeat_monitor I decreased time of action this lock in order to fix this problem. Closes-bug:[APPLINK-19632](https://adc.luxoft.com/jira/browse/APPLINK-19632)
* Code beautifyingAnatoly Leshin2016-01-212-9/+9
|
* Process EndServiceAck from mobile;Anatoly Leshin2016-01-212-7/+44
| | | | | | | | Add EndServiceAck message processing on protocol_handler level. PH notify connection_handler, CH notify application_manager. On AM level SDL turns off video/audio service for application. Fix: APPLINK-19170, APPLINK-19923
* Fixed wrong logger name for ProtocolHandlerElisey Zamakhov2015-12-183-4/+6
| | | | Issues: APPLINK-19597, APPLINK-19603
* hotfix/Fix_UT_for_Protocol_handler_in_geniviHerasym Oleh2015-12-021-5/+5
| | | | | | | Enabled 5 unit tests. Related: APPLINK-19076
* Implement Multiframe assembling by MessageIDElisey Zamakhov2015-11-0613-191/+1094
| | | | | | | | | | | | | | | | | | | Each Session resposible for assembling multiframes according to MessageID Implemented MultiframeBuilder support class Add MultiframeBuilder tests Integrated MultiframeBuilder usage in PHimpl with pop all expired frames on each handled frame on any session of any connection Added Configurable parameter in ConfigProfile for expirated time configuration Updated PASA and Genevi ini files Fixed crash on printing empty ProtocolFramePtr Added ProtocolFramePtrList, ConnectionID typedefs to protocol_packet.h Removed redundant ConnectionID usage in PHImpl Fixed memoty leak in ProtocolPacket Code style fixs in PHIMpl. IncomingDataHandler Issues:CRQ - APPLINK-17629, Defect fixed indirectly - APPLINK-17954
* Merge branch 'release/4.0.0' into developiAndrew52015-10-291-5/+5
|\
| * Moved, Enabled and partially refactored Unit testsiAndrew52015-10-209-57/+860
| |
| * Disabled some tests in CmakeLists to enable compilation with TESTSiAndrew52015-10-191-1/+1
| |
* | Added, refactored UT for protocol_handleriAndrew52015-10-289-51/+854
| |
* | Disabled some tests in CmakeLists to enable compilation with TESTSiAndrew52015-10-281-1/+1
| |
* | Add MultiFrame assembling verification with DCHECKsElisey Zamakhov2015-10-265-112/+174
| | | | | | | | | | | | | | | | | | | | | | | | Added multiframes frame data and connection key verifications Add DCHECK for ProtocolHandlerImpl::OnTMMessageSendFailed Implemented ProtocolHeader and ProtocolPacket streaming to basic_ostream Fix DCHECK logging appender to Utils Minor logging fixs in ProtocolHandlerImpl, ProtocolPacket Time_tester include and build fixs cpplint fixs Issue:APPLINK-17144, APPLINK-16741
* | Merge pull request #228 from LuxoftSDL/hotfix/anosach-bugfixAGaliuzov2015-10-051-0/+2
|\ \ | | | | | | Fixes for some components
| * | Check for empty queue in MessageQueue::pop() methodArtem Nosach2015-10-041-0/+2
| | | | | | | | | | | | | | | | | | | | | | | | Return bool value from MessageQueue::pop() method that shows if pop operation was successful. Add default constructors to all classes used as message for message queue. Closes-bug: APPLINK-14928
* | | Trigger update in case of certificate to be expiredAlexandr Galiuzov2015-10-041-0/+3
| | | | | | | | | | | | | | | | | | | | | In case when certificate from policy table will be expired in less then defined in config file hours amount the PTU will be triggered. Closes-Issue:APPLINK-12302
* | | Implement getting additional counters for handshakeAlexandr Galiuzov2015-10-041-6/+10
|/ / | | | | | | | | | | | | | | Add additional counter to policy for handshake procedure. In case something wrong with handshake this counter will be incremented. Implement: APPLINK-12596 Closes-Issue: APPLINK-12598
* | Replace legacy _EXT/INFO macroses in sourceElisey Zamakhov2015-08-311-1/+1
| | | | | | | | | | | | | | | | Removed legacy macro with _EXT extention Replaced INFO macros to TRACE and debug remove symlinks Implements:APPLINK-15680
* | Merge remote-tracking branch 'origin/release/4.0.0' into developAlexandr Galiuzov2015-08-311-14/+20
|\ \ | |/
| * Post review fixesArtem Nosach2015-08-102-12/+7
| | | | | | | | Implements: APPLINK-14656
| * Check MTU size according to new rulesArtem Nosach2015-08-062-19/+38
| | | | | | | | | | | | | | - Use 1488 bytes MTU for protocol versions 1 and 2 - Use specified in ini file MTU size for protocol versions 3 and higher Implements: APPLINK-14656
* | Merge pull request #198 from LuxoftSDL/DeleteCommand_WARNINGAGaliuzov2015-08-283-9/+9
|\ \ | | | | | | | | | Fix WARNING result interpretation in Mobile.DeleteCommandRequest
* | | Add logs to ProtocolPacket::ProtocolHeaderValidatorElisey Zamakhov2015-08-141-12/+49
|/ / | | | | | | | | | | Added autotrace for big methods, warnings for validation fail Implements: APPLINK-15573
* | Not a subject for review.Anatoly Leshin2015-07-151-2/+5
| | | | | | | | Indents, comments changes.
* | Fix malformed messages counting mechanismAnatoly Leshin2015-07-153-8/+14
|/ | | | | | | We count now only switches from correct message to malformed. Continous malformed message of any length is one malformed message. Fixes: APPLINK-13938
* Post review changes.Artem Nosach2015-06-231-4/+13
|
* Merge remote-tracking branch 'origin/APPLINK-11124' into release/4.0.0Aleksandr Galiuzov2015-04-171-18/+19
|\
| * Log level modificationElisey Zamakhov2015-04-031-18/+19
| |
* | Fix UB on reading non-4 byte address alignmentElisey Zamakhov2015-03-271-5/+12
| |
* | Review fixesElisey Zamakhov2015-03-182-5/+5
| |
* | Valgrind warning fixsElisey Zamakhov2015-03-182-1/+3
| | | | | | | | Add additional logs
* | Update malformed message occurence counting for stream with mix ofElisey Zamakhov2015-03-185-27/+72
| | | | | | | | | | malformed and comman packets Update tests
* | Add malformed messages filtering disableElisey Zamakhov2015-03-183-54/+117
| | | | | | | | | | Update messages and malformed messages frequency counting setup Update pasa ini file
* | Change malformed messages counting strategyElisey Zamakhov2015-03-182-3/+85
| |
* | Code style fixElisey Zamakhov2015-03-181-35/+39
| | | | | | | | No subject for review
* | Review fixElisey Zamakhov2015-03-182-11/+11
| |
* | Add MalformedLimitVerification testsElisey Zamakhov2015-03-184-43/+102
| | | | | | | | | | Add log messages Review/gstyle fixs
* | Update MalformedPacket_AdditionalByte testElisey Zamakhov2015-03-181-2/+77
| | | | | | | | Add MalformedPacket_Mix test
* | Improve incoming_data_handler speedElisey Zamakhov2015-03-183-17/+81
| | | | | | | | Add test with unexpected bytes in stream