Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

SDL removes the current "consumer_friendly_messages" section in Local PT in case the Updated PT omits "consumer_friendly_messages" section #3846

Closed
VjKlepikov opened this issue Jan 21, 2022 · 5 comments
Labels
Projects

Comments

@VjKlepikov
Copy link

Bug Report

SDL removes the current "consumer_friendly_messages" section in Local PT in case the Updated PT omits "consumer_friendly_messages" section

Reproduction Steps
  1. Make sure SDL is built with HTTP flow
  2. Register new application => PTU is triggered
  3. PTU omits "consumer_friendly_messages" section
Expected Behavior

SDL maintains the current "consumer_friendly_messages" section in Local PT

Observed Behavior

SDL removes the current "consumer_friendly_messages" section in Local PT in case the Updated PT omits "consumer_friendly_messages" section

OS & Version Information
Test Case, Sample Code, and / or Example App
  • Tests scripts: *./test_scripts/Policies/build_options/070_ATF_PTU_Merge_Into_Local_PT_ConsumerFriendlyMessages_Omitted_HTTP.lua
Logs: Logs.zip
@VjKlepikov
Copy link
Author

Please notice this is a regression issue that became reproducible after merge of #3842
@theresalech Can you please advise if the Livio team is planning on submitting a PR to fix?

@theresalech
Copy link
Contributor

@VjKlepikov our team will review the issue and advise.

@jacobkeeler
Copy link
Contributor

@VjKlepikov can you confirm if this issue only occurs in HTTP mode?

@jacobkeeler
Copy link
Contributor

After testing, it appears that this issue is reproducible in master if OnDriverDistraction(DD_ON) is sent prior to registering the app. It seems likely that the issue lies in this section of code

@jacobkeeler
Copy link
Contributor

closed via #3852

@theresalech theresalech added this to Features in 8.1.0 via automation Feb 7, 2022
@theresalech theresalech moved this from Features to Completed Bug Fixes/ Enhancements in 8.1.0 Feb 7, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
8.1.0
Completed Bug Fixes/ Enhancements
Development

No branches or pull requests

4 participants