Update ECU definition from internet seems to be broken at least on Windows

Post Reply
User avatar
AndreyB
Site Admin
Posts: 14334
Joined: Wed Aug 28, 2013 1:28 am
Location: Jersey City
Github Username: rusefillc
Slack: Andrey B

Update ECU definition from internet seems to be broken at least on Windows

Post by AndreyB »

TS no longer downloads fresh .ini files automatically for me, can someone who uses Windows confirm or deny having same issue?

I believe this is about TS having too old version of JRE 1.8.0_66 from back 2015 emended. I've actually replaced cacerts file with a file from a fresh 1.8 JRE instance and got the issue resolved.

https://github.com/rusefi/rusefi/issues/5536
Very limited telepathic abilities - please post logs & tunes where appropriate - http://rusefi.com/s/questions

Always looking for C/C++/Java/PHP developers! Please help us see https://rusefi.com/s/howtocontribute
User avatar
AndreyB
Site Admin
Posts: 14334
Joined: Wed Aug 28, 2013 1:28 am
Location: Jersey City
Github Username: rusefillc
Slack: Andrey B

Re: Update ECU definition from internet seems to be broken at least on Windows

Post by AndreyB »

PS: corresponding log line in TunerStudioAppDebug.txt is

Code: Select all

Debug: Failed to get ini: sun.security.validator.ValidatorException: PKIX path building failed: sun.security.provider.certpath.SunCertPathBuilderException: unable to find valid certification path to requested target
Very limited telepathic abilities - please post logs & tunes where appropriate - http://rusefi.com/s/questions

Always looking for C/C++/Java/PHP developers! Please help us see https://rusefi.com/s/howtocontribute
User avatar
AndreyB
Site Admin
Posts: 14334
Joined: Wed Aug 28, 2013 1:28 am
Location: Jersey City
Github Username: rusefillc
Slack: Andrey B

Re: Update ECU definition from internet seems to be broken at least on Windows

Post by AndreyB »

Oh right, it's the annual rusefi.com certificate update and 25% of the 1.8.0_66 CA certificates have already expired.
image.png
image.png (16.34 KiB) Viewed 36055 times
Very limited telepathic abilities - please post logs & tunes where appropriate - http://rusefi.com/s/questions

Always looking for C/C++/Java/PHP developers! Please help us see https://rusefi.com/s/howtocontribute
User avatar
NormanAlphaspeed
Posts: 64
Joined: Fri Jan 13, 2017 7:15 am
Location: Puerto Rico
Contact:

Re: Update ECU definition from internet seems to be broken at least on Windows

Post by NormanAlphaspeed »

Untitled.jpeg
Untitled.jpeg (199.53 KiB) Viewed 36013 times
Untitled.jpeg
Untitled.jpeg (231.83 KiB) Viewed 36013 times
large amount of end users in panic because choosing ini file is too difficult
User avatar
NormanAlphaspeed
Posts: 64
Joined: Fri Jan 13, 2017 7:15 am
Location: Puerto Rico
Contact:

Re: Update ECU definition from internet seems to be broken at least on Windows

Post by NormanAlphaspeed »

More reports of angry people
image.png
image.png (2.95 MiB) Viewed 35897 times
User avatar
AndreyB
Site Admin
Posts: 14334
Joined: Wed Aug 28, 2013 1:28 am
Location: Jersey City
Github Username: rusefillc
Slack: Andrey B

Re: Update ECU definition from internet seems to be broken at least on Windows

Post by AndreyB »

Very limited telepathic abilities - please post logs & tunes where appropriate - http://rusefi.com/s/questions

Always looking for C/C++/Java/PHP developers! Please help us see https://rusefi.com/s/howtocontribute
mck1117
running engine in first post
running engine in first post
Posts: 1494
Joined: Mon Jan 30, 2017 2:05 am
Location: Seattle-ish

Re: Update ECU definition from internet seems to be broken at least on Windows

Post by mck1117 »

AndreyB wrote:
Sun Sep 24, 2023 10:37 pm
Work-around is now documented at https://github.com/rusefi/rusefi/wiki/JRE#tunerstudio-cacerts-work-around
Are you recommending that users install a custom certificate authority globally for all Java apps? That's TERRIBLE security advice. You know better than that.
User avatar
AndreyB
Site Admin
Posts: 14334
Joined: Wed Aug 28, 2013 1:28 am
Location: Jersey City
Github Username: rusefillc
Slack: Andrey B

Re: Update ECU definition from internet seems to be broken at least on Windows

Post by AndreyB »

mck1117 wrote:
Sun Sep 24, 2023 10:58 pm
globally for all Java apps?
Please see where you are wrong.

On my end let me spell out source of cacerts file :)
Very limited telepathic abilities - please post logs & tunes where appropriate - http://rusefi.com/s/questions

Always looking for C/C++/Java/PHP developers! Please help us see https://rusefi.com/s/howtocontribute
mck1117
running engine in first post
running engine in first post
Posts: 1494
Joined: Mon Jan 30, 2017 2:05 am
Location: Seattle-ish

Re: Update ECU definition from internet seems to be broken at least on Windows

Post by mck1117 »

So it is just for TS, but it's still a risk. TS is using that file for downloading updates to TS, which then run as administrator on your PC. So you've now installed a CA (if only to TS) that can serve files that are expected to grant admin rights.
mck1117
running engine in first post
running engine in first post
Posts: 1494
Joined: Mon Jan 30, 2017 2:05 am
Location: Seattle-ish

Re: Update ECU definition from internet seems to be broken at least on Windows

Post by mck1117 »

Can you explain the actual issue that requires adding a CA?
User avatar
AndreyB
Site Admin
Posts: 14334
Joined: Wed Aug 28, 2013 1:28 am
Location: Jersey City
Github Username: rusefillc
Slack: Andrey B

Re: Update ECU definition from internet seems to be broken at least on Windows

Post by AndreyB »

https://github.com/rusefi/rusefi/wiki/JRE#tunerstudio-cacerts-work-around clarifies source of cacerts file, thank you for your feedback.

Matt please read github ticket in first post and let me know if I can clarify anything beyond what is already documented.
Very limited telepathic abilities - please post logs & tunes where appropriate - http://rusefi.com/s/questions

Always looking for C/C++/Java/PHP developers! Please help us see https://rusefi.com/s/howtocontribute
User avatar
AndreyB
Site Admin
Posts: 14334
Joined: Wed Aug 28, 2013 1:28 am
Location: Jersey City
Github Username: rusefillc
Slack: Andrey B

Re: Update ECU definition from internet seems to be broken at least on Windows

Post by AndreyB »

rusEFI bundles now contain 'update-ts-cacerts' folder with fixts.bat
Very limited telepathic abilities - please post logs & tunes where appropriate - http://rusefi.com/s/questions

Always looking for C/C++/Java/PHP developers! Please help us see https://rusefi.com/s/howtocontribute
User avatar
NormanAlphaspeed
Posts: 64
Joined: Fri Jan 13, 2017 7:15 am
Location: Puerto Rico
Contact:

Re: Update ECU definition from internet seems to be broken at least on Windows

Post by NormanAlphaspeed »

AndreyB wrote:
Sat Oct 14, 2023 2:50 pm
rusEFI bundles now contain 'update-ts-cacerts' folder with fixts.bat
~credit to norman for writing a crappy batch file
Post Reply