summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
...
| * Feature/sdl 0236 tire status mismatch (#3583)Iryna Lytvynenko (GitHub)2021-08-0412-11/+305
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Temporary commit with link to rpc_spec fork * Indicate that the structure is included in the Common interface * Mark the parameters as non mandatory * Update link to rpc_spec fork * Fix Message Helper linkage to VehicleInfoPlugin * Fix versioning for TirePressure param * Bump version up to 8.0 * Add link to rpc_spec fork * Address review comments Co-authored-by: Dmitriy Boltovskiy <dboltovskyi@luxoft.com> Co-authored-by: Yana Chernysheva <ychernysheva@luxoft.com> Co-authored-by: Andrii Kalinich <AKalinich@luxoft.com>
| * remove reference to ui get capabilities response parameter ↵Collin2021-07-292-24/+0
| | | | | | | | systemCapabilities.displayCapabilities (#3746)
| * Suspend notifications to mobile until HMI state is established (#3675)Jacob Keeler2021-07-272-4/+11
| | | | | | * Suspend any notifications meant for mobile app until HMI state is properly established
| * Fix resumption for menuLayout global property (#3733)Andrii Kalinich (GitHub)2021-07-229-1/+41
| | | | | | | | | | | | | | * Added methods get and set the menuLayout and added to the request on the HMI this field * fixup! Added methods get and set the menuLayout and added to the request on the HMI this field Co-authored-by: Roman Reznichenko <roman.reznichenko@dxc.com>
| * adding the interface name to the types in HMI_API.xml (#3742)Andrii Kalinich (GitHub)2021-07-221-72/+72
| | | | | | Co-authored-by: Roman Reznichenko <roman.reznichenko@dxc.com>
| * Fix cmake warning under external flow (#3730)Andrii Kalinich (GitHub)2021-07-222-1/+2
| | | | | | * Established the new policy for CMP0054 and added if and else for different Policy values
| * Remove Deprecated Methods For SDL Core 8.0.0 (#3735)JackLivio2021-07-1566-577/+7
| | | | | | | | | | | | | | * Remove deprecated methods * Style * Fix external policy build
| * Coverity Github Actions CI (#3737)JackLivio2021-07-151-0/+63
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Add Coverity CI * Fix syntax * test * test * Add enviroment * Add env * Upload coverity result * Apply only for master branch prs
| * Feature/Update Minimum Supported Version to Ubuntu 18 (#3727)Shobhit Adlakha2021-07-0927-100/+101
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Remove references to ubuntu 16 in documentation * Remove older ssl ciphers from crypto manager * Fix style * Change logs for unsupported protocols * Update clang-format version to 8 and re-run style script * Apply suggestions from code review Co-authored-by: Collin <iCollin@users.noreply.github.com> * Drop support for openssl 1.0 * Remove some references to OPENSSL_1_1_VERSION Co-authored-by: Collin <iCollin@users.noreply.github.com>
| * Add missing fields to PT Snapshot (#1537)Sergii Levchenko2021-07-082-2/+36
| |
| * enable MessageHelper::SendGetUserFriendlyMessageResponse for all policy ↵Collin2021-07-061-4/+0
| | | | | | | | modes (#3723)
| * Feature/Add Unit test cases for GetSystemCapability (#3729)Shobhit Adlakha2021-07-021-0/+194
| | | | | | | | | | | | | | * Add additional test cases * Add additional test cases and fix style * Address review comments
| * remove deprecated/unused ini files (#3731)Collin2021-06-303-904/+0
| |
| * Fix/remove hmi ptu decryption flag (#3728)Collin2021-06-293-50/+0
| | | | | | | | | | | | | | * remove references to ENABLE_HMI_PTU_DECRYPTION and USE_HMI_PTU_DECRYPTION * merge policy manager parse and parsearray methods * remove references from CMakeLists.txt
| * Fix crash during resumption callback (#3721)Jacob Keeler2021-06-233-3/+28
| | | | | | | | | | * Fix invalid read in `send_response` callback * Add check in FinalizeResumption that app is still registered
| * Fix boost logger on release build (#3724)JackLivio2021-06-182-0/+5
| | | | | | | | | | * Fix boost logger on release build * Exclude boost logger file when logs disabled
| * check for invalid enum result code in CommandRequestImpl::SendResponse (#3718)Collin2021-06-144-3/+8
| | | | | | | | | | | | | | | | | | * check for invalid enum result code in CommandRequestImpl::SendResponse * Update src/components/application_manager/src/commands/command_request_impl.cc Co-authored-by: Jacob Keeler <jacob.keeler@livioradio.com> Co-authored-by: Jacob Keeler <jacob.keeler@livioradio.com>
| * Update bson version in 3rd party CMake (#3714)Jacob Keeler2021-06-081-1/+1
| |
| * destroy websocket session on AsyncRead error (#3696)Collin2021-06-081-0/+2
| | | | | | Co-authored-by: JackLivio <jack@livio.io>
| * Fix thread stop method (#3712)Shobhit Adlakha2021-06-011-1/+1
| |
| * Feature/GitHub actions ci (#3701)JackLivio2021-05-215-37/+54
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Test github action * Build Core * Fix path * Add submodules * Move path * Debug policy cmake paths * Debug lines * Change build directory * Print sources * Change core path * ls debug * Log cmake version * Try ubuntu 18 * Set Cmake version * Change cmake depdency * Remove version wildcard * Debug collect sources * More debug messages * Check filtered files * Remove bad policy path exclude * Try caching 3rd party * Fix env var * Update 3rd party path * Fix path * Remove env var * Fix space * Update path * Update core path * Fix boost install path * Add 3rd party arch path * Set LD_LIBRARY_PATH * Add check style * Fix cache path * Update README badge status * Move 3rd party build path * Remove Travis * Move cache call earlier * Fix syntax * Conditional mkdir * Add debug line * Add logs * Fix syntax * Fix syntax * Remove log lines * Use boost logger * Update cache * Remove -j * Top level boost include * Adjust boost includes * Remove boost logger * Invalidate cache * Update CMakeLists.txt * Update CMakeLists.txt * Update CMakeLists.txt * Update CMakeLists.txt * Update sdl_core_github_ci.yml * Update CMakeLists.txt * Update CMakeLists.txt * Update CMakeLists.txt * Add pull request action
| * manage schema objects in each API class to allow for recursive objects in ↵Collin2021-05-216-38/+84
| | | | | | | | the API without leaking memory (#3703)
| * Fix saving to endpoint properties table (#3704)Jacob Keeler2021-05-204-10/+22
| | | | | | `EndpointProperty.version` is defined as an optional value, but was being used as a mandatory one. Also makes `service` property unique in `endpoint_properties` to prevent duplicate entries
| * when non-existent app sends request reply app not registered (#3705)Collin2021-05-202-0/+9
| | | | | | | | | | * when non-existant app sends request reply app not registered * fix unit tests
| * Remove logic sending invalid slider position (#3697)Jacob Keeler2021-05-141-15/+0
| | | | | | sliderPosition was set to 0 (an invalid value) in the mobile response if an ABORTED result was received from the HMI without this parameter.
| * Append LD_LIBRARY_PATH in daemon.sh (#3702)JackLivio2021-05-131-1/+1
| | | | | | | | | | | | | | | | | | * Append LD_LIBRARY_PATH in daemon.sh * Update src/appMain/daemon.sh Co-authored-by: Jacob Keeler <jacob.keeler@livioradio.com> Co-authored-by: Jacob Keeler <jacob.keeler@livioradio.com>
| * Only call Json::Value::operator[] if PTU value is object type (#3699)Collin2021-05-131-1/+1
| | | | | | | | | | | | | | * only call Json::Value::operator[] if it is object type * short object check * validate value[data].isArray()
| * Fixed vrSynonyms and ttsName content in SystemRequest(QUERY_APPS) (#2705)Yurii Postolov2021-05-111-1/+14
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Fixed SDL sends appName in vrSynonyms and ttsName in case of lower and upper bound values in json file * Update src/components/application_manager/src/application_manager_impl.cc Co-authored-by: Shobhit Adlakha <ShobhitAd@users.noreply.github.com> * Update src/components/application_manager/src/application_manager_impl.cc Co-authored-by: Shobhit Adlakha <ShobhitAd@users.noreply.github.com> * ttsName is defined as array Co-authored-by: Dmitriy Boltovskiy <dboltovskyi@luxoft.com> Co-authored-by: Shobhit Adlakha <ShobhitAd@users.noreply.github.com> Co-authored-by: yurii <ypostolov>
| * Remove some unused methods from MediaManager (#2774)Stanislav Kobziev (GitHub)2021-05-113-19/+4
| | | | | | | | | | | | | | * Deprecated redundant code * Remove unit tests for deprecated methods Co-authored-by: Andrii Kalinich <AKalinich@luxoft.com>
| * Add mobile response with disallowed by user params (#2499)Pavel Zhdanov (GitHub)2021-05-075-114/+231
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Add mobile responce with disallowed by user params * Answer PR comments * Answer PR comments * Update fix according to new hierarchy * fixup! Update fix according to new hierarchy * fixup! Update fix according to new hierarchy Co-authored-by: Andrii Kalinich <AKalinich@luxoft.com> Co-authored-by: Dmitriy Boltovskiy <dboltovskyi@luxoft.com>
| * reorder shutdown calls to prevent deadlock (#3691)Collin2021-05-061-3/+3
| |
| * Fix core prematurely sends success for delete interaction choice set (#2601)Igor Gapchuk (GitHub)2021-05-043-21/+140
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Fix core prematurely sends success for DeleteInteractionChoiceSet. DeleteInteractionChoiceSetRequest was contain implementation, according to that, SDL never not waits for HMI response and sends to Mobile response (success = true, resultCode = "SUCCESS"). This PR provides changes, that remove that implementation. Also, according to requirements, when at-least on of HMI VR_DeleteCommand response will contains any ERROR result code, except WARNINGS, SDL has to send response with that error code and success_result as false. And if on of HMI VR_DeleteCommand response will contains at-least one WARNINGS resulst_code, SDL has to send to Mobile response with WARNINGS and success_result as true. * Address review comments * fixup! Address review comments * Update src/components/application_manager/rpc_plugins/sdl_rpc_plugin/src/commands/mobile/delete_interaction_choice_set_request.cc Co-authored-by: Collin <iCollin@users.noreply.github.com> Co-authored-by: Andrii Kalinich <AKalinich@luxoft.com> Co-authored-by: Collin <iCollin@users.noreply.github.com>
| * Merge remote-tracking branch 'origin/master' into developJackLivio2021-05-041-2/+2
| |\
| | * Merge pull request #3695 from smartdevicelink/fix/boost_download_url7.1.1JackLivio2021-05-031-2/+2
| | |\ | | | | | | | | Release 7.1.1
| | | * Update Boost Download URLJackLivio2021-05-031-2/+2
| | |/
| * | Feature/Boost Logger Implementation (#3571)Shobhit Adlakha2021-04-3035-45/+623
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Implement 3rd-party Boost logger * Add boost log config file * Implement log filtering and formatting for the different sinks * Fix Timestamp calculation * Fix function trace string * Implement setting to disable particular sinks * Add append option to file sinks * Telnet logger configuration * Add cmake flag to switch beetween loggers * Remove log4cxx appender include * Style fix * Modify BoostLogConfig file for tests to match the log4cxx properties file * Exclude log4cxx related libraries * Address review comments * Add changes to only install boost log if the correct LOGGER_NAME is set * Fix implementation for removing disabled sinks * Add note for isEnabledFor function * Address review comment * Fix smartobject linking error * Address review comments
| * | Open INI file less (#3688)Collin2021-04-269-69/+221
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * parse all values from one ini file open * wrap ini_read_value to open file for ini_file_test.cc * update profile tests, fix handle leak * refactor ini_parse_line * Apply suggestions from code review Co-authored-by: Jacob Keeler <jacob.keeler@livioradio.com> * parse ini file in to smart object * remove ini_file changes * fix style * small optimization, parse key first * restore open/close config functions * deprecate ini_file * Revert "restore open/close config functions" This reverts commit a03c69403fe14790e6b6738a65820c1ae69e95c5. * move ini parsing to Profile::ParseConfiguration * add DEPREACTED to ini_file * don't print deprecated warnings from test file that is dedicated to deprecated methods * add macros to ini_file.cc * check style Co-authored-by: Jacob Keeler <jacob.keeler@livioradio.com>
| * | fix crash in libusb callback handling (#2505)Yurii Postolov2021-04-261-0/+14
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * fix crash in libusb callback handling * Answer review; removing redundant logger * Respond review comment, fixed logger. * Removed usb_connection.cc.orig * Update src/components/transport_manager/src/usb/libusb/usb_connection.cc Co-authored-by: Collin <iCollin@users.noreply.github.com> * Update src/components/transport_manager/src/usb/libusb/usb_connection.cc Co-authored-by: Collin <iCollin@users.noreply.github.com> Co-authored-by: yurii <ypostolov> Co-authored-by: Collin <iCollin@users.noreply.github.com>
| * | Store resumption data in storage folder (#3686)Jacob Keeler2021-04-225-16/+40
| | | | | | | | | | | | | | | | | | | | | * fix: store app info JSON file in AppStorageFolder * Update test cases to store app info JSON file in AppStorageFolder Co-authored-by: Sho Amano <samano@xevo.com>
| * | fix respond on SDL.GetListOfPermission request (#3627)Vadym Luchko (GitHub)2021-04-229-57/+127
| | | | | | | | | | | | | | | | | | | | | * fix respond on SDL.GetListOfPermission request * Error response in case when we could not collect permissions Co-authored-by: Jacob Keeler <jacob.keeler@livioradio.com>
| * | Fix/Return requesType in sample policy manager response (#3690)Shobhit Adlakha2021-04-211-1/+5
| | | | | | | | | | | | | | | * Add change to return response with requestType from the sample_policy_manager * Remove unrequired logs
| * | SDL sends wrong OnHMIStatus for voice app in case of a phone call. (#2582)Mykola Korniichuk (GitHub)2021-04-212-2/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * SDL sends wrong OnHMIStatus for voice app in case of a phone call. Once SDL received OnEventChanged(Phone_Call, true) from HMI, it should send OnHMIStatusUpdate with new HMILevel. Expected HMILevel for voice communication application is "LIMITED", but it is "FULL". PhoneCallHmiState should take into account voice communication app as an additional check. * Fix for state_controller unit tests. Due to changes in a logic for PhoneCallHmiState, unit tests have to be updated as well. ApplicationType enum is removed since it is not actual anymore. As a result, PrepareHMIStateResults functions are modified according to new application types and updated expected HMI states. Tests for NaviStreaming HMI state are enabled. NaviStreamingHmiState is used for these tests instead of VideoStreamingHmiState. New tests for different application types are added. * fix styling issues for state_controller_test * Fix hmi state merge conflicts * Revert test code * Fix unit test * Style Co-authored-by: JackLivio <jack@livio.io>
| * | replace applicable post-operators to pre-operators (#3687)Collin2021-04-2042-91/+100
| | | | | | | | | | | | | | | | | | | | | * replace applicable X++ with ++X * replace applicable X-- with --X * change for loop updates
| * | Fix SDL respond UNSUPPORTED_RESOURCE on SystemRequest (QUERY_APPS) (#2502)Yurii Postolov2021-04-161-1/+9
| | | | | | | | | | | | | | | | | | | | | * SDL respond UNSUPPORTED_RESOURCE on SystemRequest (QUERY_APPS) if SDL is below version 4.0 * answer review comments Co-authored-by: Collin <iCollin@users.noreply.github.com>
| * | Update travis distro to Ubuntu 20.04 (#3626)Jacob Keeler2021-04-153-5/+5
| | | | | | | | | | | | | | | | | | | | | * Update travis distro to 20.04 * Add cppcheck exclusions for false positives * Fix cppcheck issues
| * | Format hex dump in control_message_matcher (#2311)Sho Amano2021-04-151-1/+3
| | | | | | | | | Co-authored-by: Jacob Keeler <jacob.keeler@livioradio.com>
| * | Change default HeartBeat value (#2513)Anna Pipko (GitHub)2021-04-152-3/+3
| | | | | | | | | | | | | | | kDefaultHeartBeatTimeout = 5000ms Co-authored-by: Anna Pipko (GitHub) <APipko@luxoft.com>
| * | Specify a dedicated folder for icon storage (#2202)Sho Amano2021-04-151-1/+2
| | |
| * | fix: reduce logs during streaming (#1778)Sho Amano2021-04-1412-45/+45
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Reduce some logs to TRACE level Each time SDL Core receives a video packet from mobile, it generates a large amount of logs. Instead of completely removing them, this commit decreases their levels to TRACE. Head unit systems with video streaming feature can set log output level to DEBUG (or higher) to get rid of the logs. Co-authored-by: Jacob Keeler <jacob.keeler@livioradio.com>
| * | Set initialize state for the Interger type (#2548)Pavel Zhdanov (GitHub)2021-04-140-0/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The default constructor of Interger class assigns the min value for the wrapped variable, hoowever it keeps the initialize state us `Uninitialized`. So in case when latter user apply such operations as `++`, `+=` the value will be changed but the state is not. The commit adds ability to change initialize state for the class after mentioned operations. So user does not have to call additionaly `initialize` method.